Patchwork [net-2.6,3/6] be2net: remove a BUG_ON in be_cmds.c

login
register
mail settings
Submitter Ajit Khaparde
Date Aug. 22, 2010, 9:25 a.m.
Message ID <20100822092529.GA22268@serverengines.com>
Download mbox | patch
Permalink /patch/62364/
State Changes Requested
Delegated to: David Miller
Headers show

Comments

Ajit Khaparde - Aug. 22, 2010, 9:25 a.m.
Async notifications other than link status are possible in certain
configurations. Remove the BUG_ON.

Signed-off-by: Ajit Khaparde <ajitk@serverengines.com>
---
 drivers/net/benet/be_cmds.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

Patch

diff --git a/drivers/net/benet/be_cmds.c b/drivers/net/benet/be_cmds.c
index 3d30549..22bd062 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);