contrib/rdmacm-mux: fix clang compilation
authorMarcel Apfelbaum <marcel.apfelbaum@gmail.com>
Fri, 18 Jan 2019 12:46:14 +0000 (14:46 +0200)
committerMarcel Apfelbaum <marcel.apfelbaum@gmail.com>
Sat, 19 Jan 2019 09:01:57 +0000 (11:01 +0200)
Fix Commit a5d2f6f877 (contrib/rdmacm-mux: Add implementation
                       of RDMA User MAD multiplexer).

The above commit introduces a new contrib target, adding a global dependency
to libumad library in case pvrdma configuration option is enabled.
Clang forbids it:
    clang-6.0: error: -libumad: 'linker' input unused
              [-Werror,-Wunused-command-line-argument]

Fix by limiting the scope to the rdmacm-mux target itself.

Reported-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Tested-by: Cornelia Huck <cohuck@redhat.com>
Message-Id: <20190118124614.24548-4-marcel.apfelbaum@gmail.com>
Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
Makefile
contrib/rdmacm-mux/Makefile.objs

index dccba1dca27ffe80be59ee7d8e2407a53bd1e422..c9bdb67274a2b44bdee6eda1e851163f7799594c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -581,6 +581,8 @@ vhost-user-scsi$(EXESUF): $(vhost-user-scsi-obj-y) libvhost-user.a
        $(call LINK, $^)
 vhost-user-blk$(EXESUF): $(vhost-user-blk-obj-y) libvhost-user.a
        $(call LINK, $^)
+
+rdmacm-mux$(EXESUF): LIBS += "-libumad"
 rdmacm-mux$(EXESUF): $(rdmacm-mux-obj-y) $(COMMON_LDADDS)
        $(call LINK, $^)
 
index e1ff4fe5693a833558075f6ca00c91344bf09bad..3df744af89a3a86f9ea8d3b4af2bfa7dbaaeb47d 100644 (file)
@@ -1,4 +1,3 @@
 ifdef CONFIG_PVRDMA
-CFLAGS += -libumad
 rdmacm-mux-obj-y = main.o
 endif