This is much better. Sorry for the confusion. With a clear comment like this, I don't think it'll happen again.
thanks, Catherine On 1/17/06, Herbert Xu <[EMAIL PROTECTED]> wrote: > On Tue, Jan 17, 2006 at 06:29:26PM -0800, David S. Miller wrote: > > > > There is a big comment in __xfrm_state_delete(), would you like > > something more? :-) > > How about something like this? This should make someone think twice > before treating it as a duplicate. > > [IPSEC]: Turn first xfrm_state_put into atomic_dec > > We need to do two xfrm_state_put's in a row in order to kill a state > that's not linked into the system. Since we know that the first put > is not going to free the structure, we can turn it into an atomic_dec. > > This should also tell people that the two put's are intentional and > not a bug. > > Signed-off-by: Herbert Xu <[EMAIL PROTECTED]> > > Cheers, > -- > Visit Openswan at http://www.openswan.org/ > Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> > Home Page: http://gondor.apana.org.au/~herbert/ > PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt > > > - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
