* [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c
@ 2010-08-25 10:31 Ajit Khaparde
2010-08-25 21:23 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Ajit Khaparde @ 2010-08-25 10:31 UTC (permalink / raw)
To: David Miller; +Cc: netdev
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>
---
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 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);
--
1.7.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c
2010-08-25 10:31 Ajit Khaparde
@ 2010-08-25 21:23 ` David Miller
0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2010-08-25 21:23 UTC (permalink / raw)
To: ajitk; +Cc: netdev
From: Ajit Khaparde <ajitk@serverengines.com>
Date: Wed, 25 Aug 2010 16:01:20 +0530
> 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>
Applied.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c
@ 2010-09-03 16:24 Ajit Khaparde
2010-09-03 16:54 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Ajit Khaparde @ 2010-09-03 16:24 UTC (permalink / raw)
To: David Miller; +Cc: netdev
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>
---
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);
--
1.7.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c
2010-09-03 16:24 [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c Ajit Khaparde
@ 2010-09-03 16:54 ` David Miller
0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2010-09-03 16:54 UTC (permalink / raw)
To: ajitk; +Cc: netdev
From: Ajit Khaparde <ajitk@serverengines.com>
Date: Fri, 3 Sep 2010 21:54:13 +0530
> 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>
Applied.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-09-03 16:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-03 16:24 [PATCH net-2.6 3/3] be2net: remove a BUG_ON in be_cmds.c Ajit Khaparde
2010-09-03 16:54 ` David Miller
-- strict thread matches above, loose matches on Subject: below --
2010-08-25 10:31 Ajit Khaparde
2010-08-25 21:23 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).