diff options
author | Ajit Khaparde <ajitk@serverengines.com> | 2010-07-29 02:18:58 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-07-31 02:59:06 -0400 |
commit | 0fc48c37ff3969dde71a43fa7c8f176d4bd90a3e (patch) | |
tree | 5d2d400a09be92fde256ba2501fcd5ea426d0179 /drivers/net/benet/be_main.c | |
parent | 48e9989e033966fd738d062ea9730fe10085fdd1 (diff) |
be2net: fix to avoid sending get_stats request if one is already being processed.
GET_STATS request uses the same memory region as the response.
If a new request for get stats is fired before the response for
the previous get_stats request is received, the response will
corrupt the new request, causing the f/w to misbehave.
Signed-off-by: Somnath K <somnathk@serverengines.com>
Signed-off-by: Ajit Khaparde <ajitk@serverengines.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/benet/be_main.c')
-rw-r--r-- | drivers/net/benet/be_main.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/benet/be_main.c b/drivers/net/benet/be_main.c index e4a8ae3a1c8d..74e146f470c6 100644 --- a/drivers/net/benet/be_main.c +++ b/drivers/net/benet/be_main.c | |||
@@ -1801,7 +1801,8 @@ static void be_worker(struct work_struct *work) | |||
1801 | struct be_adapter *adapter = | 1801 | struct be_adapter *adapter = |
1802 | container_of(work, struct be_adapter, work.work); | 1802 | container_of(work, struct be_adapter, work.work); |
1803 | 1803 | ||
1804 | be_cmd_get_stats(adapter, &adapter->stats.cmd); | 1804 | if (!adapter->stats_ioctl_sent) |
1805 | be_cmd_get_stats(adapter, &adapter->stats.cmd); | ||
1805 | 1806 | ||
1806 | /* Set EQ delay */ | 1807 | /* Set EQ delay */ |
1807 | be_rx_eqd_update(adapter); | 1808 | be_rx_eqd_update(adapter); |