[S] Change in ...nfs-ganesha[next]: Build pseudo root before creating other FSAL exports
by Xueqian Hu (GerritHub)
Xueqian Hu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1196572?usp=email )
Change subject: Build pseudo root before creating other FSAL exports
......................................................................
Build pseudo root before creating other FSAL exports
Today, we will build pseudo root after parsing and creating all
exports in ganesha conf file. In pseudofs_lookup_path, while
allocating a new fsal_obj_handle, the fileid will start from 0, and
get increased by 1 each time. For a normal case, pseudo root will be
the only export in the PSEUDO FSAL, the fileid of pseudo root will
always be 0. However, if adding other exports in PSEUDO FSAL, and
restarting ganesha, those exports will get created first. That is mean,
if we have NUM exports added, those exports fileids will be
0, 1, ..., (NUM - 1), and pseudo root's fileid will be changed to NUM.
It will be good to keep fileid as 0 for pseudo root, otherwise, for
a running client, it will hit error for inode number change:
[2694122.907113] NFS: server 10.53.93.152 error: fileid changed
In this patch, move the call to build pseudo root before building
other FSAL exports which will make sure pseudo root's file id will
always be 0.
Change-Id: I45563c8d2ffa40858627dd0eb5d4145b046e142b
Signed-off-by: Xueqian Hu <xh140312(a)outlook.com>
---
M src/support/exports.c
1 file changed, 6 insertions(+), 6 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/72/1196572/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1196572?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I45563c8d2ffa40858627dd0eb5d4145b046e142b
Gerrit-Change-Number: 1196572
Gerrit-PatchSet: 1
Gerrit-Owner: Xueqian Hu <xh140312(a)outlook.com>