February 2021 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: nfs4: fix bad attr comparison in xdr_fattr4_encode
by Jeff Layton (GerritHub) 12 Feb '21

12 Feb '21

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510751 ) Change subject: nfs4: fix bad attr comparison in xdr_fattr4_encode ...................................................................... nfs4: fix bad attr comparison in xdr_fattr4_encode attribute_to_set is a FATTR4_* constant, not an ATTR4_* one. Fix the condition to do the correct comparison. Change-Id: I05320f674faec72a48be611e2700da5928f2e375 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/51/510751/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510751 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: I05320f674faec72a48be611e2700da5928f2e375 Gerrit-Change-Number: 510751 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: nfs4: encode only valid attrs
by Jeff Layton (GerritHub) 12 Feb '21

12 Feb '21

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510750 ) Change subject: nfs4: encode only valid attrs ...................................................................... nfs4: encode only valid attrs I was seeing some warnings like this on Linux clients when accessing shares that had security labels enabled: nfs_setsecurity() 0 security_inode_notifysecctx() -13 The inodes in question had no labels on them, but ganesha was setting the FATTR4_SEC_LABEL bit in the reply and sending a zero-length label in the reply. The ceph code will clear ATTR4_SEC_LABEL in that situation, but the encoder doesn't consult the valid_mask. Skip encoding any attribute that has a legit attrmask bit, when that bit is not set in the valid_mask. Change-Id: I0efb283a0ef0e5fb49bbcb0f1cb03d909161493a Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/50/510750/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510750 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: I0efb283a0ef0e5fb49bbcb0f1cb03d909161493a Gerrit-Change-Number: 510750 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: mdcache: only query for seclabel if it was requested
by Jeff Layton (GerritHub) 12 Feb '21

12 Feb '21

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510749 ) Change subject: mdcache: only query for seclabel if it was requested ...................................................................... mdcache: only query for seclabel if it was requested Change-Id: I383cae4b41ad7a290006899e7743eeb81ec348ef Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h 2 files changed, 12 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/510749/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510749 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: I383cae4b41ad7a290006899e7743eeb81ec348ef Gerrit-Change-Number: 510749 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

FW: VBAT: The 1st Virtual Bakeaton
by Frank Filz 09 Feb '21

09 Feb '21

-----Original Message----- From: Steve Dickson [mailto:SteveD@RedHat.com] Sent: Wednesday, January 13, 2021 3:45 PM To: Linux NFS Mailing List <linux-nfs(a)vger.kernel.org> Cc: NFSv4 <nfsv4(a)ietf.org> Subject: VBAT: The 1st Virtual Bakeaton Hello, Happy New Years! Ben started the conversations about virtual bakeathon and built a network we can use for a VBAT. We were floated the idea of having it on the last week of Feb (Mon 22ed to Fri 26th) which people seems to agree with... so... Lets have a virtual Bakeathon at the end of Feb!!! I've update the nfsv4bat.org site [1] with all the details on how to register and enable a host to used the VBAT network. The VBAT network consists of VPN connections using IPv6 address to do the routing. Again all the details are on [1]. Participant's communications will be run through a Discord server [2], which supports both audio and IRC capabilities... Meaning we can talk and type to each other... Again the details are on [1]. This is the first attempt at a virtual event so there are going to be bumps in the road, but as a community I'm sure we can make this work. Point being... if you find problems please try to come up with a solutions... not a complaint ;-) The official testing start end of Feb but the network is up and running right now! So I *strongly* suggest you try and build a host to used the network well be for the end of Feb... (aka ASAP!) Finally, I'm working on getting VBAT t-shirts for registered participants to order... free of charge... So registering early would definitely help the effort out Let make this happen!!! steved. [1] http://www.nfsv4bat.org/Events/2021/Feb/BAT/index.html [2] https://discord.com/

1 0

Change in ...nfs-ganesha[next]: NLM: Don't allow locks on non-REGULAR_FILE
by Frank Filz (GerritHub) 08 Feb '21

08 Feb '21

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510500 ) Change subject: NLM: Don't allow locks on non-REGULAR_FILE ...................................................................... NLM: Don't allow locks on non-REGULAR_FILE Change-Id: I88c3cb70229042bb01225a46951100d23cfca97a Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NLM/nlm_util.c 1 file changed, 12 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/510500/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510500 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: I88c3cb70229042bb01225a46951100d23cfca97a Gerrit-Change-Number: 510500 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: FSAL_CEPH: use ceph_ll_lookup_vino if available
by Jeff Layton (GerritHub) 08 Feb '21

08 Feb '21

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510467 ) Change subject: FSAL_CEPH: use ceph_ll_lookup_vino if available ...................................................................... FSAL_CEPH: use ceph_ll_lookup_vino if available The original libcephfs interface didn't have a way to reliably look up a snap inode by snapid+inode number if it wasn't already in cache via other means. This has been remedied in libcephfs now, so try to use it if it's available. If it's not, then use a shim that does what the old code did. Change-Id: Ic087fdbadf898a57b9bd1e54f0eed05222a2b65f Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/FSAL/FSAL_CEPH/export.c M src/cmake/modules/FindCEPHFS.cmake M src/include/config-h.in.cmake 3 files changed, 32 insertions(+), 16 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/67/510467/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510467 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: Ic087fdbadf898a57b9bd1e54f0eed05222a2b65f Gerrit-Change-Number: 510467 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Announce Push of V4-dev.50
by Frank Filz 05 Feb '21

05 Feb '21

Branch next Tag:V4-dev.50 Merge Highlights * remove unused assignment * log the set id fail Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 52d3aa5 Frank S. Filz V4-dev.50 76692cd LilyZ log the set id fail 6e63993 LilyZ remove unused assignment

1 0

Change in ...nfs-ganesha[next]: log the set id fail
by LilyZ (GerritHub) 04 Feb '21

04 Feb '21

LilyZ has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510246 ) Change subject: log the set id fail ...................................................................... log the set id fail Signed-off-by: LilyZ <floridsleeves(a)gmail.com> Change-Id: I9ed14c545d2f29c0c7dba30d04c7d5b2f48654e1 --- M src/FSAL/posix_acls.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/510246/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510246 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: I9ed14c545d2f29c0c7dba30d04c7d5b2f48654e1 Gerrit-Change-Number: 510246 Gerrit-PatchSet: 1 Gerrit-Owner: LilyZ <floridsleeves(a)gmail.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: remove unused assignment
by LilyZ (GerritHub) 04 Feb '21

04 Feb '21

LilyZ has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510245 ) Change subject: remove unused assignment ...................................................................... remove unused assignment Signed-off-by: LilyZ <floridsleeves(a)gmail.com> Change-Id: Id3397a48da09f1248d574b11eae8f046b8c83163 --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/45/510245/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/510245 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: Id3397a48da09f1248d574b11eae8f046b8c83163 Gerrit-Change-Number: 510245 Gerrit-PatchSet: 1 Gerrit-Owner: LilyZ <floridsleeves(a)gmail.com> Gerrit-MessageType: newchange

1 0