October 2019 - Devel - NFS Ganesha List Archives

RHEL 7.2 clients cannot use NFS 4.1 to Nfs Ganesha 2.7.6 shares
by Rungta, Vandana 09 Oct '19

09 Oct '19

Hello, With Nfs Ganesha 2.7.6 when we use NFS clients on older kernels ( RHEL 7.2 – 3.10.0-327.el7.x86_64 ) to mount a share using NFS vers=4.1 all file operations fail. Clients can navigate directories, but cannot read/write data to any file. Open fails because the older kernel cannot handle OPEN_DELEGATE_NONE_EXT specified in the Open response. If OPEN_DELEGATE_NONE_EXT was returned only when OPEN4_SHARE_ACCESS_WANT_XXX was specified, then we could be backwards compatible. With Nfs Ganesha 2.5.x ( no delegation support), NFS clients on the older kernels can successfully mount and use the share with NFS vers 4.1 ( Also reported here: https://lists.nfs-ganesha.org/archives/list/devel@lists.nfs-ganesha.org/thr… ) Thoughts? Thanks, Vandana

2 1

Change in ...nfs-ganesha[next]: Only return OPEN_DELEGATE_NONE_EXT if delegations WANTed
by Frank Filz (GerritHub) 09 Oct '19

09 Oct '19

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470892 ) Change subject: Only return OPEN_DELEGATE_NONE_EXT if delegations WANTed ...................................................................... Only return OPEN_DELEGATE_NONE_EXT if delegations WANTed Only return OPEN_DELEGATE_NONE_EXT if one of the OPEN4_SHARE_ACCESS_WANT_DELEG_xxx flags is specified on the request, otherwise stick with OPEN_DELEGATE_NONE. The code will fill in delegation.open_delegation4_u.od_whynone.ond_why for any delegation rejection, however, the XDR will ignore that if delegation.delegation_type is OPEN_DELEGATE_NONE. Change-Id: Ia14c93b2835f7c6ce3cbd5b2273b4b0398c362c3 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/nfs4_op_open.c 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/470892/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470892 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ia14c93b2835f7c6ce3cbd5b2273b4b0398c362c3 Gerrit-Change-Number: 470892 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Remove useless pointer variable.
by xijinyu (GerritHub) 09 Oct '19

09 Oct '19

xijinyu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470831 ) Change subject: Remove useless pointer variable. ...................................................................... Remove useless pointer variable. Signed-off-by: Xi Jinyu <xijinyu(a)cmss.chinamobile.com> Change-Id: I6ac49ded8b5376288595259cb958f1379c7fe627 --- M src/os/freebsd/mntent_compat.c 1 file changed, 0 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/470831/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470831 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I6ac49ded8b5376288595259cb958f1379c7fe627 Gerrit-Change-Number: 470831 Gerrit-PatchSet: 1 Gerrit-Owner: xijinyu <xijinyu(a)cmss.chinamobile.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: MOUNT: Add mnt3_ prefix to common typedef names
by Frank Filz (GerritHub) 08 Oct '19

08 Oct '19

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470782 ) Change subject: MOUNT: Add mnt3_ prefix to common typedef names ...................................................................... MOUNT: Add mnt3_ prefix to common typedef names Change-Id: I0396cac66341ec8250661e89977d30c4965527a6 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/mnt_Export.c M src/Protocols/XDR/xdr_mount.c M src/include/mount.h M src/include/nfs_proto_data.h 4 files changed, 23 insertions(+), 23 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/82/470782/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470782 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I0396cac66341ec8250661e89977d30c4965527a6 Gerrit-Change-Number: 470782 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: PROXY: use MAXNAMLEN instead of sizeof(name)
by Frank Filz (GerritHub) 08 Oct '19

08 Oct '19

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470781 ) Change subject: PROXY: use MAXNAMLEN instead of sizeof(name) ...................................................................... PROXY: use MAXNAMLEN instead of sizeof(name) The local variable "name" was removed, so we fell back to an unfortunate typedef of name in mount.h. Change-Id: I99a4a1d8c7e162083966c5318072ed07b97f34e6 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_PROXY/handle.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/470781/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470781 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I99a4a1d8c7e162083966c5318072ed07b97f34e6 Gerrit-Change-Number: 470781 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

NOTICE: V2.9-rc1 merge will be Thursday morning Pacific Time
by Frank Filz 08 Oct '19

08 Oct '19

In preparation for Bakeathon, I will be merging and tagging V2.9-rc1 Thursday morning Pacific Time. We will have 2-4 weeks or so of rc merges before finally merging V2.9.0. In this light, if you have patches that are still under review, please work to resolve any review issues and have the patches ready for merge. Thanks Frank Filz

1 0

Change in ...nfs-ganesha[next]: fix for race condition between fridgethr_destroy and fridgethr_freeze
by Name of user not set (GerritHub) 08 Oct '19

08 Oct '19

koen.struyve(a)hgst.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470764 ) Change subject: fix for race condition between fridgethr_destroy and fridgethr_freeze ...................................................................... fix for race condition between fridgethr_destroy and fridgethr_freeze Change-Id: Icb1b8911602aabd1c81bd2ee725984933c96db69 Signed-off-by: Koen Struyve <koen.struyve(a)wdc.com> --- M src/support/fridgethr.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/64/470764/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470764 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Icb1b8911602aabd1c81bd2ee725984933c96db69 Gerrit-Change-Number: 470764 Gerrit-PatchSet: 1 Gerrit-Owner: koen.struyve(a)hgst.com Gerrit-MessageType: newchange

1 0

Announce Push of V2.9-dev.11
by Frank Filz 04 Oct '19

04 Oct '19

Branch next Tag:V2.9-dev.11 Release Highlights * Updating component_log_level not getting reflected in FSALs. * Make LizardFS FSAL compatible with new NFS-ganesha API * Do not collect latency stats for client * Scripts install fixes * Add get options to ganesha_conf tool Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: e23090c Frank S. Filz V2.9-dev.11 556759d Xi Jinyu Add get options to ganesha_conf tool 4ea7629 Daniel Gryniewicz Scripts install fixes b743659 Sachin Punadikar Do not collect latency stats for client 041a2d7 Patryk Bęza Make LizardFS FSAL compatible with new NFS-ganesha API 04e33a9 ashish.sangwan Updating component_log_level not getting reflected in FSALs. Compile ganesha.nfsd binary with fPIC

1 0

Change in ...nfs-ganesha[next]: Make NFSv4 READDIR use xdr_putbuffs
by Frank Filz (GerritHub) 04 Oct '19

04 Oct '19

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470525 ) Change subject: Make NFSv4 READDIR use xdr_putbuffs ...................................................................... Make NFSv4 READDIR use xdr_putbuffs Encode all the directory entries into an xdrmem buffer that can then be added to the ioq via xdr_putbufs. We also do a more exact calulation of the space available for the directory entries so significantly more entries can be returned if the entries are small. Change-Id: I6975dcb6758a898d773c3a4c90e4c90e96638acd Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/nfs4_op_readdir.c M src/Protocols/NFS/nfs_proto_tools.c M src/include/nfs_proto_tools.h M src/include/nfsv41.h 4 files changed, 504 insertions(+), 252 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/25/470525/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470525 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I6975dcb6758a898d773c3a4c90e4c90e96638acd Gerrit-Change-Number: 470525 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Per client and per export stats
by Sachin Punadikar (GerritHub) 04 Oct '19

04 Oct '19

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470445 ) Change subject: Per client and per export stats ...................................................................... Per client and per export stats Currently ganesha_stats provides indications that whether stats are available per client/export for different versions of NFS protocol, but details are not available. Provision is made to extract few statistics per client and per export for different NFS protocol versions being served. These stats would give some insights on what kind of workload is being exerted by a client and what are typical latencies for an export. Change-Id: I5ba3ad7d8d71f75f8f314b907f55eb8335d1f112 Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/include/server_stats_private.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/scripts/ganeshactl/ganesha_stats.py M src/support/client_mgr.c M src/support/export_mgr.c M src/support/server_stats.c 6 files changed, 508 insertions(+), 12 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/45/470445/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/470445 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I5ba3ad7d8d71f75f8f314b907f55eb8335d1f112 Gerrit-Change-Number: 470445 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com> Gerrit-MessageType: newchange

1 0