diff options
author | Kinglong Mee <kinglongmee@gmail.com> | 2019-07-08 21:13:28 +0800 |
---|---|---|
committer | Rinku Kothiya <rkothiya@redhat.com> | 2019-08-22 05:56:23 +0000 |
commit | 9a3a94cb689bf761a323a99ac7ff1930f0805f97 (patch) | |
tree | 7cd3516e26191b587c0bc0a5040318afa77f5576 | |
parent | 044c66460560ee06968a3ff5801191c70fd5e628 (diff) |
cluster/ec: inherit healing from lock when it has info
If lock has info, fop should inherit healing mask from it.
Otherwise, fop cannot inherit right healing when changed_flags is zero.
Change-Id: Ife80c9169d2c555024347a20300b0583f7e8a87f
updates: bz#1739424
Signed-off-by: Kinglong Mee <mijinlong@horiscale.com>
-rw-r--r-- | xlators/cluster/ec/src/ec-common.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/xlators/cluster/ec/src/ec-common.c b/xlators/cluster/ec/src/ec-common.c index 35c2256ff4c..1781776239e 100644 --- a/xlators/cluster/ec/src/ec-common.c +++ b/xlators/cluster/ec/src/ec-common.c @@ -1412,11 +1412,12 @@ ec_get_size_version(ec_lock_link_t *link) set_dirty = ec_set_dirty_flag(link, ctx, dirty); /* If ec metadata has already been retrieved, do not try again. */ - if (ctx->have_info && (!set_dirty)) { + if (ctx->have_info) { if (ec_is_data_fop(fop->id)) { fop->healing |= lock->healing; } - goto unlock; + if (!set_dirty) + goto unlock; } /* Determine if there's something we need to retrieve for the current |