RE: [PATCH v3 1/9] PCI/P2PDMA: Add CONFIG_PCI_P2PDMA_CORE

[Tian, Kevin]

> From: Pranjal Shrivastava <praan@xxxxxxxxxx>
> Sent: Friday, June 12, 2026 2:38 AM
> 
> On Wed, Jun 10, 2026 at 04:43:15PM +0100, Matt Evans wrote:
> > --- a/drivers/pci/Kconfig
> > +++ b/drivers/pci/Kconfig
> > @@ -206,11 +206,7 @@ config PCIE_TPH
> >  config PCI_P2PDMA
> >  	bool "PCI peer-to-peer transfer support"
> >  	depends on ZONE_DEVICE
> > -	#
> > -	# The need for the scatterlist DMA bus address flag means PCI
> P2PDMA
> > -	# requires 64bit
> > -	#
> > -	depends on 64BIT
> > +	select PCI_P2PDMA_CORE
> >  	select GENERIC_ALLOCATOR
> >  	select NEED_SG_DMA_FLAGS
> >  	help
> 
> Nit: Did we drop depends on 64BIT intentionally here? I guess the full
> PCI_P2PDMA stack still selects NEED_SG_DMA_FLAGS? IIRC,
> NEED_SG_DMA_FLAGS
> doesn't select 64BIT?
> 

seems that comment is stale. According to the commit msg:

" it would make vfio-pci only available if CONFIG_ZONE_DEVICE is
present (e.g. 64-bit systems), "

so it sounds a redundant dependency hence is removed.