9P: hook up rdma_cancelled() into p9_rdma_trans?

From: Paul Bolle
Date: Mon Jul 15 2013 - 04:44:14 EST


0) Building trans_rdma.o (in v3.11-rc1) triggers a warning:
net/9p/trans_rdma.c:594:12: warning: ârdma_cancelledâ defined but not used [-Wunused-function]

1) Did commit 80b45261a0 ("9P: Add cancelled() to the transport
functions.") forget to actually hook up rdma_cancelled() into
p9_rdma_trans()? Say, like this:

diff --git a/net/9p/trans_rdma.c b/net/9p/trans_rdma.c
index 928f2bb..0c42b1c 100644
--- a/net/9p/trans_rdma.c
+++ b/net/9p/trans_rdma.c
@@ -732,6 +732,7 @@ static struct p9_trans_module p9_rdma_trans = {
.close = rdma_close,
.request = rdma_request,
.cancel = rdma_cancel,
+ .cancelled = rdma_cancelled,
};

/**

That (compile tested only) patch silences this warning. Is there a patch
queued to do something like this? I couldn't find it.


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/