Hi,

On 21/02/18 21:46, Wei Liu wrote:
Move the functions that reference x86 hvm data structures to its own
file.  Rename pci_clean_dpci_irqs to arch_pci_clean_irqs.

NIT: Double space.


There is still one location in that file which references
arch.hvm_domain, but it is fine because ARM guest is HVM.

I guess you mean hvm_domain.mem_sharing_enabled? If so, we don't support memory sharing on ARM. This would need to be stub.


Signed-off-by: Wei Liu <wei.l...@citrix.com>
---
Cc: Stefano Stabellini <sstabell...@kernel.org>
Cc: Julien Grall <julien.gr...@arm.com>
Cc: Jan Beulich <jbeul...@suse.com>
Cc: Andrew Cooper <andrew.coop...@citrix.com>

ARM doesn't select HAS_PCI, that's why ARM build is not broken by
this. AIUI ARM will select HAS_PCI at some point, hence I only move
the x86 bits.

Potentially there are more x86-ism in that code such as the way to handle MSI. Anyway, this patch makes sense from an Arm perspective:

Acked-by: Julien Grall <julien.gr...@arm.com>

Cheers,

--
Julien Grall

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

Reply via email to