Announce Push of V4-dev.35
by Frank Filz
Branch next
Tag:V4-dev.35
Merge Highlights
* Don't deadlock if bindresvport_sa fails.
* Add FSAL_KVSFS
* NLM: fix the bug of using null pointer
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
Contents:
ce3b703 Frank S. Filz V4-dev.35
fffe4c0 sepia-liu NLM: fix the bug of using null pointer
7a9e1d1 Philippe DENIEL Update RPM specfile so that it can build a package
for FSAL_KVSFS
08ceeba Philippe DENIEL Add FSAL_KVSFS
bd1cf64 Solomon Boulos Don't deadlock if bindresvport_sa fails.
4 years, 4 months
Change in ...nfs-ganesha[next]: NLM: fix the bug of using null pointer
by sepia-liu (GerritHub)
sepia-liu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502868 )
Change subject: NLM: fix the bug of using null pointer
......................................................................
NLM: fix the bug of using null pointer
When set enable_NLM false, the uninitialized ht_nlm_client
can not be used, it is NULL pointer.
Signed-off-by: sepia-liu <liuwei_coder(a)163.com>
Change-Id: I6e26452fee158be955e09ca4b41c59d62e325fc8
---
M src/SAL/nfs4_recovery.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/68/502868/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502868
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: I6e26452fee158be955e09ca4b41c59d62e325fc8
Gerrit-Change-Number: 502868
Gerrit-PatchSet: 1
Gerrit-Owner: sepia-liu <liuwei_coder(a)163.com>
Gerrit-MessageType: newchange
4 years, 4 months
Change in ...nfs-ganesha[next]: Don't deadlock if bindresvport_sa fails.
by Solomon Boulos (GerritHub)
Solomon Boulos has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502811 )
Change subject: Don't deadlock if bindresvport_sa fails.
......................................................................
Don't deadlock if bindresvport_sa fails.
My workaround for the bindresvport_sa race ignored the
error case, where I'd still hold onto rpcLock.
Change-Id: Ie8a5786fb283d3f7d255affaa4f0312f1b21374a
Signed-off-by: Solomon Boulos <boulos(a)google.com>
---
M src/FSAL/FSAL_PROXY_V3/rpc.c
1 file changed, 9 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/502811/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502811
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: Ie8a5786fb283d3f7d255affaa4f0312f1b21374a
Gerrit-Change-Number: 502811
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Boulos <boulos(a)google.com>
Gerrit-MessageType: newchange
4 years, 4 months
Announce Push of V4-dev.34
by Frank Filz
Branch next
Tag:V4-dev.34
Merge Highlights
* PROXY_V3: work around bindresvport threadsafety.
* Fix two clang build errors.
* Move nfs_init_stats_time outside USE_DBUS.
* In nfs_export_get_root_entry release ref if not dir also fix locking
* Introduce fsal_lookup_path helper
* Specify & for inline asm output operands
* FSAL_CEPH: paper over -ENOTCONN return from close when shutting down
* Handle error conversion for ENXIO for NFS V3 and V4.
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
Contents:
5b5ab87 Frank S. Filz V4-dev.34
8b75a13 Gaurav Gangalwar Handle error conversion for ENXIO for NFS V3 and
V4.
5af0950 Jeff Layton FSAL_CEPH: paper over -ENOTCONN return from close when
shutting down
9a8489b Matthew DeVore Specify & for inline asm output operands
2acd535 Frank S. Filz Introduce fsal_lookup_path helper
edb9d74 Frank S. Filz In nfs_export_get_root_entry release ref if not dir
also fix locking
cb72562 Solomon Boulos Move nfs_init_stats_time outside USE_DBUS.
ef71584 Solomon Boulos Fix two clang build errors.
48c464b Solomon Boulos PROXY_V3: work around bindresvport threadsafety.
4 years, 4 months
Change in ...nfs-ganesha[next]: FSAL_CEPH: paper over -ENOTCONN return from close when shutting down
by Jeff Layton (GerritHub)
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502619 )
Change subject: FSAL_CEPH: paper over -ENOTCONN return from close when shutting down
......................................................................
FSAL_CEPH: paper over -ENOTCONN return from close when shutting down
When we're shutting down the server, we'll usually abort the connection
first. The mdcache will then try to clean up entries and issue a ->close
to each, and FSAL_CEPH ends up returning -ENOTCONN in that situation
which causes a lot of log spam.
Fix this by just ignoring -ENOTCONN errors in ceph_close_my_fd when
ganesha is shutting down.
Change-Id: If8231998a61e759be4d044f102ae6dbcebfdc975
Reported-and-Tested-by: liuwei <liuwei_coder(a)163.com>
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/FSAL/FSAL_CEPH/handle.c
1 file changed, 11 insertions(+), 3 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/502619/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502619
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: If8231998a61e759be4d044f102ae6dbcebfdc975
Gerrit-Change-Number: 502619
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
Gerrit-MessageType: newchange
4 years, 4 months
Change in ...nfs-ganesha[next]: Introduce fsal_lookup_path helper
by Frank Filz (GerritHub)
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502536 )
Change subject: Introduce fsal_lookup_path helper
......................................................................
Introduce fsal_lookup_path helper
Instead of using FSAL API lookup_path method to accomplish subdir mounts
(and 9P attach), do a path walk in fsal_helper.c using the normal lookup
method. This brings that path walk into common code applicable to all
FSALs and works if mount_path_pseudo is true (in which case the mount
path may not be able to be looked up in the backend). The helper function
checks that each component is not ".." and is a directory. This prevents
security hacking by using ".." to escape the export or using a symlink to
escape the export. It also neatly assures we never allow mounting a single
file. We also allow extra '/' in the path beyond the export root and allow
"." components.
Change-Id: I301a7cf34185fd7f57c73c21a069366d92abd25f
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/FSAL/fsal_helper.c
M src/Protocols/9P/9p_attach.c
M src/Protocols/NFS/mnt_Mnt.c
M src/include/fsal.h
4 files changed, 181 insertions(+), 42 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/36/502536/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502536
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: I301a7cf34185fd7f57c73c21a069366d92abd25f
Gerrit-Change-Number: 502536
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
Gerrit-MessageType: newchange
4 years, 4 months
Change in ...nfs-ganesha[next]: Proxy V3: Support nested (subdirectory) mounts.
by Solomon Boulos (GerritHub)
Solomon Boulos has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502442 )
Change subject: Proxy V3: Support nested (subdirectory) mounts.
......................................................................
Proxy V3: Support nested (subdirectory) mounts.
There's been an @todo in lookup_path since my
first version of this code with "oh yeah, go
through and look for forward slashes". I decided
to finally handle that :).
I tested both by hand:
sudo mount -t nfs -o proto=vers=3 ${SERVER}:/export/nested /mnt/sub
and with a cthon run of the same nested export.
Change-Id: Ib4cff04b1e3a49114b8b19c4e69257e8004d0bfe
Signed-off-by: Solomon Boulos <boulos(a)google.com>
---
M src/FSAL/FSAL_PROXY_V3/main.c
1 file changed, 61 insertions(+), 16 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/42/502442/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502442
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: Ib4cff04b1e3a49114b8b19c4e69257e8004d0bfe
Gerrit-Change-Number: 502442
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Boulos <boulos(a)google.com>
Gerrit-MessageType: newchange
4 years, 4 months