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: Tue, 01 Feb 2011 23:04:05 +0100	[thread overview]
Message-ID: <4D488355.8010706@jbfavre.org> (raw)
In-Reply-To: <1296577389.13091.288.camel@zakaz.uk.xensource.com>

Le 01/02/2011 17:23, Ian Campbell a écrit :
> On Tue, 2011-02-01 at 15:38 +0000, Jean Baptiste Favre wrote:
>> OK, just found it:
>> after domU boot:
>> - log in
>> - ping -s 86 10.0.0.1 (fails)
>> - rmmod sky2
>> - modprobe sky2 copybreak=0 (no packet copied)
>> - ping -s 86 10.0.0.1 (works)
>>
>> So it's clearly related to that option.
> 
> Awesome!
> 
>> Now the question: what am I supposed to do ?
> 
> I think the next step is to try and reproduce on native 32 bit, with RAM
> artificially limited via the mem= kernel command in option, this will
> let us determine if this is a generic issue or is somehow Xen specific.
OK, so I'll install a 32bits Debian Squeeze with 2.6.37 32bits kernel to
check it.

> The main difference caused by the copybreak option is that for larger
> frames (i.e. always when copybreak==0) it hits a code path which uses
> pci_map_single and pci_map_page to access the received data. When len <
> copybreak it takes a path which uses pci_dma_sync_single_for_cpu, so it
> seems like the later path is broken somehow. If the issue does turn out
> to be related to Xen then I think that points to the swiotlb code.
> 
> I assume you are not seeing "rx mapping error" in your domU dmesg? Did
> you post a full guest console log at some point? Comparing the logs for
> the 256MB, 398MB and 512MB guest RAM case might be useful.
No sure I've ever posted that logs. But I can redo my tests :)

> Konrad, is there some way to force swiotlb use even for native to make
> the native test cases more relevant? Or is there some other direction we
> should try first?
> 
>> It seems that adding this options in /etc/modprobe.d/sky2.conf does not
>> work, neither using /etc/modules
> 
> That will be a userspace issue in your distro, perhaps with openwrt they
> are not expected to work that way.
All tests were done with Debian. As the module is included in initramfs
and not on domU, since I've no kernel installed in the domU, module is
loaded before system FS, so /etc/modprobe.d/sky2.conf is not taken into
account.
I think that the best solution here is to compile kernel with  static
sky2 driver inclusion and use kernel commandline to pass sky2 copybreak
option.

Regards,
JB

  parent reply	other threads:[~2011-02-01 22:04 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 [this message]
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
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=4D488355.8010706@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).