| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: RFC: IPSEC patch 0 for netlink events |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Tue, 29 Mar 2005 09:43:35 +1000 |
| Cc: | hadi@xxxxxxxxxx, Masahide NAKAMURA <nakam@xxxxxxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, netdev <netdev@xxxxxxxxxxx> |
| In-reply-to: | <424875C7.3080306@xxxxxxxxx> |
| References: | <1111864971.1092.904.camel@xxxxxxxxxxxxxxxx> <20050326194707.GA9872@xxxxxxxxxxxxxxxxxxx> <1111867875.1089.915.camel@xxxxxxxxxxxxxxxx> <20050327081848.GA13428@xxxxxxxxxxxxxxxxxxx> <1111950449.1089.938.camel@xxxxxxxxxxxxxxxx> <424875C7.3080306@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040907i |
On Mon, Mar 28, 2005 at 11:23:19PM +0200, Patrick McHardy wrote: > > To answer this question: no. xfrm_state_lock can be nested in > x->lock, but not the other way around. If you want to avoid that That's true for now but it's something that I'd like to change for 2.6.13. We should use the refcnt as much as possible and modify the locking so that the state lock is only used to guard the state of the state :) > the state changes below you, you could notify before insertion. Agreed. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | mv643xx(3/20): use MII library for ethtool functions, Dale Farnsworth |
|---|---|
| Next by Date: | mv643xx(5/20): update ppc7d platform for new mv643xx_eth platform data, Dale Farnsworth |
| Previous by Thread: | Re: RFC: IPSEC patch 0 for netlink events, Patrick McHardy |
| Next by Thread: | Re: RFC: IPSEC patch 0 for netlink events, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |