diff options
| author | Jeff Moyer <jmoyer@redhat.com> | 2005-06-22 22:04:55 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-06-22 22:04:55 -0700 |
| commit | 6ca4f65e6b390d09e1de7280cf9fd4f5d8e4b48b (patch) | |
| tree | 1ca8b0d0126c5ce1cd7ffc0fe0ccd80235f3307b /include/linux/netpoll.h | |
| parent | f31f5f051269746179b01017fc5e3dcf6b37c67e (diff) | |
| download | talos-obmc-linux-6ca4f65e6b390d09e1de7280cf9fd4f5d8e4b48b.tar.gz talos-obmc-linux-6ca4f65e6b390d09e1de7280cf9fd4f5d8e4b48b.zip | |
[NETPOLL]: Set poll_owner to -1 before unlocking in netpoll_poll_unlock()
This trivial patch moves the assignment of poll_owner to -1 inside of
the lock. This fixes a potential SMP race in the code.
Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/netpoll.h')
| -rw-r--r-- | include/linux/netpoll.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/netpoll.h b/include/linux/netpoll.h index c0d8b90c5202..449a4fde6587 100644 --- a/include/linux/netpoll.h +++ b/include/linux/netpoll.h @@ -53,8 +53,8 @@ static inline void netpoll_poll_lock(struct net_device *dev) static inline void netpoll_poll_unlock(struct net_device *dev) { if (dev->np) { - spin_unlock(&dev->np->poll_lock); dev->np->poll_owner = -1; + spin_unlock(&dev->np->poll_lock); } } |
