From: Alex Zeffertt <alex.zeffertt@eu.citrix.com>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Xen-devel <xen-devel@lists.xensource.com>,
Keir Fraser <Keir.Fraser@eu.citrix.com>,
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: xen crash related to pci passthrough
Date: Fri, 28 May 2010 10:28:21 +0100 [thread overview]
Message-ID: <4BFF8CB5.9030002@eu.citrix.com> (raw)
In-Reply-To: <4BFEDDDC.3060904@goop.org>
This is only tangentially related, but...
In xen/arch/x86/irq.c:__pirq_guest_unbind() this code looks wrong to me:
memmove(&action->guest[i], &action->guest[i+1], IRQ_MAX_GUESTS-i-1);
Should it be:
memmove(&action->guest[i], &action->guest[i+1],
sizeof(action->guest[0])*(IRQ_MAX_GUESTS-i-1));
?
Regards,
Alex
On 27/05/10 22:02, Jeremy Fitzhardinge wrote:
> On 05/27/2010 01:02 PM, Keir Fraser wrote:
>> On 27/05/2010 20:51, "Jeremy Fitzhardinge"<jeremy@goop.org> wrote:
>>
>>
>>> 3. attach device to PV domU
>>> dom0# xl pci-attach f13pv64 0000:03:00.0
>>> 4. unbind from pciback
>>> dom0# echo 0000:03:00.0> /sys/bus/pci/drivers/pciback/unbind
>>> 5. rmmod driver in domU
>>> domU# rmmod e1000e
>>> 6. Crash!
>>>
>>> The device in question is an Intel 82574L ethernet controller, using
>>> msi-x interrupts.
>>>
>> What happens if you pci-detach before unbinding from pciback? Not that Xen
>> should crash of course, but a crash resulting from a mistake in detach
>> ordring in dom0 would be less worrying than some alternatives.
>>
>
> Doing things in a more sensible order (rmmod in domU, detach, unbind)
> seems to work OK. I was deliberately seeing what would happen if I
> tried pulling the device out from under a domain.
>
> J
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>
next prev parent reply other threads:[~2010-05-28 9:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-27 19:51 xen crash related to pci passthrough Jeremy Fitzhardinge
2010-05-27 20:02 ` Keir Fraser
2010-05-27 21:02 ` Jeremy Fitzhardinge
2010-05-28 9:28 ` Alex Zeffertt [this message]
2010-05-28 9:52 ` Keir Fraser
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=4BFF8CB5.9030002@eu.citrix.com \
--to=alex.zeffertt@eu.citrix.com \
--cc=Keir.Fraser@eu.citrix.com \
--cc=jeremy@goop.org \
--cc=konrad.wilk@oracle.com \
--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).