On Tue, Oct 20, 2026 at 08:13:05AM -0700, Pavel Popov wrote:
> Peer-to-Peer DMA (P2PDMA) between two devices requires either a shared
> upstream bridge or a host bridge matching pci_p2pdma_whitelist[]. In
> VMware guests, passthrough devices appear directly on virtual root
> buses without host bridge devices. In Hyper-V guests, each passthrough
> device appears in a PCI domain of its own without a host bridge
> device. Consequently, P2PDMA checks evaluate endpoint Device IDs
> against the host bridge whitelist, causing P2PDMA requests to fail
> regardless of host support.
>
> This affects discrete devices such as GPUs, NICs and NVMe drives.
>
> Add an explicit list of the Intel Xeon server generations from
> Skylake-SP to Diamond Rapids to cpu_supports_p2pdma(). These are the
> CPUs whose host bridges are in pci_p2pdma_whitelist[] without
> restrictions. AMD Zen and newer are already covered by
> cpu_supports_p2pdma(), so only Intel entries are needed.
>
> Tested with the Intel Level Zero P2P benchmark (ze_peer) on Arc Pro B60
> cards in a Linux guest on VMware ESXi 9.0 (Emerald Rapids host).
>
> Signed-off-by: Pavel Popov <[email protected]>
> ---
> Changes in v3:
> - Drop the hypervisor check and move the Xeon list into
> cpu_supports_p2pdma(), next to the AMD check, as suggested by Logan
> Gunthorpe and Leon Romanovsky [1]
> - Fold the VMware and Hyper-V patches into one, as the check no longer
> depends on the hypervisor
>
> Changes in v2:
> - Replace the Skylake-and-newer threshold with an explicit list of Intel
> Xeon server generations, from Skylake-SP to Diamond Rapids (Logan
> Gunthorpe)
> - Move the CONFIG_X86 #ifdef out of hypervisor_supports_p2pdma()
> (Logan Gunthorpe)
> - Rebased on current mainline
>
> [1] https://lore.kernel.org/r/20261016153147.GC219304@unreal
>
> v2: https://lore.kernel.org/r/[email protected]
> v1: https://lore.kernel.org/r/[email protected]
>
> drivers/pci/p2pdma.c | 26 ++++++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
> index 9334eb314663..2650d605d87d 100644
> --- a/drivers/pci/p2pdma.c
> +++ b/drivers/pci/p2pdma.c
> @@ -21,6 +21,11 @@
> #include <linux/seq_buf.h>
> #include <linux/xarray.h>
>
> +#ifdef CONFIG_X86
> +#include <asm/cpu_device_id.h>
> +#include <asm/intel-family.h>
> +#endif
> +
> struct pci_p2pdma {
> struct gen_pool *pool;
> bool p2pmem_published;
> @@ -519,6 +524,23 @@ static void seq_buf_print_bus_devfn(struct seq_buf *buf,
> struct pci_dev *pdev)
> seq_buf_printf(buf, "%s;", pci_name(pdev));
> }
>
> +#ifdef CONFIG_X86
Do we really need all these #ifdefs? I'm pretty sure they're not needed for the
includes.
Thanks
> +/* Intel Xeon CPUs whose host bridges support p2pdma */
> +static const struct x86_cpu_id p2pdma_intel_cpu_ids[] = {
> + X86_MATCH_VFM(INTEL_SKYLAKE_X, NULL),
> + X86_MATCH_VFM(INTEL_ICELAKE_X, NULL),
> + X86_MATCH_VFM(INTEL_ICELAKE_D, NULL),
> + X86_MATCH_VFM(INTEL_SAPPHIRERAPIDS_X, NULL),
> + X86_MATCH_VFM(INTEL_EMERALDRAPIDS_X, NULL),
> + X86_MATCH_VFM(INTEL_GRANITERAPIDS_X, NULL),
> + X86_MATCH_VFM(INTEL_GRANITERAPIDS_D, NULL),
> + X86_MATCH_VFM(INTEL_ATOM_CRESTMONT_X, NULL),
> + X86_MATCH_VFM(INTEL_ATOM_DARKMONT_X, NULL),
> + X86_MATCH_VFM(INTEL_DIAMONDRAPIDS_X, NULL),
> + {}
> +};
> +#endif
> +
> static bool cpu_supports_p2pdma(void)
> {
> #ifdef CONFIG_X86
> @@ -527,6 +549,10 @@ static bool cpu_supports_p2pdma(void)
> /* Any AMD CPU whose family ID is Zen or newer supports p2pdma */
> if (c->x86_vendor == X86_VENDOR_AMD && c->x86 >= 0x17)
> return true;
> +
> + /* Listed Intel Xeon CPUs support p2pdma */
> + if (x86_match_cpu(p2pdma_intel_cpu_ids))
> + return true;
> #endif
>
> return false;
>
> base-commit: c03c1a14507dfaf8b856c2608f9ac216021fc222
> --
> 2.53.0
>
>
