|
|
![](http://www.xen.org/images/globals/pixel.gif) |
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] Re: [RFC] [PATCH] xen, vtd: Check ownership of a domain
Thanks, Jan.
BTW Keir: this particular instance of the patch caused the customer's
dom0 to crash on boot, so it's not ready to be applied yet. (Oh the
joys of fixing a bug for a system you don't have on-site.) I'll send
a new patch when I have one that works properly.
-George
On Thu, Jul 28, 2011 at 6:48 PM, Jan Beulich <JBeulich@xxxxxxxxxx> wrote:
>>>> On 27.07.11 at 12:36, George Dunlap <George.Dunlap@xxxxxxxxxxxxx> wrote:
>> Keir, Jan, et al:
>>
>> Can you take a look at this patch to see if you think this is the
>> right way to solve this problem?
>
> Makes sense, and looks reasonable to me (though I'm far from being
> a passthrough expert).
>
>> This particular patch is still being tested by the partner who saw the
>> problem; but a less safe change -- just returning "OK" if the pci
>> device didn't exist -- did solve the problem. It seemed like this
>> check was probably there for a reason, so I wanted to keep it in
>> place. (Although the c/s in which this check was introduced,
>> 18906:2941b1a9, only mentioned locking changes, not introduction of
>> new checks.)
>
> Yes, retaining the check is definitely needed.
>
> Jan
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|