Re: [Xen-devel] [PATCH v5 06/15] public / x86: introduce __HYPERCALL_iommu_op

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 07.08.18 at 10:10, <Paul.Durrant@xxxxxxxxxx> wrote:
>> From: Tian, Kevin [mailto:kevin.tian@xxxxxxxxx]
>> Sent: 07 August 2018 04:00
>> 
>> > From: Paul Durrant
>> > Sent: Saturday, August 4, 2018 1:22 AM
>> >
>> > diff --git a/xen/arch/x86/iommu_op.c b/xen/arch/x86/iommu_op.c
>> > new file mode 100644
>> > index 0000000000..744c0fce27
>> > --- /dev/null
>> > +++ b/xen/arch/x86/iommu_op.c
>> > @@ -0,0 +1,184 @@
>> >
>> +/*********************************************************
>> ***
>> > ******************
>> > + * x86/iommu_op.c
>> > + *
>> > + * Paravirtualised IOMMU functionality
>> 
>> although only x86 is supported, logic in this file is vendor agnostic. Better
>> move to a common place now...
>> 
> 
> Personally I don't like to expand scope into architectures which I have no 
> facility to test.  IIUC ARM also very much relies on an identity GFN:BFN map 
> so this functionality is unlikely to get used by anything other than x86 for 
> the foreseeable future.

Well - moving to a common place doesn't mean it needs to be built
for ARM.

Jan



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