February 2019 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: Drop dirent ref when releasing dirent
by Daniel Gryniewicz (GerritHub) 04 Feb '19

04 Feb '19

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

1 0

Change in ...nfs-ganesha[next]: config_samples: properly quote and terminate the sample watch_url set...
by Jeff Layton (GerritHub) 02 Feb '19

02 Feb '19

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443081 Change subject: config_samples: properly quote and terminate the sample watch_url setting ...................................................................... config_samples: properly quote and terminate the sample watch_url setting Change-Id: I932d06d1241d4dcaf1468c3634f45cf279c8d0ef Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_samples/ceph.conf 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/443081/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443081 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: I932d06d1241d4dcaf1468c3634f45cf279c8d0ef Gerrit-Change-Number: 443081 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Announce Push of V2.8-dev.16
by Frank Filz 02 Feb '19

02 Feb '19

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

1 0

Updated: Ganesha Community Call
by Frank Filz via BlueJeans Network 01 Feb '19

01 Feb '19

To join the meeting on a computer or mobile phone: https://bluejeans.com/373354489?src=textEmail Frank Filz has updated the information for your meeting. To join from a Red Hat Deskphone or Softphone, dial: 84336. Connecting directly from a room system? 1.) Dial: 199.48.152.152 or bjn.vc 2.) Enter Meeting ID: 373354489 Just want to dial in on your phone? 1.) Dial one of the following numbers: 408-915-6466 (US) See all numbers: https://www.redhat.com/en/conference-numbers 2.) Enter Meeting ID: 373354489 3.) Press # Want to test your video connection? https://bluejeans.com/111

1 0

Change in ...nfs-ganesha[next]: Pull up strl* fixes
by Daniel Gryniewicz (GerritHub) 01 Feb '19

01 Feb '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443015 Change subject: Pull up strl* fixes ...................................................................... Pull up strl* fixes Change-Id: I9486aeca6360b9ed1e0f9642eb0437a29d4d13f5 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/libntirpc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/15/443015/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443015 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: I9486aeca6360b9ed1e0f9642eb0437a29d4d13f5 Gerrit-Change-Number: 443015 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: rados_urls: set up watch if RADOS_URLS block configures it
by Jeff Layton (GerritHub) 01 Feb '19

01 Feb '19

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

1 0

Change in ...nfs-ganesha[next]: rados_urls: break out URL parsing into helper function
by Jeff Layton (GerritHub) 01 Feb '19

01 Feb '19

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443012 Change subject: rados_urls: break out URL parsing into helper function ...................................................................... rados_urls: break out URL parsing into helper function Change-Id: Ib93e4c3f28f1d4a076a80e374c0ffed07d052365 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 61 insertions(+), 50 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/12/443012/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443012 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: Ib93e4c3f28f1d4a076a80e374c0ffed07d052365 Gerrit-Change-Number: 443012 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: rados_urls: break out rados client setup code into separate function
by Jeff Layton (GerritHub) 01 Feb '19

01 Feb '19

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443011 Change subject: rados_urls: break out rados client setup code into separate function ...................................................................... rados_urls: break out rados client setup code into separate function Change-Id: I0d2c351f74218ba87eecb3b9f7c680d660fdbc2e Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 36 insertions(+), 27 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/443011/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443011 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: I0d2c351f74218ba87eecb3b9f7c680d660fdbc2e Gerrit-Change-Number: 443011 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: rados_urls: clean up the RADOS_URLS #ifdef-ery
by Jeff Layton (GerritHub) 01 Feb '19

01 Feb '19

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

1 0

Change in ...nfs-ganesha[next]: rados_urls: make rados_urls_set_param_from_conf static
by Jeff Layton (GerritHub) 01 Feb '19

01 Feb '19

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443009 Change subject: rados_urls: make rados_urls_set_param_from_conf static ...................................................................... rados_urls: make rados_urls_set_param_from_conf static Change-Id: Ib01cf1660e46a9a887f26343d4c47bebc049bcd4 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/09/443009/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/443009 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: Ib01cf1660e46a9a887f26343d4c47bebc049bcd4 Gerrit-Change-Number: 443009 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0