10:50 a.m.
Daniel Gryniewicz has uploaded this change for review. (
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929
Change subject: MDCACHE - Add MDCACHE {} config block
......................................................................
MDCACHE - Add MDCACHE {} config block
Add a config block name MDCACHE that is a copy of CACHEINODE. Both can
be configured, but MDCACHE will override CACHEINODE. This allows us to
deprecate CACHEINODE.
Change-Id: I49012723132ae6105b904a60d1a96bb2bf78d51b
Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c
M src/config_samples/ceph.conf
M src/config_samples/config.txt
M src/config_samples/ganesha.conf.example
M src/doc/man/ganesha-cache-config.rst
M src/doc/man/ganesha-config.rst
6 files changed, 31 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/454929/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929
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: I49012723132ae6105b904a60d1a96bb2bf78d51b
Gerrit-Change-Number: 454929
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
Gerrit-MessageType: newchange
11:25 a.m.
New subject: Change in ...nfs-ganesha[next]: MDCACHE - Add MDCACHE {} config block
Hi Daniel,
With the change below, I see parameters in CACHEINODE block are not really
assigned to mdcache_param global. Does that mean we should replace it with
MDCACHE in existing configs?
Even if I don't have MDCACHE block, the mdcache_param will get reset to
default values after the second load_config_from_parse() call.
Breakpoint 1, mdcache_set_param_from_conf (parse_tree=0x7fffffffe3c0,
err_type=0x7ffff7de0891 <config_error_is_harmless+24>)
at
/home/nutanix/nfs-ganesha/src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c:123
123 {
(gdb) print mdcache_param
$1 = {nparts = 0, cache_size = 0, getattr_dir_invalidation = false, dir =
{avl_chunk = 0, avl_chunk_split = 0, avl_detached_mult = 0,
avl_detached_max = 0}, entries_hwmark = 0, entries_release_size = 0,
chunks_hwmark = 0, lru_run_interval = 0, fd_limit_percent = 0,
fd_hwmark_percent = 0, fd_lwmark_percent = 0, reaper_work = 0,
reaper_work_per_lane = 0, biggest_window = 0, required_progress = 0,
futility_count = 0, dirmap_hwmark = 0}
(gdb) n
124 (void) load_config_from_parse(parse_tree,
(gdb)
129 if (!config_error_is_harmless(err_type)) {
(gdb) print mdcache_param
$2 = {nparts = 7, cache_size = 32633, getattr_dir_invalidation = false, dir
= {avl_chunk = 128, avl_chunk_split = 0,
avl_detached_mult = 1, avl_detached_max = 0}, entries_hwmark = 100,
entries_release_size = 100, chunks_hwmark = 100000,
lru_run_interval = 90, fd_limit_percent = 99, fd_hwmark_percent = 90,
fd_lwmark_percent = 50, reaper_work = 0,
reaper_work_per_lane = 50, biggest_window = 40, required_progress = 5,
futility_count = 8, dirmap_hwmark = 10000}
(gdb) n
135 (void) load_config_from_parse(parse_tree,
(gdb)
140 if (!config_error_is_harmless(err_type)) {
(gdb) print mdcache_param
$3 = {nparts = 7, cache_size = 32633, getattr_dir_invalidation = false, dir
= {avl_chunk = 128, avl_chunk_split = 0,
avl_detached_mult = 1, avl_detached_max = 0}, entries_hwmark = 100000,
entries_release_size = 100, chunks_hwmark = 100000,
lru_run_interval = 90, fd_limit_percent = 99, fd_hwmark_percent = 90,
fd_lwmark_percent = 50, reaper_work = 0,
reaper_work_per_lane = 50, biggest_window = 40, required_progress = 5,
futility_count = 8, dirmap_hwmark = 10000}
On Fri, May 17, 2019 at 10:51 AM Daniel Gryniewicz (GerritHub) <
gerrit(a)gerrithub.io> wrote:
Daniel Gryniewicz has uploaded this change for *review*.
View Change <https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929>
MDCACHE - Add MDCACHE {} config block
Add a config block name MDCACHE that is a copy of CACHEINODE. Both can
be configured, but MDCACHE will override CACHEINODE. This allows us to
deprecate CACHEINODE.
Change-Id: I49012723132ae6105b904a60d1a96bb2bf78d51b
Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c
M src/config_samples/ceph.conf
M src/config_samples/config.txt
M src/config_samples/ganesha.conf.example
M src/doc/man/ganesha-cache-config.rst
M src/doc/man/ganesha-config.rst
6 files changed, 31 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/454929/1
To view, visit change 454929
<https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929>;. To
unsubscribe, or for help writing mail filters, visit settings
<https://review.gerrithub.io/settings>;.
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I49012723132ae6105b904a60d1a96bb2bf78d51b
Gerrit-Change-Number: 454929
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
Gerrit-MessageType: newchange
_______________________________________________
Devel mailing list -- devel(a)lists.nfs-ganesha.org
To unsubscribe send an email to devel-leave(a)lists.nfs-ganesha.org
11:51 a.m.
New subject: Change in ...nfs-ganesha[next]: MDCACHE - Add MDCACHE {} config block
Interesting. I tested it when it went in, so I'm pretty sure it worked
at one point.
I'd just say switch to MDCACHE at this point and be done.
Daniel
On 11/5/20 2:25 PM, Pradeep wrote:
Hi Daniel,
With the change below, I see parameters in CACHEINODE block are not
really assigned to mdcache_param global. Does that mean we should
replace it with MDCACHE in existing configs?
Even if I don't have MDCACHE block, the mdcache_param will get reset to
default values after the second load_config_from_parse() call.
Breakpoint 1, mdcache_set_param_from_conf (parse_tree=0x7fffffffe3c0,
err_type=0x7ffff7de0891 <config_error_is_harmless+24>)
at
/home/nutanix/nfs-ganesha/src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c:123
123 {
(gdb) print mdcache_param
$1 = {nparts = 0, cache_size = 0, getattr_dir_invalidation = false, dir
= {avl_chunk = 0, avl_chunk_split = 0, avl_detached_mult = 0,
avl_detached_max = 0}, entries_hwmark = 0, entries_release_size =
0, chunks_hwmark = 0, lru_run_interval = 0, fd_limit_percent = 0,
fd_hwmark_percent = 0, fd_lwmark_percent = 0, reaper_work = 0,
reaper_work_per_lane = 0, biggest_window = 0, required_progress = 0,
futility_count = 0, dirmap_hwmark = 0}
(gdb) n
124 (void) load_config_from_parse(parse_tree,
(gdb)
129 if (!config_error_is_harmless(err_type)) {
(gdb) print mdcache_param
$2 = {nparts = 7, cache_size = 32633, getattr_dir_invalidation = false,
dir = {avl_chunk = 128, avl_chunk_split = 0,
avl_detached_mult = 1, avl_detached_max = 0}, entries_hwmark = 100,
entries_release_size = 100, chunks_hwmark = 100000,
lru_run_interval = 90, fd_limit_percent = 99, fd_hwmark_percent = 90,
fd_lwmark_percent = 50, reaper_work = 0,
reaper_work_per_lane = 50, biggest_window = 40, required_progress =
5, futility_count = 8, dirmap_hwmark = 10000}
(gdb) n
135 (void) load_config_from_parse(parse_tree,
(gdb)
140 if (!config_error_is_harmless(err_type)) {
(gdb) print mdcache_param
$3 = {nparts = 7, cache_size = 32633, getattr_dir_invalidation = false,
dir = {avl_chunk = 128, avl_chunk_split = 0,
avl_detached_mult = 1, avl_detached_max = 0}, entries_hwmark =
100000, entries_release_size = 100, chunks_hwmark = 100000,
lru_run_interval = 90, fd_limit_percent = 99, fd_hwmark_percent = 90,
fd_lwmark_percent = 50, reaper_work = 0,
reaper_work_per_lane = 50, biggest_window = 40, required_progress =
5, futility_count = 8, dirmap_hwmark = 10000}
On Fri, May 17, 2019 at 10:51 AM Daniel Gryniewicz (GerritHub)
<gerrit(a)gerrithub.io <mailto:gerrit@gerrithub.io>> wrote:
Daniel Gryniewicz has uploaded this change for *review*.
View Change <https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929>
MDCACHE - Add MDCACHE {} config block
Add a config block name MDCACHE that is a copy of CACHEINODE. Both can
be configured, but MDCACHE will override CACHEINODE. This allows us to
deprecate CACHEINODE.
Change-Id: I49012723132ae6105b904a60d1a96bb2bf78d51b
Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net
<mailto:dang@fprintf.net>>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c
M src/config_samples/ceph.conf
M src/config_samples/config.txt
M src/config_samples/ganesha.conf.example
M src/doc/man/ganesha-cache-config.rst
M src/doc/man/ganesha-config.rst
6 files changed, 31 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
<http://review.gerrithub.io:29418/ffilz/nfs-ganesha> refs/changes/29/454929/1
To view, visit change 454929
<https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/454929>;. To
unsubscribe, or for help writing mail filters, visit settings
<https://review.gerrithub.io/settings>;.
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I49012723132ae6105b904a60d1a96bb2bf78d51b
Gerrit-Change-Number: 454929
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com
<mailto:dang@redhat.com>>
Gerrit-MessageType: newchange
_______________________________________________
Devel mailing list -- devel(a)lists.nfs-ganesha.org
<mailto:devel@lists.nfs-ganesha.org>
To unsubscribe send an email to devel-leave(a)lists.nfs-ganesha.org
<mailto:devel-leave@lists.nfs-ganesha.org>
1261
days inactive
1799
days old
2 comments
3 participants
participants (3)
-
Daniel Gryniewicz -
Daniel Gryniewicz (GerritHub) -
Pradeep