Thread (11 messages) 11 messages, 5 authors, 2019-07-17

Re: linux-next: build failure after merge of the rdma tree

From: Mark Zhang <hidden>
Date: 2019-07-09 07:04:22
Also in: lkml
Subsystem: infiniband subsystem, the rest · Maintainers: Jason Gunthorpe, Leon Romanovsky, Linus Torvalds

Hi Stephen,

Can you please try the patch below, thank you.

net/mlx5: Remove include ib_verbs.h in rdma_counter.h

rdma_counter.h include ib_verbs.h which in turn needs rdma_port_counter
from rdma_counter.h, but it is not defined yet.

Fixes: 413d3347503b ("RDMA/counter: Add set/clear per-port auto mode 
support")
Signed-off-by: Mark Zhang <redacted>
diff --git a/include/rdma/rdma_counter.h b/include/rdma/rdma_counter.h
index 68827700ba95..eb99856e8b30 100644
--- a/include/rdma/rdma_counter.h
+++ b/include/rdma/rdma_counter.h
@@ -9,10 +9,10 @@
  #include <linux/mutex.h>
  #include <linux/pid_namespace.h>

-#include <rdma/ib_verbs.h>
  #include <rdma/restrack.h>
  #include <rdma/rdma_netlink.h>

+struct ib_device;
  struct ib_qp;

  struct auto_mode_param {
On 7/9/2019 11:30 AM, Stephen Rothwell wrote:
quoted hunk ↗ jump to hunk
Hi all,

After merging the rdma tree, today's linux-next build (x86_64
allmodconfig) failed like this:

In file included from /home/sfr/next/next/include/rdma/rdma_counter.h:12,
                  from <command-line>:
/home/sfr/next/next/include/rdma/ib_verbs.h:2126:27: error: field 'port_counter' has incomplete type
   struct rdma_port_counter port_counter;
                            ^~~~~~~~~~~~

Caused by commit

   413d3347503b ("RDMA/counter: Add set/clear per-port auto mode support")

rdma_counter.h include ib_verbs.h which in turn needs rdma_port_counter
from rdma_counter.h, but it is not defined yet :-(

I have applied the following patch for today.

From: Stephen Rothwell <redacted>
Date: Tue, 9 Jul 2019 13:17:49 +1000
Subject: [PATCH] RDMA: don't try to build rdma_counter.h for now

rdma_counter.h include ib_verbs.h which in turn needs rdma_port_counter
from rdma_counter.h, but it is not defined yet :-(

Signed-off-by: Stephen Rothwell <redacted>
---
  include/Kbuild | 1 +
  1 file changed, 1 insertion(+)
diff --git a/include/Kbuild b/include/Kbuild
index 78434c59701f..8dab85cdf4f4 100644
--- a/include/Kbuild
+++ b/include/Kbuild
@@ -939,6 +939,7 @@ header-test-			+= rdma/ib.h
  header-test-			+= rdma/iw_portmap.h
  header-test-			+= rdma/opa_port_info.h
  header-test-			+= rdma/rdmavt_cq.h
+header-test-			+= rdma/rdma_counter.h
  header-test-			+= rdma/restrack.h
  header-test-			+= rdma/signature.h
  header-test-			+= rdma/tid_rdma_defs.h
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help