diff options
author | Yuval Mintz <Yuval.Mintz@qlogic.com> | 2015-06-01 15:08:18 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-06-01 12:04:31 -0700 |
commit | c6e36d8c1a76be7a7afa2669483857dadec1e99c (patch) | |
tree | 6ba06f3353eaff41b0a9ccc01d9418033a3e4ed5 /drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | |
parent | xen: netback: read hotplug script once at start of day. (diff) | |
download | linux-dev-c6e36d8c1a76be7a7afa2669483857dadec1e99c.tar.xz linux-dev-c6e36d8c1a76be7a7afa2669483857dadec1e99c.zip |
bnx2x: Move statistics implementation into semaphores
Commit dff173de84958 ("bnx2x: Fix statistics locking scheme") changed the
bnx2x locking around statistics state into using a mutex - but the lock
is being accessed via a timer which is forbidden.
[If compiled with CONFIG_DEBUG_MUTEXES, logs show a warning about
accessing the mutex in interrupt context]
This moves the implementation into using a semaphore [with size '1']
instead.
Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: Ariel Elior <Ariel.Elior@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/bnx2x/bnx2x.h')
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x.h b/drivers/net/ethernet/broadcom/bnx2x/bnx2x.h index a3b0f7a0c61e..1f82a04ce01a 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x.h +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x.h @@ -1774,7 +1774,7 @@ struct bnx2x { int stats_state; /* used for synchronization of concurrent threads statistics handling */ - struct mutex stats_lock; + struct semaphore stats_lock; /* used by dmae command loader */ struct dmae_command stats_dmae; |