diff options
author | Ajit Khaparde <ajitk@serverengines.com> | 2010-09-03 06:24:13 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-09-03 09:52:47 -0700 |
commit | 323f30b361b2e6febb9065ab4a1a5298acb66ac1 (patch) | |
tree | 0fb7d8ad95b2fbc120754485f27836f5165eae2b | |
parent | d053de911bff69ba7cdda36d3107c1da0fba7ccd (diff) | |
download | op-kernel-dev-323f30b361b2e6febb9065ab4a1a5298acb66ac1.zip op-kernel-dev-323f30b361b2e6febb9065ab4a1a5298acb66ac1.tar.gz |
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>
-rw-r--r-- | drivers/net/benet/be_cmds.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/benet/be_cmds.c b/drivers/net/benet/be_cmds.c index 78f32fe..34abcc9 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); |