Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Bernard Metzler
Dalessandro" >, "Mike Marciniszyn" >, linux-r...@vger.kernel.org, >linux-...@vger.kernel.org, iommu@lists.linux-foundation.org >Subject: [EXTERNAL] [PATCH 1/6] RMDA/sw: don't allow drivers using >dma_virt_ops on highmem configs > >dma_virt_ops requires that all p

Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Christoph Hellwig
On Thu, Nov 05, 2020 at 03:29:58PM +, Robin Murphy wrote: > It's commonly done using the "def_bool" shorthand. I fact, I think simply > "def_bool !HIGHMEM" would suffice for the fundamental definition here. Indeed, I'll switch it over. ___ iommu

Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Christoph Hellwig
On Thu, Nov 05, 2020 at 12:15:46PM +, Robin Murphy wrote: > On 2020-11-05 07:42, Christoph Hellwig wrote: >> dma_virt_ops requires that all pages have a kernel virtual address. >> Introduce a INFINIBAND_VIRT_DMA Kconfig symbol that depends on !HIGHMEM >> and a large enough dma_addr_t, and make

Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Robin Murphy
On 2020-11-05 14:41, Jason Gunthorpe wrote: On Thu, Nov 05, 2020 at 08:42:00AM +0100, Christoph Hellwig wrote: dma_virt_ops requires that all pages have a kernel virtual address. Introduce a INFINIBAND_VIRT_DMA Kconfig symbol that depends on !HIGHMEM and a large enough dma_addr_t, and make all

Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Jason Gunthorpe
On Thu, Nov 05, 2020 at 08:42:00AM +0100, Christoph Hellwig wrote: > dma_virt_ops requires that all pages have a kernel virtual address. > Introduce a INFINIBAND_VIRT_DMA Kconfig symbol that depends on !HIGHMEM > and a large enough dma_addr_t, and make all three driver depend on the > new symbol.

Re: [PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-05 Thread Robin Murphy
On 2020-11-05 07:42, Christoph Hellwig wrote: dma_virt_ops requires that all pages have a kernel virtual address. Introduce a INFINIBAND_VIRT_DMA Kconfig symbol that depends on !HIGHMEM and a large enough dma_addr_t, and make all three driver depend on the new symbol. Signed-off-by: Christoph

[PATCH 1/6] RMDA/sw: don't allow drivers using dma_virt_ops on highmem configs

2020-11-04 Thread Christoph Hellwig
dma_virt_ops requires that all pages have a kernel virtual address. Introduce a INFINIBAND_VIRT_DMA Kconfig symbol that depends on !HIGHMEM and a large enough dma_addr_t, and make all three driver depend on the new symbol. Signed-off-by: Christoph Hellwig --- drivers/infiniband/Kconfig