Message ID | 20111107145458.29997.79829.stgit@zurg |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Mon, Nov 07, 2011 at 06:54:58PM +0300, Konstantin Khlebnikov wrote: > This patch fixes the bug added in commit v3.1-rc7-1055-gf9b491e > SKB can be NULL at this point, at least for cdc-ncm. > > Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org> Acked-by: Richard Cochran <richardcochran@gmail.com> > --- > drivers/net/usb/usbnet.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/usb/usbnet.c b/drivers/net/usb/usbnet.c > index 7d60821..fae0fbd 100644 > --- a/drivers/net/usb/usbnet.c > +++ b/drivers/net/usb/usbnet.c > @@ -1057,7 +1057,8 @@ netdev_tx_t usbnet_start_xmit (struct sk_buff *skb, > unsigned long flags; > int retval; > > - skb_tx_timestamp(skb); > + if (skb) > + skb_tx_timestamp(skb); > > // some devices want funky USB-level framing, for > // win32 driver (usually) and/or hardware quirks > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Richard Cochran <richardcochran@gmail.com> Date: Mon, 7 Nov 2011 18:39:19 +0100 > On Mon, Nov 07, 2011 at 06:54:58PM +0300, Konstantin Khlebnikov wrote: >> This patch fixes the bug added in commit v3.1-rc7-1055-gf9b491e >> SKB can be NULL at this point, at least for cdc-ncm. >> >> Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org> > > Acked-by: Richard Cochran <richardcochran@gmail.com> Applied, but the overall logic in the usbnet transmit path definitely could use a good reconsideration and cleanup. I'm even open to having generic support at the generic net device TX level to fixup the segmentation layout of the SKB so that it meets the device's requirements. We can do it more efficiently there too. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/usb/usbnet.c b/drivers/net/usb/usbnet.c index 7d60821..fae0fbd 100644 --- a/drivers/net/usb/usbnet.c +++ b/drivers/net/usb/usbnet.c @@ -1057,7 +1057,8 @@ netdev_tx_t usbnet_start_xmit (struct sk_buff *skb, unsigned long flags; int retval; - skb_tx_timestamp(skb); + if (skb) + skb_tx_timestamp(skb); // some devices want funky USB-level framing, for // win32 driver (usually) and/or hardware quirks
This patch fixes the bug added in commit v3.1-rc7-1055-gf9b491e SKB can be NULL at this point, at least for cdc-ncm. Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org> --- drivers/net/usb/usbnet.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html