Re: [PATCH 4/5] NFS: remove RPC PipeFS mount point reference from blocklayout routines

On Tue, Nov 29, 2011 at 9:35 PM, Myklebust, Trond
[off-list ref] wrote:

quoted

-----Original Message-----
From: tao.peng-mb1K0bWo544@public.gmane.org [mailto:tao.peng-mb1K0bWo544@public.gmane.org]
Sent: Tuesday, November 29, 2011 7:40 AM
To: skinsbursky-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org
Cc: Myklebust, Trond; linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; xemul-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org;
neilb-l3A5Bk7waGM@public.gmane.org; netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
jbottomley-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org; bfields-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org; davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org;
devel-GEFAQzZX7r8dnm+yROfE0A@public.gmane.org
Subject: RE: [PATCH 4/5] NFS: remove RPC PipeFS mount point reference
from blocklayout routines

quoted

-----Original Message-----
From: Stanislav Kinsbursky [mailto:skinsbursky-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org]
Sent: Tuesday, November 29, 2011 8:19 PM
To: Peng, Tao
Cc: Trond.Myklebust-HgOvQuBEEgTQT0dZR+AlfA@public.gmane.org; linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Pavel
Emelianov; neilb-l3A5Bk7waGM@public.gmane.org; netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; James Bottomley; bfields-uC3wQj2KruMpug/h7KTFAQ@public.gmane.orgg;
davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org; devel-GEFAQzZX7r8dnm+yROfE0A@public.gmane.org
Subject: Re: [PATCH 4/5] NFS: remove RPC PipeFS mount point reference
from blocklayout routines

29.11.2011 16:00, tao.peng-mb1K0bWo544@public.gmane.org пишет:

quoted

quoted

-----Original Message-----
From: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
[mailto:linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org] On Behalf Of

Stanislav

quoted

quoted

Kinsbursky
Sent: Tuesday, November 29, 2011 6:11 PM
To: Trond.Myklebust-HgOvQuBEEgTQT0dZR+AlfA@public.gmane.org
Cc: linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; xemul-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org; neilb@suse.de;
netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux- kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
jbottomley-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org; bfields-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org;
davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org; devel-GEFAQzZX7r8dnm+yROfE0A@public.gmane.org
Subject: [PATCH 4/5] NFS: remove RPC PipeFS mount point reference
from blocklayout routines

This is a cleanup patch. We don't need this reference anymore,
because blocklayout pipes dentries now creates and destroys in
per-net operations and on PipeFS mount/umount notification.
Note that nfs4blocklayout_register_net() now returns 0 instead of
-ENOENT in case of PipeFS superblock absence. This is ok, because
blocklayout pipe dentry will be created on PipeFS mount event.

When is the "pipefs mount event" going to happen? When inserting
kernel modules or when user issues

mount command?

quoted

When user issues mount command.
Kernel mounts of PipeFS has been removed with all these patch sets
I've sent already.

Then it is going to break blocklayout user space program blkmapd, which is
stared before mounting any file system and it tries to open the pipe file
when started.

Why on earth is blkmapd doing this instead of listening for file creation notifications like the other rpc_pipefs daemons do?

Not sure how the original implementer chose this but I think it is
likely because we do not expect the pipe file to be created or deleted
dynamically.

-- 
Thanks,
Tao
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html