path: root/
diff options
authorXavier Hernandez <>2015-01-07 12:29:48 +0100
committerPranith Kumar Karampuri <>2015-05-09 17:29:46 -0700
commitb46e65db722c14985db62a1679e0388d217b713b (patch)
treefac847af8f529505bf364ed54df25658552fbae3 /
parent73b61d709712692b48a1c9b18b4547410fb9078c (diff)
ec: Fix failures with missing files
When a file does not exist on a brick but it does on others, there could be problems trying to access it because there was some loc_t structures with null 'pargfid' but 'name' was set. This forced inode resolution based on <pargfid>/name instead of <gfid> which would be the correct one. To solve this problem, 'name' is always set to NULL when 'pargfid' is not present. Another problem was caused by an incorrect management of errors while doing incremental locking. The only allowed error during an incremental locking was ENOTCONN, but missing files on a brick can be returned as ESTALE. This caused an EIO on the operation. This patch doesn't care of errors during an incremental locking. At the end of the operation it will check if there are enough successfully locked bricks to continue or not. Change-Id: I9360ebf8d819d219cea2d173c09bd37679a6f15a BUG: 1176062 Signed-off-by: Xavier Hernandez <> Reviewed-on: Tested-by: NetBSD Build System Tested-by: Gluster Build System <> Reviewed-by: Pranith Kumar Karampuri <>
Diffstat (limited to '')
1 files changed, 1 insertions, 0 deletions
diff --git a/ b/
index 40fcad8..0c9e61c 100755
--- a/
+++ b/
@@ -208,6 +208,7 @@ function is_bad_test ()
./tests/basic/tier/tier.t \
./tests/basic/ec/ec-4-1.t \
./tests/basic/ec/ec.t \
+ ./tests/basic/ec/self-heal.t \
./tests/basic/quota-nfs.t \
./tests/basic/ec/ec-6-2.t \
./tests/bugs/quota/bug-1035576.t \