xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
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
>

  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).