On 22/10/2025 10:51 am, Teddy Astie wrote:
> These functions are basically wrappers of map_domain_page; move
We tend to write map_domain_page() with brackets, to make it clearer
that we're talking about a function.
> diff --git a/xen/drivers/passthrough/vtd/extern.h
> b/xen/drivers/passthrough/vtd/extern.h
> index c16583c951..a62310b3e7 100644
> --- a/xen/drivers/passthrough/vtd/extern.h
> +++ b/xen/drivers/passthrough/vtd/extern.h
> @@ -114,4 +113,14 @@ void quirk_iommu_caps(struct vtd_iommu *iommu);
> bool platform_supports_intremap(void);
> bool platform_supports_x2apic(void);
>
> +static inline void *map_vtd_domain_page(u64 maddr)
> +{
> + return map_domain_page(_mfn(paddr_to_pfn(maddr)));
maddr_to_mfn() drops the _mfn() wrapper. u64 wants to become paddr_t.
Otherwise, Acked-by: Andrew Cooper <[email protected]>. I can
fix up when queueing.
~Andrew
