From Jeff Layton <jlayton@redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/421129 Change subject: fs_locations: split location field into server and rootpath strings ...................................................................... fs_locations: split location field into server and rootpath strings Sriram did some work a few months ago to plumb in a new vfs-layer mechanism for fetching fs_locations info. Unfortunately, it ended up squashing the "server" and "rootpath" fields into a single string, delimited by a ':'. Presumably this was done because the xattr presents this info in that format, as later in the generic code, that string is split into two for encoding the fs_locations attr on the wire. In later patches we're going to want to make this more suitable for generating fs_locations_info, and keeping them conjoined like this will be more problematic for that use later. Split the location string back into two in fsal_fs_locations_t, and just have FSAL_VFS concatenate a new ':' delimited string when it needs to generate one for the fsid hash. Change-Id: If3ea29d6c0053b8aeb4644c26dc46c2c12b3e88d Signed-off-by: Jeff Layton <jlayton@redhat.com> --- M src/FSAL/FSAL_GPFS/handle.c M src/FSAL/FSAL_VFS/handle.c M src/FSAL/FSAL_VFS/subfsal_helpers.c M src/Protocols/NFS/nfs_proto_tools.c M src/include/fsal_types.h M src/include/nfs4_fs_locations.h M src/support/nfs4_fs_locations.c 7 files changed, 51 insertions(+), 40 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/421129/1 -- To view, visit https://review.gerrithub.io/421129 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: If3ea29d6c0053b8aeb4644c26dc46c2c12b3e88d Gerrit-Change-Number: 421129 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton@redhat.com>