Linux-Wireless Archive on lore.kernel.org
 help / Atom feed
* [PATCH] libertas_tf: don't set URB_ZERO_PACKET on IN USB transfer
@ 2019-02-10 19:47 Lubomir Rintel
  2019-02-11 13:38 ` Steve deRosier
  0 siblings, 1 reply; 2+ messages in thread
From: Lubomir Rintel @ 2019-02-10 19:47 UTC (permalink / raw)
  To: Kalle Valo
  Cc: David S. Miller, linux-wireless, linux-kernel, Lubomir Rintel, stable

It doesn't make sense and the USB core warns on each submit of such
URB, easily flooding the message buffer with tracebacks.

Analogous issue was fixed in regular libertas driver in commit 6528d8804780
("libertas: don't set URB_ZERO_PACKET on IN USB transfer").

Cc: stable@vger.kernel.org
Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
---
 drivers/net/wireless/marvell/libertas_tf/if_usb.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/net/wireless/marvell/libertas_tf/if_usb.c b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
index 789337ea676a..6ede6168bd85 100644
--- a/drivers/net/wireless/marvell/libertas_tf/if_usb.c
+++ b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
@@ -433,8 +433,6 @@ static int __if_usb_submit_rx_urb(struct if_usb_card *cardp,
 			  skb_tail_pointer(skb),
 			  MRVDRV_ETH_RX_PACKET_BUFFER_SIZE, callbackfn, cardp);
 
-	cardp->rx_urb->transfer_flags |= URB_ZERO_PACKET;
-
 	lbtf_deb_usb2(&cardp->udev->dev, "Pointer for rx_urb %p\n",
 		cardp->rx_urb);
 	ret = usb_submit_urb(cardp->rx_urb, GFP_ATOMIC);
-- 
2.20.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] libertas_tf: don't set URB_ZERO_PACKET on IN USB transfer
  2019-02-10 19:47 [PATCH] libertas_tf: don't set URB_ZERO_PACKET on IN USB transfer Lubomir Rintel
@ 2019-02-11 13:38 ` Steve deRosier
  0 siblings, 0 replies; 2+ messages in thread
From: Steve deRosier @ 2019-02-11 13:38 UTC (permalink / raw)
  To: Lubomir Rintel; +Cc: Kalle Valo, David S. Miller, linux-wireless, LKML, stable

On Sun, Feb 10, 2019 at 11:48 AM Lubomir Rintel <lkundrak@v3.sk> wrote:
>
> It doesn't make sense and the USB core warns on each submit of such
> URB, easily flooding the message buffer with tracebacks.
>
> Analogous issue was fixed in regular libertas driver in commit 6528d8804780
> ("libertas: don't set URB_ZERO_PACKET on IN USB transfer").
>
> Cc: stable@vger.kernel.org
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
> ---
>  drivers/net/wireless/marvell/libertas_tf/if_usb.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/wireless/marvell/libertas_tf/if_usb.c b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
> index 789337ea676a..6ede6168bd85 100644
> --- a/drivers/net/wireless/marvell/libertas_tf/if_usb.c
> +++ b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
> @@ -433,8 +433,6 @@ static int __if_usb_submit_rx_urb(struct if_usb_card *cardp,
>                           skb_tail_pointer(skb),
>                           MRVDRV_ETH_RX_PACKET_BUFFER_SIZE, callbackfn, cardp);
>
> -       cardp->rx_urb->transfer_flags |= URB_ZERO_PACKET;
> -
>         lbtf_deb_usb2(&cardp->udev->dev, "Pointer for rx_urb %p\n",
>                 cardp->rx_urb);
>         ret = usb_submit_urb(cardp->rx_urb, GFP_ATOMIC);
> --
> 2.20.1
>

Reviewed-by: Steve deRosier <derosier@cal-sierra.com>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-10 19:47 [PATCH] libertas_tf: don't set URB_ZERO_PACKET on IN USB transfer Lubomir Rintel
2019-02-11 13:38 ` Steve deRosier

Linux-Wireless Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-wireless/0 linux-wireless/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-wireless linux-wireless/ https://lore.kernel.org/linux-wireless \
		linux-wireless@vger.kernel.org linux-wireless@archiver.kernel.org
	public-inbox-index linux-wireless


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-wireless


AGPL code for this site: git clone https://public-inbox.org/ public-inbox