From: Paul Durrant <Paul.Durrant@citrix.com>
To: Anthony Wright <anthony@overnetdata.com>,
Xen-devel <xen-devel@lists.xen.org>
Subject: Re: Bug in hash changes to netback in 4.7.2 kernel
Date: Tue, 6 Sep 2016 13:54:24 +0000 [thread overview]
Message-ID: <d77c995fbe1546f79579a9835ba2d099@AMSPEX02CL03.citrite.net> (raw)
In-Reply-To: <e37afb43-d9e3-a5e0-be50-6c654fe9fa04@overnetdata.com>
> -----Original Message-----
> From: Anthony Wright [mailto:anthony@overnetdata.com]
> Sent: 06 September 2016 14:37
> To: Paul Durrant <Paul.Durrant@citrix.com>; Xen-devel <xen-
> devel@lists.xen.org>
> Subject: Re: Bug in hash changes to netback in 4.7.2 kernel
>
> On 06/09/2016 13:57, Paul Durrant wrote:
> >> -----Original Message-----
> >> From: Anthony Wright [mailto:anthony@overnetdata.com]
> >> Sent: 06 September 2016 13:23
> >> To: Xen-devel <xen-devel@lists.xen.org>
> >> Cc: Paul Durrant <Paul.Durrant@citrix.com>
> >> Subject: Bug in hash changes to netback in 4.7.2 kernel
> >>
> >> When I run Xen (4.7.0) nested in VirtualBox (5.0.24_Ubuntu r108355)
> >> with a
> >> linux-4.7.2 Dom0 kernel, none of my DomU's (linux-3.17.3) have
> >> network connectivity because they reject all packets with the error
> 'Invalid extra type:
> >> 4'. When I run exactly the same setup on bare metal, I don't get the
> >> error messages.
> >>
> >> From poking around in the code this seems to be because the 4.7.2
> >> kernel wrongly decides that the DomU's will understand
> >> EXTRA_TYPE_HASH, and so attach it to the network packet. Since the
> >> DomU's don't understand the extra info their netfront driver rejects the
> whole packet.
> > The code in xenvif_select_queue() deliberately clears the skb->sw_hash
> field (which gates adding the new extra type) if the hash algorithm selected
> by the frontend is 'none', which should be the default. So, unless you have a
> frontend that is implementing the control ring protocol, but failing to
> recognize the new extra type I'm not how you're seeing the problem...
> unless somehow a packet which hash is getting into netback's start_xmit
> without first having gone through select_queue?
> I very much doubt that the frontend is implementing the control ring
> protocol, the DomUs are running stock linux-3.17.3. I build the system from
> source, so happy to re-compile with debug code.
Well, if you could have a look to see whether the skbs have sw_hash set when they get into net_rx_action() that would be something, and also check whether xenvif_select_queue() is being properly called. I'll take a look and see if I can repro this myself when I get some time.
Paul
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-09-06 13:54 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <31342491.14.1473163834539.JavaMail.root@zimbra.overnetdata.com>
2016-09-06 12:22 ` Bug in hash changes to netback in 4.7.2 kernel Anthony Wright
2016-09-06 12:57 ` Paul Durrant
2016-09-06 13:37 ` Anthony Wright
2016-09-06 13:54 ` Paul Durrant [this message]
2016-11-16 5:43 Todd Fleisher
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d77c995fbe1546f79579a9835ba2d099@AMSPEX02CL03.citrite.net \
--to=paul.durrant@citrix.com \
--cc=anthony@overnetdata.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).