From 9af6fb54d81b992b9ebaf5dd56aed34fd7dba032 Mon Sep 17 00:00:00 2001 From: Pranith Kumar K Date: Mon, 27 Nov 2017 09:50:32 +0530 Subject: cluster-syncop: Address comments in 3ad68df725ac32f83b5ea7c0976e2327e7037c8c Change-Id: I325f718c6c440076c9d9dcd5ad1a0c6bde5393b1 Signed-off-by: Pranith Kumar K --- libglusterfs/src/cluster-syncop.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'libglusterfs') diff --git a/libglusterfs/src/cluster-syncop.c b/libglusterfs/src/cluster-syncop.c index 75ba6404d9..50542eb7c1 100644 --- a/libglusterfs/src/cluster-syncop.c +++ b/libglusterfs/src/cluster-syncop.c @@ -1220,6 +1220,10 @@ cluster_tiebreaker_inodelk (xlator_t **subvols, unsigned char *on, num_success++; continue; } + + /* TODO: If earlier subvols fail with an error other + * than EAGAIN, we could still have 2 clients competing + * for the lock*/ if (replies[i].op_ret == -1 && replies[i].op_errno == EAGAIN) { cluster_fop_success_fill (replies, numsubvols, locked_on); @@ -1231,8 +1235,6 @@ cluster_tiebreaker_inodelk (xlator_t **subvols, unsigned char *on, FOP_SEQ (subvols, on, numsubvols, replies, locked_on, frame, inodelk, dom, &loc, F_SETLKW, &flock, NULL); - } else { - memset (locked_on, 0, numsubvols); } break; } -- cgit