Re: [Xen-devel] [PATCH] IOMMU: Prevent VT-d device IOTLB operations on wrong IOMMU

["Jan Beulich" <jbeulich@xxxxxxxx>]

>>> Andrew Cooper <andrew.cooper3@xxxxxxxxxx> 06/12/14 2:05 PM >>>
>>On 12/06/14 12:40, Malcolm Crossley wrote:
>> --- a/xen/drivers/passthrough/ats.h
>> +++ b/xen/drivers/passthrough/ats.h
>> @@ -24,6 +24,7 @@ struct pci_ats_dev {
>>      u8 bus;
>>      u8 devfn;
>>      u16 ats_queue_depth;    /* ATS device invalidation queue depth */
>> +    void *iommu;
>
>This could really do with a comment here explaining why a void *, and
>which two types of structure it could end up pointing at.

Right. Or make it a transparent union in the first place. And in any case, 
making
more clear that it's for nothing but looking at, add "const".

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel