netdev.vger.kernel.org archive mirror
 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
  0 siblings, 1 reply; 44+ 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] 44+ messages in thread

end of thread, other threads:[~2012-11-21 11:09 UTC | newest]

Thread overview: 44+ 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 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:17   ` Ian Campbell
2012-10-09 14:27     ` Eric Dumazet
2012-10-09 14:40       ` Ian Campbell
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:09           ` Ian Campbell
2012-10-10 14:49             ` Sander Eikelenboom
2012-10-11  8:02               ` Ian Campbell
2012-10-11 10:00                 ` Sander Eikelenboom
2012-10-11 10:05                   ` Eric Dumazet
2012-10-11 10:14                     ` Ian Campbell
2012-10-11 10:20                       ` Sander Eikelenboom
2012-11-15  2:31                       ` 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-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 14:14                                       ` Ian Campbell
2012-11-20 14:32                                         ` Jan Beulich
2012-11-20 15:06                                           ` Ian Campbell
2012-11-20 15:28                                             ` Eric Dumazet
2012-11-20 15:54                                               ` Ian Campbell
2012-11-20 15:44                                             ` Jan Beulich
2012-11-20 16:14                                               ` Eric Dumazet
2012-11-20 13:30                                 ` Stefan Bader
2012-11-20 13:45                                   ` Sander Eikelenboom
2012-11-20 14:45                                 ` Eric Dumazet
2012-11-20 15:05                                   ` Ian Campbell
2012-11-21  2:52                                 ` ANNIE LI
2012-11-21 11:09                                   ` Ian Campbell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).