staging: brcm80211: bugfix for crash on heavy transmit traffic
authorRoland Vossen <rvossen@broadcom.com>
Mon, 21 Feb 2011 08:07:12 +0000 (09:07 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 23 Feb 2011 22:05:42 +0000 (14:05 -0800)
With heavy transmit traffic, once in a while (range 15mins-1hr)
a tx packet was added to a full transmit queue. Under certain
conditions an other packet in the queue gets bumped to make room
for the new packet. This is not considered an error condition, but
normal operation. Despite that, there was an ASSERT(0) that caused
the driver to oops. The ASSERT(0) has been removed. Driver was tested
afterwards.

Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c

index 91a2de2..b617b64 100644 (file)
@@ -5146,8 +5146,6 @@ wlc_prec_enq_head(struct wlc_info *wlc, struct pktq *q, struct sk_buff *pkt,
                                 tx_failed[WME_PRIO2AC(p->priority)].bytes,
                                 pkttotlen(wlc->osh, p));
                }
-
-               ASSERT(0);
                pkt_buf_free_skb(wlc->osh, p, true);
                wlc->pub->_cnt->txnobuf++;
        }