summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorAjit Khaparde2010-09-03 08:24:13 +0200
committerDavid S. Miller2010-09-03 18:52:47 +0200
commit323f30b361b2e6febb9065ab4a1a5298acb66ac1 (patch)
tree0fb7d8ad95b2fbc120754485f27836f5165eae2b /drivers
parentbe2net: fix a bug in UE detection logic (diff)
downloadkernel-qcow2-linux-323f30b361b2e6febb9065ab4a1a5298acb66ac1.tar.gz
kernel-qcow2-linux-323f30b361b2e6febb9065ab4a1a5298acb66ac1.tar.xz
kernel-qcow2-linux-323f30b361b2e6febb9065ab4a1a5298acb66ac1.zip
be2net: remove a BUG_ON in be_cmds.c
Async notifications other than link status are possible in certain configurations. Remove the BUG_ON in the mcc completion processing path. Signed-off-by: Ajit Khaparde <ajitk@serverengines.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/net/benet/be_cmds.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/benet/be_cmds.c b/drivers/net/benet/be_cmds.c
index 78f32fe68c0e..34abcc9403d6 100644
--- a/drivers/net/benet/be_cmds.c
+++ b/drivers/net/benet/be_cmds.c
@@ -140,10 +140,8 @@ int be_process_mcc(struct be_adapter *adapter, int *status)
while ((compl = be_mcc_compl_get(adapter))) {
if (compl->flags & CQE_FLAGS_ASYNC_MASK) {
/* Interpret flags as an async trailer */
- BUG_ON(!is_link_state_evt(compl->flags));
-
- /* Interpret compl as a async link evt */
- be_async_link_state_process(adapter,
+ if (is_link_state_evt(compl->flags))
+ be_async_link_state_process(adapter,
(struct be_async_event_link_state *) compl);
} else if (compl->flags & CQE_FLAGS_COMPLETED_MASK) {
*status = be_mcc_compl_process(adapter, compl);