diff options
| author | Jesper Dangaard Brouer <hawk@comx.dk> | 2009-06-08 03:11:43 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-06-10 01:11:25 -0700 |
| commit | eaa184a1a1cda3991420c47bc960ddea6cbacff2 (patch) | |
| tree | 609a773a114cf2efee117354c66ba2b8da811a90 /net/sctp/protocol.c | |
| parent | 382bfeec48e43c416af9b271b9b70d2f61ef32d2 (diff) | |
| download | lwn-eaa184a1a1cda3991420c47bc960ddea6cbacff2.tar.gz lwn-eaa184a1a1cda3991420c47bc960ddea6cbacff2.zip | |
sctp: protocol.c call rcu_barrier() on unload.
On module unload call rcu_barrier(), this is needed as synchronize_rcu()
is not strong enough. The kmem_cache_destroy() does invoke
synchronize_rcu() but it does not provide same protection.
Signed-off-by: Jesper Dangaard Brouer <hawk@comx.dk>
Acked-by: Vlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp/protocol.c')
| -rw-r--r-- | net/sctp/protocol.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c index cb2c50dbd421..79cbd47f4df7 100644 --- a/net/sctp/protocol.c +++ b/net/sctp/protocol.c @@ -1370,6 +1370,8 @@ SCTP_STATIC __exit void sctp_exit(void) sctp_proc_exit(); cleanup_sctp_mibs(); + rcu_barrier(); /* Wait for completion of call_rcu()'s */ + kmem_cache_destroy(sctp_chunk_cachep); kmem_cache_destroy(sctp_bucket_cachep); } |