All of lore.kernel.org
 help / color / mirror / Atom feed
* compound skb frag pages appearing in start_xmit
@ 2012-10-09 13:47 Ian Campbell
  2012-10-09 13:54 ` Eric Dumazet
  2012-10-09 13:54 ` Eric Dumazet
  0 siblings, 2 replies; 92+ messages in thread
From: Ian Campbell @ 2012-10-09 13:47 UTC (permalink / raw)
  To: Eric Dumazet; +Cc: xen-devel, netdev, Konrad Rzeszutek Wilk, Sander Eikelenboom

Hi Eric,

Sander has discovered an issue where xen-netback is given a compound
page as one of the skb frag pages to transmit. Currently netback can
only handle PAGE_SIZE'd frags and bugs out.

I suspect this is something to do with 69b08f62e174 "net: use bigger
pages in __netdev_alloc_frag", although perhaps not because it looks
like only tg3 uses it and Sander has an r8169. Also tg3 seems to only
call netdev_alloc_frag for sizes < PAGE_SIZE. I'm probably missing
something.

Are all net drivers expected to be able to handle compound pages in the
frags? Obviously it is to their benefit to do so, so it is something
I'll want to look into for netback.

I expect the main factor here is bridging/forwarding, since the
receiving NIC and its driver appear to support compound pages but the
outgoing NIC (netback in this case) does not.

I guess my question is should I be rushing to fix netback ASAP or should
I rather be looking for a bug somewhere which caused a frag of this type
to get as far as netback's start_xmit in the first place?

Or am I just barking up the wrong tree to start with?

Thanks,
Ian.

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

end of thread, other threads:[~2012-11-23 15:08 UTC | newest]

Thread overview: 92+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-09 13:47 compound skb frag pages appearing in start_xmit Ian Campbell
2012-10-09 13:54 ` Eric Dumazet
2012-10-09 13:54 ` Eric Dumazet
2012-10-09 14:01   ` Eric Dumazet
2012-10-09 14:23     ` Ian Campbell
2012-10-09 14:33       ` Eric Dumazet
2012-10-09 14:54         ` Ian Campbell
2012-10-09 14:54         ` Ian Campbell
2012-10-09 14:33       ` Eric Dumazet
2012-10-09 14:23     ` Ian Campbell
2012-10-09 14:01   ` Eric Dumazet
2012-10-09 14:17   ` Ian Campbell
2012-10-09 14:27     ` Eric Dumazet
2012-10-09 14:27     ` Eric Dumazet
2012-10-09 14:40       ` Ian Campbell
2012-10-09 14:40       ` Ian Campbell
2012-10-09 14:51         ` Eric Dumazet
2012-10-09 14:51         ` Eric Dumazet
2012-10-10 10:13         ` [Xen-devel] " Ian Campbell
2012-10-10 12:24           ` Sander Eikelenboom
2012-10-10 12:29             ` [Xen-devel] " Ian Campbell
2012-10-10 13:31               ` Sander Eikelenboom
2012-10-10 13:31               ` Sander Eikelenboom
2012-10-10 12:29             ` Ian Campbell
2012-10-10 13:09           ` [Xen-devel] " Ian Campbell
2012-10-10 14:49             ` Sander Eikelenboom
2012-10-10 14:49             ` [Xen-devel] " Sander Eikelenboom
2012-10-11  8:02               ` Ian Campbell
2012-10-11  8:02               ` [Xen-devel] " Ian Campbell
2012-10-11 10:00                 ` Sander Eikelenboom
2012-10-11 10:00                 ` [Xen-devel] " Sander Eikelenboom
2012-10-11 10:05                   ` Eric Dumazet
2012-10-11 10:14                     ` Ian Campbell
2012-10-11 10:20                       ` Sander Eikelenboom
2012-10-11 10:20                       ` [Xen-devel] " Sander Eikelenboom
2012-11-15  2:31                       ` ANNIE LI
2012-11-15  2:31                       ` [Xen-devel] " ANNIE LI
2012-11-19 15:43                         ` Sander Eikelenboom
2012-11-20  8:30                           ` [Xen-devel] " Stefan Bader
2012-11-20  9:21                             ` Ian Campbell
2012-11-20 11:36                               ` [Xen-devel] " Ian Campbell
2012-11-21  2:42                                 ` ANNIE LI
2012-11-21  2:42                                 ` ANNIE LI
2012-11-20 11:36                               ` Ian Campbell
2012-11-20 11:40                               ` [PATCH] xen/netfront: handle compound page fragments on transmit Ian Campbell
2012-11-20 12:28                                 ` [Xen-devel] " Jan Beulich
2012-11-20 13:35                                   ` Ian Campbell
2012-11-20 13:51                                     ` Jan Beulich
2012-11-20 13:51                                     ` [Xen-devel] " Jan Beulich
2012-11-20 14:14                                       ` Ian Campbell
2012-11-20 14:32                                         ` Jan Beulich
2012-11-20 14:32                                         ` [Xen-devel] " Jan Beulich
2012-11-20 15:06                                           ` Ian Campbell
2012-11-20 15:06                                           ` [Xen-devel] " Ian Campbell
2012-11-20 15:28                                             ` Eric Dumazet
2012-11-20 15:54                                               ` Ian Campbell
2012-11-20 16:04                                                 ` netchannel vs MAX_SKB_FRAGS (Was: Re: [PATCH] xen/netfront: handle compound page fragments on transmit) Ian Campbell
2012-11-23  1:30                                                   ` ANNIE LI
2012-11-23  2:10                                                     ` James Harper
2012-11-23  6:49                                                       ` ANNIE LI
2012-11-23  9:22                                                       ` Paul Durrant
2012-11-23  9:26                                                         ` James Harper
2012-11-23  9:34                                                           ` Ian Campbell
2012-11-23  9:18                                                     ` Ian Campbell
2012-11-23 15:08                                                       ` annie li
2012-11-20 15:54                                               ` [PATCH] xen/netfront: handle compound page fragments on transmit Ian Campbell
2012-11-20 15:28                                             ` Eric Dumazet
2012-11-20 15:44                                             ` [Xen-devel] " Jan Beulich
2012-11-20 16:14                                               ` Eric Dumazet
2012-11-20 15:44                                             ` Jan Beulich
2012-11-20 14:14                                       ` Ian Campbell
2012-11-20 13:35                                   ` Ian Campbell
2012-11-20 12:28                                 ` Jan Beulich
2012-11-20 13:30                                 ` Stefan Bader
2012-11-20 13:30                                 ` Stefan Bader
2012-11-20 13:45                                   ` Sander Eikelenboom
2012-11-20 13:45                                   ` Sander Eikelenboom
2012-11-20 14:45                                 ` Eric Dumazet
2012-11-20 15:05                                   ` Ian Campbell
2012-11-20 15:05                                   ` Ian Campbell
2012-11-20 14:45                                 ` Eric Dumazet
2012-11-21  2:52                                 ` ANNIE LI
2012-11-21 11:09                                   ` Ian Campbell
2012-11-21 11:09                                   ` Ian Campbell
2012-11-21  2:52                                 ` ANNIE LI
2012-11-20 11:40                               ` Ian Campbell
2012-11-20  8:30                           ` compound skb frag pages appearing in start_xmit Stefan Bader
2012-10-11 10:14                     ` Ian Campbell
2012-10-11 10:05                   ` Eric Dumazet
2012-10-10 13:09           ` Ian Campbell
2012-10-10 10:13         ` Ian Campbell
2012-10-09 14:17   ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.