Change in ...nfs-ganesha[next]: nfs4: fix bad attr comparison in xdr_fattr4_encode
by Jeff Layton (GerritHub)
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
3 years, 9 months
Change in ...nfs-ganesha[next]: nfs4: encode only valid attrs
by Jeff Layton (GerritHub)
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
3 years, 9 months
Change in ...nfs-ganesha[next]: mdcache: only query for seclabel if it was requested
by Jeff Layton (GerritHub)
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
3 years, 9 months
FW: VBAT: The 1st Virtual Bakeaton
by Frank Filz
-----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/
3 years, 9 months
Change in ...nfs-ganesha[next]: FSAL_CEPH: use ceph_ll_lookup_vino if available
by Jeff Layton (GerritHub)
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
3 years, 9 months
Announce Push of V4-dev.50
by Frank Filz
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
3 years, 9 months