Change in ...nfs-ganesha[next]: Drop dirent ref when releasing dirent
by Daniel Gryniewicz (GerritHub)
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443260
Change subject: Drop dirent ref when releasing dirent
......................................................................
Drop dirent ref when releasing dirent
During a readdir, the dirents in a chunk take a ref on the entry they
point to, so that the entry can't be reaped until the chunk has been
processed. Make sure that, if a dirent is released while it still has a
ref, we drop the ref. Otherwise, under heavy create/delete/readdir
load, we can leak refs to entries.
Change-Id: Id759bb40f33c92867c1fda93f9449433f686d4d5
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_avl.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
2 files changed, 28 insertions(+), 24 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/60/443260/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443260
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: Id759bb40f33c92867c1fda93f9449433f686d4d5
Gerrit-Change-Number: 443260
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
Gerrit-MessageType: newchange
5 years, 10 months
Announce Push of V2.8-dev.16
by Frank Filz
Branch next
Tag:V2.8-dev.16
NOTE: This merge includes an ntirpc pullup - please update your submodule
Release Highlights
* 9p: don't try to shut down workers if they were never started
* RADOS URLS configuration
* Status of "v4_full" is missing
* locktest does not handle EACCES error for fcntl() F_SETLK operation
* Fix missing object reference in write_data
* Take state_lock before checking for deleg conflicts
* SAL: Handle open claim of type OPEN_DELEG_CUR_FH
* Pull up strl* fixes
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
Contents:
7af8eb8 Frank S. Filz V2.8-dev.16
5f1ce86 Daniel Gryniewicz Pull up strl* fixes
4ac1eaf Soumya Koduri SAL: Handle open claim of type OPEN_DELEG_CUR_FH
9d1b48b Soumya Koduri Take state_lock before checking for deleg conflicts
86dd7fe Soumya Koduri Fix missing object reference in write_data
6761cad Trishali Nayar locktest does not handle EACCES error for fcntl()
F_SETLK operation
5c1dfa7 Sachin Punadikar Status of "v4_full" is missing
b75f5f8 Jeff Layton rados_urls: set up watch if RADOS_URLS block configures
it
53bb21b Jeff Layton rados_urls: break out URL parsing into helper function
ac799d0 Jeff Layton rados_urls: break out rados client setup code into
separate function
441db0c Jeff Layton rados_urls: clean up the RADOS_URLS #ifdef-ery
f96a8f6 Jeff Layton rados_urls: make rados_urls_set_param_from_conf static
0347da7 Jeff Layton config_parsing: shut down the URL parser(s) on shutdown
82629d8 Jeff Layton doc: add a RADOS_URLS section to ganesha-core-config.rst
7f37067 Jeff Layton 9p: don't try to shut down workers if they were never
started
5 years, 10 months
Change in ...nfs-ganesha[next]: rados_urls: set up watch if RADOS_URLS block configures it
by Jeff Layton (GerritHub)
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443013
Change subject: rados_urls: set up watch if RADOS_URLS block configures it
......................................................................
rados_urls: set up watch if RADOS_URLS block configures it
We can send ganesha a SIGHUP to tell it to reread its configuration. In
a containerized environment however, sending such a signal represents a
potential security issue as we need to open up permissions to allow it.
This patch adds a new watch_url option to the RADOS_URLS config block.
When that points to a valid RADOS object, set a watch on that object.
When we get a notification on that object, send a SIGHUP to ourselves
to force a configuration reload.
Change-Id: Ia83329eb0774f8b0bff428b60fc72df1a70b47c8
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/MainNFSD/nfs_admin_thread.c
M src/MainNFSD/nfs_init.c
M src/config_parsing/conf_url_rados.c
M src/config_samples/ceph.conf
M src/doc/man/ganesha-core-config.rst
R src/include/conf_url_rados.h
6 files changed, 135 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/13/443013/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443013
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: Ia83329eb0774f8b0bff428b60fc72df1a70b47c8
Gerrit-Change-Number: 443013
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
Gerrit-MessageType: newchange
5 years, 10 months
Change in ...nfs-ganesha[next]: rados_urls: clean up the RADOS_URLS #ifdef-ery
by Jeff Layton (GerritHub)
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443010
Change subject: rados_urls: clean up the RADOS_URLS #ifdef-ery
......................................................................
rados_urls: clean up the RADOS_URLS #ifdef-ery
Define a no-op static inline that will be used if RADOS_URLS is
disabled, and get rid of the #ifdef RADOS_URLS calls sprinkled
throughout the code.
Change-Id: I85f8dca73711a64ed06a5039eb3d87456c45eae4
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/config_parsing/conf_url.c
M src/config_parsing/conf_url_rados.c
M src/config_parsing/conf_url_rados.h
3 files changed, 6 insertions(+), 9 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/10/443010/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443010
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: I85f8dca73711a64ed06a5039eb3d87456c45eae4
Gerrit-Change-Number: 443010
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
Gerrit-MessageType: newchange
5 years, 10 months