diff mbox

[rdma-core] debian: Whitelist coherent DMA supporting architectures

Message ID 20180430170043.414-1-benjamin.drung@profitbricks.com (mailing list archive)
State Accepted
Headers show

Commit Message

Benjamin Drung April 30, 2018, 5 p.m. UTC
rdma-core defines `udma_to_device_barrier`, `udma_from_device_barrier`,
and `mmio_flush_writes` in `util/udma_barrier.h` for architectures that
support coherent DMA. Instead of blacklisting the unsupported
architectures, use an explicit whitelist. This will disable coherent DMA
support for riscv64 (RISC-V 64 bits little-endian).

Closes: #894995
Signed-off-by: Benjamin Drung <benjamin.drung@profitbricks.com>
---
 debian/rules | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Doug Ledford May 1, 2018, 3:43 p.m. UTC | #1
On Mon, 2018-04-30 at 19:00 +0200, Benjamin Drung wrote:
> rdma-core defines `udma_to_device_barrier`, `udma_from_device_barrier`,
> and `mmio_flush_writes` in `util/udma_barrier.h` for architectures that
> support coherent DMA. Instead of blacklisting the unsupported
> architectures, use an explicit whitelist. This will disable coherent DMA
> support for riscv64 (RISC-V 64 bits little-endian).
> 
> Closes: #894995
> Signed-off-by: Benjamin Drung <benjamin.drung@profitbricks.com>

Thanks, applied.
diff mbox

Patch

diff --git a/debian/rules b/debian/rules
index 1fa4a4eb..74312181 100755
--- a/debian/rules
+++ b/debian/rules
@@ -4,7 +4,7 @@  include /usr/share/dpkg/architecture.mk
 
 export DEB_BUILD_MAINT_OPTIONS=hardening=+all
 
-NO_COHERENT_DMA_ARCHS = alpha armel armhf hppa m68k mips mips64el mipsel sh4
+COHERENT_DMA_ARCHS = amd64 arm64 i386 ia64 powerpc powerpcspe ppc64 ppc64el s390x sparc64 x32
 
 %:
 	dh $@ --builddirectory=build-deb
@@ -43,7 +43,7 @@  override_dh_auto_test:
 
 override_dh_auto_install:
 # Some providers are disabled on architectures that are not able to do coherent DMA
-ifeq (,$(filter-out $(NO_COHERENT_DMA_ARCHS),$(DEB_HOST_ARCH)))
+ifneq (,$(filter-out $(COHERENT_DMA_ARCHS),$(DEB_HOST_ARCH)))
 	for package in ibverbs-providers libibverbs-dev rdma-core; do \
 		test -e debian/$$package.install.backup || cp debian/$$package.install debian/$$package.install.backup; \
 	done