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

Re: [Xen-devel] [PATCH v5 6/6] amd/iommu: skip bridge devices when updating IOMMU page tables



>>> On 23.11.18 at 15:36, <roger.pau@xxxxxxxxxx> wrote:
> On Thu, Nov 22, 2018 at 06:20:46AM -0700, Jan Beulich wrote:
>> >>> On 22.11.18 at 13:47, <roger.pau@xxxxxxxxxx> wrote:
>> > I think the is_hardware_domain part can be dropped from the
>> > conditional I'm adding. update_paging_mode shouldn't be used to decide
>> > whether a domain can or cannot have bridges attached. Whether a DomU
>> > can or cannot have a host bridge assigned should be decided at
>> > assignation time, and hence update_paging_mode shouldn't have hardware
>> > domain specific checks.
>> 
>> Okay, we're in agreement then.
>> 
>> > Regarding the check in amd_iommu_add_device, if it's removed from
>> > there amd_iommu_add_device would return an error when adding a host
>> > bridge device, and that would cause setup_one_hwdom_device to return
>> > early and not setup vPCI handlers for host bridges, so I think we want
>> > to leave that one as-is.
>> 
>> Right, I can see why it may be better to retain it there.
> 
> Thanks. Would you like me to resend the series with this fix, or
> should I wait for feedback on the remaining patches?

I'd leave that up to you. I have yet to find time to look at patches
1, 2, and 3 again.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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