Hi Anna, will this one go in for rc1 as well?
Ben
On 1 Aug 2016, at 15:11, Anna Schumaker wrote:
Hi Ben,
On 07/26/2016 06:58 AM, Benjamin Coddington wrote:
Hi Anna,
On 20 Jun 2016, at 11:38, Benjamin Coddington wrote:
On 20 Jun 2016, at 11:08, Anna Schumaker wrote:
Hi Ben,
On 06/15/2016 03:02 PM, Benjamin Coddington wrote:
A NFSv4 mount of a subdirectory will show an extra slash (as in
'server://path') in proc's mountinfo which will not match the
device name
and path. This can cause problems for programs searching for the
mount.
Fix this by checking for a leading slash in the dentry path, if so
trim
away any trailing slashes in the device name.
Signed-off-by: Benjamin Coddington <bcodding@xxxxxxxxxx>
---
fs/nfs/namespace.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/nfs/namespace.c b/fs/nfs/namespace.c
index c8162c660c44..5551e8ef67fd 100644
--- a/fs/nfs/namespace.c
+++ b/fs/nfs/namespace.c
@@ -98,7 +98,7 @@ rename_retry:
return end;
}
namelen = strlen(base);
- if (flags & NFS_PATH_CANONICAL) {
+ if (*end == '/') {
Are we getting here through nfs_show_devname()? Because it looks
like that function is passing 0 for flags instead of
NFS_PATH_CANONICAL.
We can get here with or withount NFS_PATH_CANONICAL, through
nfs_show_devname() or nfs4_path() or nfs_do_submount().
Earlier in this function we have a check for:
if ((flags & NFS_PATH_CANONICAL) && *end != '/')
Should we be checking if NFS_PATH_CANONICAL is set here, too?
I don't think we need it.
The "if (flags & NFS_PATH_CANONICAL) {" was meant to cover the case
where
NFS_PATH_CANONICAL had earlier added a leading '/' to the path, and
so would
trim away any trailing '/' from the base. So, don't combine
server:/ and
/path into server://path after adding a leading '/' to the path.
The logic here is simplified such that we should _never_ concat
these two:
a base:/ and a /path, otherwise we'll have server://path.
Further confusing the matter is that for v3 mounts, the base is
server:/path
and the path is '/' (which is added by NFS_PATH_CANONICAL), but for
v4
mounts the base is server:/ and the path is /path/to/export
Did this make any sense, or should I try to make it clearer or rework
this?
Your explanation made sense to me, and if tests are failing without
this patch then we should probably go with it.
Thanks,
Anna
This patch seems like just a minor fix, but the double slashes breaks
some mount parsing in tests we have.
Ben
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs"
in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
