diff options
| author | Pranith Kumar K <pranithk@gluster.com> | 2011-04-07 06:40:46 +0000 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2011-04-07 11:40:09 -0700 |
| commit | db347c756609e5fb6792bfd69046c97fd6aacc0e (patch) | |
| tree | aa7a619e239f547a3c48fa23cd6a6d02514f5d9d | |
| parent | 476b672347641c3f0d6b346b13feaf530ecbbe93 (diff) | |
mgmt/glusterd: Fix gluster-profile performance issues
Add count-fop-hits option to io-stats
use count-fop-hits instead of dump-fd-stats for profile
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Signed-off-by: Vijay Bellur <vijay@gluster.com>
BUG: 2636 ("profile info" lists 'nan' in latency when heavy i/o is running)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2636
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-op-sm.c | 6 | ||||
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-volgen.c | 1 |
2 files changed, 4 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c index d5676d79a67..3253c4f82af 100644 --- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c +++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c @@ -2027,7 +2027,7 @@ glusterd_is_profile_on (glusterd_volinfo_t *volinfo) GF_ASSERT (volinfo); latency_key = "diagnostics.latency-measurement"; - fd_stats_key = "diagnostics.dump-fd-stats"; + fd_stats_key = "diagnostics.count-fop-hits"; ret = dict_get_str_boolean (volinfo->dict, fd_stats_key, _gf_false); @@ -5355,7 +5355,7 @@ glusterd_add_profile_volume_options (glusterd_volinfo_t *volinfo) GF_ASSERT (volinfo); latency_key = "diagnostics.latency-measurement"; - fd_stats_key = "diagnostics.dump-fd-stats"; + fd_stats_key = "diagnostics.count-fop-hits"; ret = dict_set_str (volinfo->dict, latency_key, "on"); if (ret) { @@ -5386,7 +5386,7 @@ glusterd_remove_profile_volume_options (glusterd_volinfo_t *volinfo) GF_ASSERT (volinfo); latency_key = "diagnostics.latency-measurement"; - fd_stats_key = "diagnostics.dump-fd-stats"; + fd_stats_key = "diagnostics.count-fop-hits"; dict_del (volinfo->dict, latency_key); dict_del (volinfo->dict, fd_stats_key); } diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index 88ab2a27b3c..5844c7c7e8c 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -107,6 +107,7 @@ static struct volopt_map_entry glusterd_volopt_map[] = { {"diagnostics.latency-measurement", "debug/io-stats", NULL, NULL, NO_DOC }, {"diagnostics.dump-fd-stats", "debug/io-stats", NULL, NULL, NO_DOC }, + {"diagnostics.count-fop-hits", "debug/io-stats", NULL, NULL, NO_DOC }, {"diagnostics.brick-log-level", "debug/io-stats", "!log-level", NULL, DOC}, {"diagnostics.client-log-level", "debug/io-stats", "!log-level", NULL, DOC}, |