Re: [Xen-devel] [PATCH v3 06/13] xen/passthrough: iommu: Split generic IOMMU code

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 11.03.14 at 16:49, Julien Grall <julien.grall@xxxxxxxxxx> wrote:
> --- a/xen/drivers/passthrough/pci.c
> +++ b/xen/drivers/passthrough/pci.c
>...
> +static int iommu_populate_page_table(struct domain *d)
> +{

I continue to be of the opinion that this is misplaced here. There's
nothing PCI-related in this function, and I doubt you can get away
on ARM without similar code (if you can, this should go into
.../x86/iommu.c imo).

> --- /dev/null
> +++ b/xen/include/asm-x86/iommu.h
>...
> +void iommu_set_dom0_mapping(struct domain *d);

How is this x86-specific?

Jan


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