]> err.no Git - linux-2.6/commit
[IPSEC] xfrm: Undo afinfo lock proliferation
authorHerbert Xu <herbert@gondor.apana.org.au>
Sun, 28 May 2006 06:03:58 +0000 (23:03 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Sun, 18 Jun 2006 04:28:37 +0000 (21:28 -0700)
commit546be2405be119ef55467aace45f337a16e5d424
tree9b09f0041f9f82a20ab25ace3c7360e4a4b7989f
parent9cb3528cdbffc513eb9fb8faa45d41e397355830
[IPSEC] xfrm: Undo afinfo lock proliferation

The number of locks used to manage afinfo structures can easily be reduced
down to one each for policy and state respectively.  This is based on the
observation that the write locks are only held by module insertion/removal
which are very rare events so there is no need to further differentiate
between the insertion of modules like ipv6 versus esp6.

The removal of the read locks in xfrm4_policy.c/xfrm6_policy.c might look
suspicious at first.  However, after you realise that nobody ever takes
the corresponding write lock you'll feel better :)

As far as I can gather it's an attempt to guard against the removal of
the corresponding modules.  Since neither module can be unloaded at all
we can leave it to whoever fixes up IPv6 unloading :)

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/xfrm.h
net/ipv4/xfrm4_policy.c
net/ipv4/xfrm4_state.c
net/ipv6/xfrm6_policy.c
net/ipv6/xfrm6_state.c
net/xfrm/xfrm_policy.c
net/xfrm/xfrm_state.c