xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Jean Baptiste Favre <xen-devel@jbfavre.org>
To: xen-devel@lists.xensource.com
Subject: Re: PCI passthrough issue
Date: Fri, 04 Feb 2011 14:15:16 +0100	[thread overview]
Message-ID: <4D4BFBE4.6080809@jbfavre.org> (raw)
In-Reply-To: <1296818935.13091.648.camel@zakaz.uk.xensource.com>

[-- Attachment #1: Type: text/plain, Size: 2447 bytes --]

Le 04/02/2011 12:28, Ian Campbell a écrit :
> On Fri, 2011-02-04 at 11:25 +0000, Jean Baptiste Favre wrote:
>> Hello,
>>
>> Le 04/02/2011 12:04, Ian Campbell a écrit :
>>> On Fri, 2011-02-04 at 10:12 +0000, Jean Baptiste Favre wrote:
>>>> Hello Ian,
>>>> Applyed your patches.
>>>
>>> Thanks.
>>>
>>>> Now, I've:
>>>> # ping -s86 10.0.0.1
>>>> PING 10.0.0.1 (10.0.0.1): 86 data bytes
>>>> __netif_receive_skb dropping skb proto 0x20
>>>>
>>>>
>>>> So problem seems to occur in net/core/dev.c file, according to the patch
>>>> bellow
>>>
>>> Interesting. the number printed in the warning is type == skb->protocol
>>> == 0x20 which is not a valid protocol that I can find anywhere (nor is
>>> 0x2000 in case I'm mixing my endianesses up). Neither is 0x20 it a valid
>>> Ethernet frame length (min 64) so it's not that sort of confusion
>>> AFAICT.
>>>
>>> skb->protocol is initialised in sky2_status_intr with the return value
>>> of "eth_type_trans(skb, dev)" which as far as I can tell cannot return
>>> 0x20.
>>>
>>> The domU network configuration is using the sky2 device directly, no
>>> bridging, VLAN, tunnels or anything else like that?
>> At boot it uses bridge.
>> For the test, I delete bridge and set IP address directly on eth0
> 
> OK, good.
> 
> [...]
>> generic-segmentation-offload: on
> [...]
>> receive-hashing: on
> 
> Can you also try turning these two off (independently and together).
No change with ethtool -K gso off
Can not change receive-hashing:
# ethtool -K eth1 rxhash off
Cannot set device flag settings: Invalid argument



>> What is a bit strange here is that I don't any more the KERN_CRIT printk
>> message.
>> Could be a false positive ?
> 
> Worth bearing in mind, lets see what the next test run produces.
Seems that I got this messge only with copybreak=0.
With default value (128), no such message

More, with copybreak=0, all packets are dropped (even a ping with
default packet size is dropped. Same with ping -s1)

>> I'm currently compiling new kernel with your last patch. will keep you
>> updated
No new messages, despite your patch :(

> Thanks.
> 
> Please gather the tcpdump's too.
Both tcpdump from GW and domU are Attached.

Commands were:

domU# tcpdump -n -w domU.cap -s0 -i eth0 ether host 00:1f:c6:eb:71:43 or
ether broadcast

gw# tcpdump -n -w gw.cap -s0 -i br0 ether host 00:1f:c6:eb:71:43 or
ether broadcast


[-- Attachment #2: gw.cap --]
[-- Type: application/octet-stream, Size: 8906 bytes --]

[-- Attachment #3: domU.cap --]
[-- Type: application/octet-stream, Size: 7484 bytes --]

[-- Attachment #4: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

  reply	other threads:[~2011-02-04 13:15 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-12 15:38 PCI passthrough issue Jean Baptiste Favre
2011-01-12 15:43 ` Konrad Rzeszutek Wilk
2011-01-12 15:53   ` Jean Baptiste Favre
2011-01-12 16:09     ` Jean Baptiste Favre
2011-01-12 16:36     ` Konrad Rzeszutek Wilk
2011-01-12 16:56       ` Jean Baptiste Favre
2011-01-12 17:26         ` Konrad Rzeszutek Wilk
2011-01-12 17:44           ` Jean Baptiste Favre
2011-01-12 18:32             ` Konrad Rzeszutek Wilk
2011-01-12 20:07               ` Jean Baptiste Favre
2011-01-12 21:40                 ` Konrad Rzeszutek Wilk
2011-01-12 21:46                   ` Jean Baptiste Favre
2011-01-12 22:18                     ` Jean Baptiste Favre
2011-01-13 11:28                       ` Jean Baptiste Favre
2011-01-13 19:18                         ` Jean Baptiste Favre
2011-01-13 20:19                           ` Konrad Rzeszutek Wilk
2011-01-13 20:44                             ` Jean Baptiste Favre
2011-01-14 14:53                               ` Konrad Rzeszutek Wilk
2011-01-14 23:29                                 ` Jean Baptiste Favre
2011-01-17  8:59                                   ` Jean Baptiste Favre
2011-01-17 13:58                                     ` Jean Baptiste Favre
2011-01-22 10:22                                       ` Jean Baptiste Favre
2011-01-27 20:27                                         ` Konrad Rzeszutek Wilk
2011-01-27 21:47                                           ` Jean Baptiste Favre
2011-01-28 15:47                                             ` Jean Baptiste Favre
2011-02-01 11:34                                               ` Ian Campbell
2011-02-01 12:17                                                 ` Jean Baptiste Favre
2011-02-01 13:20                                                   ` Ian Campbell
2011-02-01 14:12                                                     ` Jean Baptiste Favre
2011-02-01 14:18                                                       ` Ian Campbell
2011-02-01 15:14                                                         ` Jean Baptiste Favre
2011-02-01 15:38                                                           ` Jean Baptiste Favre
2011-02-01 16:23                                                             ` Ian Campbell
2011-02-01 19:37                                                               ` Konrad Rzeszutek Wilk
2011-02-01 22:06                                                                 ` Jean Baptiste Favre
2011-02-02  9:12                                                                   ` Ian Campbell
2011-02-01 23:01                                                                 ` Jean Baptiste Favre
2011-02-02  9:44                                                                   ` Ian Campbell
2011-02-02 15:38                                                                     ` Konrad Rzeszutek Wilk
2011-02-02 15:55                                                                       ` Ian Campbell
2011-02-02 15:56                                                                       ` Jean Baptiste Favre
2011-02-01 22:04                                                               ` Jean Baptiste Favre
2011-02-02  9:27                                                                 ` Ian Campbell
2011-02-02 10:24                                                                   ` Jean Baptiste Favre
2011-02-02 10:59                                                                     ` Ian Campbell
2011-02-02 11:33                                                                       ` Jean Baptiste Favre
2011-02-02 17:42                                                                     ` Konrad Rzeszutek Wilk
2011-02-04  8:43                                                                       ` Jean Baptiste Favre
2011-02-04  8:53                                                                         ` Ian Campbell
2011-02-04  8:54                                                                           ` Jean Baptiste Favre
2011-02-04 10:12                                                                             ` Jean Baptiste Favre
2011-02-04 11:04                                                                               ` Ian Campbell
2011-02-04 11:25                                                                                 ` Jean Baptiste Favre
2011-02-04 11:28                                                                                   ` Ian Campbell
2011-02-04 13:15                                                                                     ` Jean Baptiste Favre [this message]
2011-02-04 13:50                                                                                       ` Ian Campbell
2011-02-04 14:01                                                                                         ` Jean Baptiste Favre
2011-02-09  9:59                                                                                           ` Jean Baptiste Favre
2011-02-18 21:14                                                                                             ` Jean Baptiste Favre
2011-02-25 14:40                                                                                               ` Konrad Rzeszutek Wilk
2011-02-28 10:00                                                                                                 ` Jean Baptiste Favre
2011-02-28 12:18                                                                                                 ` Jean Baptiste Favre
2011-02-28 15:01                                                                                                   ` Konrad Rzeszutek Wilk
2011-02-28 15:17                                                                                                     ` Jean Baptiste Favre
     [not found]                                                                                                       ` <4D6CB851.40103@jbfavre.org>
2011-03-03 22:12                                                                                                         ` Konrad Rzeszutek Wilk
2011-03-03 22:47                                                                                                           ` Jean Baptiste Favre
2011-03-03 22:58                                                                                                             ` Konrad Rzeszutek Wilk
2011-03-04  7:25                                                                                                               ` Jean Baptiste Favre
2011-03-16  3:14                                                                                                                 ` Konrad Rzeszutek Wilk
2011-03-25 13:06                                                                                                                   ` Jean Baptiste Favre
2011-02-01 15:38                                                           ` Ian Campbell
2011-02-01 16:16                                                             ` Jean Baptiste Favre
2011-01-12 21:44                 ` Jean Baptiste Favre

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=4D4BFBE4.6080809@jbfavre.org \
    --to=xen-devel@jbfavre.org \
    --cc=xen-devel@lists.xensource.com \
    /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).