Hi Joe,
> On Thu, 2017-08-31 at 01:21 +0530, Ganapathi Bhat wrote:
> > Current driver prints dev_alloc_skb failures everytime while
> > submitting RX URBs. This failure might be frequent in some low
> > resource platforms. So, wait for a threshold failure count before
> > start priting the error. This change is a follow up for the 'commit
> > 7b368e3d15c3
> > ("mwifiex: resubmit failed to submit RX URBs in main thread")'
>
> []
>
> > diff --git a/drivers/net/wireless/marvell/mwifiex/usb.c
> > b/drivers/net/wireless/marvell/mwifiex/usb.c
> []
> > @@ -300,9 +300,16 @@ static int mwifiex_usb_submit_rx_urb(struct
> urb_context *ctx, int size)
> > if (card->rx_cmd_ep != ctx->ep) {
> > ctx->skb = dev_alloc_skb(size);
> > if (!ctx->skb) {
> > - mwifiex_dbg(adapter, ERROR,
> > - "%s: dev_alloc_skb failed\n", __func__);
> > + if (++card->rx_urb_failure_count >
> > + MWIFIEX_RX_URB_FAILURE_THRESHOLD) {
> > + mwifiex_dbg(adapter, ERROR,
> > + "%s: dev_alloc_skb failed, failure
> count = %u\n",
> > + __func__,
> > + card->rx_urb_failure_count);
> > + }
> > return -ENOMEM;
>
> Why not use a ratelimit?
Since this is for receive, the packets are from AP side and we cannot lower the rate from AP. On some low performance systems this change will be helpful.
Regards,
Ganapathi
