Re: [Devel] [PATCH] SUNRPC: search for service transports in networknamespace context

From: Stanislav Kinsbursky
Date: Tue Dec 20 2011 - 05:31:23 EST


20.12.2011 14:25, Glauber Costa ÐÐÑÐÑ:
On 12/20/2011 03:14 PM, Stanislav Kinsbursky wrote:
Service transports are parametrized by network namespace. And thus lookup of
transport instance have to take network namespace into account.

Signed-off-by: Stanislav Kinsbursky<skinsbursky@xxxxxxxxxxxxx>

---
fs/lockd/svc.c | 2 +-
fs/nfsd/nfsctl.c | 4 ++--
include/linux/sunrpc/svc_xprt.h | 3 ++-
net/sunrpc/svc_xprt.c | 6 +++++-
4 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/fs/lockd/svc.c b/fs/lockd/svc.c
index c061b9a..ff379ff 100644
--- a/fs/lockd/svc.c
+++ b/fs/lockd/svc.c
@@ -193,7 +193,7 @@ static int create_lockd_listener(struct svc_serv *serv, const char *name,
{
struct svc_xprt *xprt;

- xprt = svc_find_xprt(serv, name, family, 0);
+ xprt = svc_find_xprt(serv, name,&init_net, family, 0);
if (xprt == NULL)
return svc_create_xprt(serv, name,&init_net, family, port,
SVC_SOCK_DEFAULTS);

I might be missing something, but isn't it (at least part of it) already
merged in commit fc5d00b04a3a58cac8620403dfe9f43f72578ec1 ?


Nope, those one was parameterizing transport creation by network namespace.
This one is parameterizing search in poll of existent service transports by network namespace.

--
Best regards,
Stanislav Kinsbursky
--
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/