diff options
author | Monam Agarwal <monamagarwal123@gmail.com> | 2014-03-23 15:12:46 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-03-27 00:18:09 -0400 |
commit | fcb144b5df260d5005165589f958594facf1d6ae (patch) | |
tree | 955b894ce2971a5e38af4879f0365de3f4e7868b /net/core/netpoll.c | |
parent | cd18721e527270f9afa5c8daabfb6403b01e3a92 (diff) |
net/core: Use RCU_INIT_POINTER(x, NULL) in netpoll.c
This patch replaces rcu_assign_pointer(x, NULL) with RCU_INIT_POINTER(x, NULL)
The rcu_assign_pointer() ensures that the initialization of a structure
is carried out before storing a pointer to that structure.
And in the case of the NULL pointer, there is no structure to initialize.
So, rcu_assign_pointer(p, NULL) can be safely converted to RCU_INIT_POINTER(p, NULL)
Signed-off-by: Monam Agarwal <monamagarwal123@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/netpoll.c')
-rw-r--r-- | net/core/netpoll.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/core/netpoll.c b/net/core/netpoll.c index 7291dde93469..41c4e9ce1141 100644 --- a/net/core/netpoll.c +++ b/net/core/netpoll.c | |||
@@ -809,7 +809,7 @@ void __netpoll_cleanup(struct netpoll *np) | |||
809 | if (ops->ndo_netpoll_cleanup) | 809 | if (ops->ndo_netpoll_cleanup) |
810 | ops->ndo_netpoll_cleanup(np->dev); | 810 | ops->ndo_netpoll_cleanup(np->dev); |
811 | 811 | ||
812 | rcu_assign_pointer(np->dev->npinfo, NULL); | 812 | RCU_INIT_POINTER(np->dev->npinfo, NULL); |
813 | call_rcu_bh(&npinfo->rcu, rcu_cleanup_netpoll_info); | 813 | call_rcu_bh(&npinfo->rcu, rcu_cleanup_netpoll_info); |
814 | } | 814 | } |
815 | } | 815 | } |