diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2008-04-21 21:12:33 -0400 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2008-04-22 01:37:23 -0400 |
commit | 618442509128fe4514be94de70ce54075cd9a706 (patch) | |
tree | 9aea814978791abd8c4f9a5c60de879b2811c063 /security/selinux/netif.c | |
parent | 0f5e64200f20fc8f5b759c4010082f577ab0af3f (diff) |
SELinux fixups needed for preemptable RCU from -rt
The attached patch needs to move from -rt to mainline given preemptable RCU.
This patch fixes SELinux code that implicitly assumes that disabling
preemption prevents an RCU grace period from completing, an assumption that
is valid for Classic RCU, but not necessarily for preemptable RCU. Explicit
rcu_read_lock() calls are thus added.
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Acked-by: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/selinux/netif.c')
-rw-r--r-- | security/selinux/netif.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/security/selinux/netif.c b/security/selinux/netif.c index c658b84c319..b4e14bc0bf3 100644 --- a/security/selinux/netif.c +++ b/security/selinux/netif.c | |||
@@ -239,11 +239,13 @@ static void sel_netif_kill(int ifindex) | |||
239 | { | 239 | { |
240 | struct sel_netif *netif; | 240 | struct sel_netif *netif; |
241 | 241 | ||
242 | rcu_read_lock(); | ||
242 | spin_lock_bh(&sel_netif_lock); | 243 | spin_lock_bh(&sel_netif_lock); |
243 | netif = sel_netif_find(ifindex); | 244 | netif = sel_netif_find(ifindex); |
244 | if (netif) | 245 | if (netif) |
245 | sel_netif_destroy(netif); | 246 | sel_netif_destroy(netif); |
246 | spin_unlock_bh(&sel_netif_lock); | 247 | spin_unlock_bh(&sel_netif_lock); |
248 | rcu_read_unlock(); | ||
247 | } | 249 | } |
248 | 250 | ||
249 | /** | 251 | /** |