[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Re: [RFC] [PATCH] xen, vtd: Check ownership of a domain context using internal structures



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.