diff options
| author | Yaniv Kaul <ykaul@redhat.com> | 2019-10-04 11:17:44 +0300 |
|---|---|---|
| committer | Xavi Hernandez <xhernandez@redhat.com> | 2019-10-14 08:21:52 +0000 |
| commit | 704961a39ffd019487e36ac6dbd425399cb85cf2 (patch) | |
| tree | bbdff1293d5cd5602a04d42822f2024d481a3754 /xlators/cluster/dht/src/dht-common.c | |
| parent | 240eec91c6ab5818c3fa4a9087ba46bbed0e8d2b (diff) | |
Multiple files: make root gfid a static variable
In many places we use it, compare to it, etc. It could be a static variable,
as it really doesn't change. I think it's better than initializing to 0
and then doing gfid[15] = 1 or other tricks.
I think there are additional oppportunuties to make more variables static.
This is an attempt at an easy one.
Change-Id: I7f23a30a94056d8f043645371ab841cbd0f90d19
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-common.c')
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 232685bd56d..b295ce8bd03 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -6395,9 +6395,7 @@ dht_statfs(call_frame_t *frame, xlator_t *this, loc_t *loc, dict_t *xdata) int i = -1; inode_t *inode = NULL; inode_table_t *itable = NULL; - uuid_t root_gfid = { - 0, - }; + static uuid_t root_gfid = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1}; loc_t newloc = { 0, }; @@ -6423,7 +6421,6 @@ dht_statfs(call_frame_t *frame, xlator_t *this, loc_t *loc, dict_t *xdata) } loc = &local->loc2; - root_gfid[15] = 1; inode = inode_find(itable, root_gfid); if (!inode) { |