From 40b4e44049e37d987cd5329cd6a854f894a8a1cf Mon Sep 17 00:00:00 2001 From: Raghavendra Bhat Date: Wed, 13 Apr 2011 00:18:17 +0000 Subject: cluster/afr: declare favorite_child as int instead of unsigned int In afr_private_t structure favorite child is declared as unsigned int. In init function of afr we set favorite child to -1, if that option is not found in volfile. But favorite child value will be set to a huge value instead of -1 since it is an unsigned int and in statedump file favorite child value is displayed as a huge value instead of -1. Signed-off-by: Raghavendra Bhat Signed-off-by: Anand Avati BUG: 2668 ([glusterfs-3.2.9qa7]: createbench error) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2668 --- xlators/cluster/afr/src/afr-common.c | 2 +- xlators/cluster/afr/src/afr.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/xlators/cluster/afr/src/afr-common.c b/xlators/cluster/afr/src/afr-common.c index ce1eb60cde5..3d0ffb1d89e 100644 --- a/xlators/cluster/afr/src/afr-common.c +++ b/xlators/cluster/afr/src/afr-common.c @@ -2466,7 +2466,7 @@ afr_priv_dump (xlator_t *this) gf_proc_dump_build_key(key, key_prefix, "read_child"); gf_proc_dump_write(key, "%d", priv->read_child); gf_proc_dump_build_key(key, key_prefix, "favorite_child"); - gf_proc_dump_write(key, "%u", priv->favorite_child); + gf_proc_dump_write(key, "%d", priv->favorite_child); gf_proc_dump_build_key(key, key_prefix, "data_lock_server_count"); gf_proc_dump_write(key, "%u", priv->data_lock_server_count); gf_proc_dump_build_key(key, key_prefix, "metadata_lock_server_count"); diff --git a/xlators/cluster/afr/src/afr.h b/xlators/cluster/afr/src/afr.h index 6af96f9db18..7bd24e0ac6e 100644 --- a/xlators/cluster/afr/src/afr.h +++ b/xlators/cluster/afr/src/afr.h @@ -68,7 +68,7 @@ typedef struct _afr_private { gf_boolean_t entry_change_log; /* on/off */ int read_child; /* read-subvolume */ - unsigned int favorite_child; /* subvolume to be preferred in resolving + int favorite_child; /* subvolume to be preferred in resolving split-brain cases */ unsigned int data_lock_server_count; -- cgit