diff options
| author | Xavier Hernandez <xhernandez@datalab.es> | 2015-08-05 23:42:41 +0200 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2015-08-06 10:12:22 -0700 |
| commit | 7298b622ab39c2e78d6d745ae8b6e8413e1d9f1a (patch) | |
| tree | 5b8e7a1688532f2a3d80733a16304e5b6306cea8 /xlators/cluster/ec/src/ec-data.h | |
| parent | a3faffb259d5288907fac33a2822a8f61c3e86fe (diff) | |
cluster/ec: Fix tracking of good bricks
The bitmask of good and bad bricks was kept in the context of the
corresponding inode or fd. This was problematic when an external
process (another client or the self-heal process) did heal the
bricks but no one changed the bitmaks of other clients.
This patch removes the bitmask stored in the context and calculates
which bricks are healthy after locking them and doing the initial
xattrop. After that, it's updated using the result of each fop.
Change-Id: I225e31cd219a12af4ca58871d8a4bb6f742b223c
BUG: 1236065
Signed-off-by: Xavier Hernandez <xhernandez@datalab.es>
Reviewed-on: http://review.gluster.org/11844
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Diffstat (limited to 'xlators/cluster/ec/src/ec-data.h')
| -rw-r--r-- | xlators/cluster/ec/src/ec-data.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/xlators/cluster/ec/src/ec-data.h b/xlators/cluster/ec/src/ec-data.h index 18da06cea04..285f71e702d 100644 --- a/xlators/cluster/ec/src/ec-data.h +++ b/xlators/cluster/ec/src/ec-data.h @@ -58,7 +58,6 @@ struct _ec_config struct _ec_fd { - uintptr_t bad; loc_t loc; uintptr_t open; int32_t flags; @@ -66,7 +65,6 @@ struct _ec_fd struct _ec_inode { - uintptr_t bad; ec_lock_t *inode_lock; gf_boolean_t have_info; gf_boolean_t have_config; @@ -206,7 +204,6 @@ struct _ec_fop_data uintptr_t remaining; uintptr_t received; /* Mask of responses */ uintptr_t good; - uintptr_t bad; ec_wind_f wind; ec_handler_f handler; |