diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2016-01-22 12:11:40 +0530 |
---|---|---|
committer | Xavier Hernandez <xhernandez@datalab.es> | 2016-02-14 23:16:38 -0800 |
commit | 87faa0d763e1c416f12becc84b4c62cf6d545d1d (patch) | |
tree | 09f4a03579280e2521d0e15d50c37640a20a8bd8 /xlators/cluster | |
parent | d09d3406ce3eea551bd196402fdd46e3b8734bf1 (diff) |
cluster/ec: Never return -ve state
Ec manager shouldn't return -ve states, but it is, fixed that.
>Change-Id: I3f97c6ba2dbf9da724e8e1ee9b2c9da73f40013d
>BUG: 1300929
>Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
>Reviewed-on: http://review.gluster.org/13278
>Tested-by: Xavier Hernandez <xhernandez@datalab.es>
>Smoke: Gluster Build System <jenkins@build.gluster.com>
>CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
>NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
BUG: 1305742
Change-Id: I3c0db88b286afca355a8a9ec73589eca57710da6
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/13400
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Xavier Hernandez <xhernandez@datalab.es>
Diffstat (limited to 'xlators/cluster')
-rw-r--r-- | xlators/cluster/ec/src/ec-inode-read.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/ec/src/ec-inode-read.c b/xlators/cluster/ec/src/ec-inode-read.c index cbaa9bd9d3a..28b6b629b7a 100644 --- a/xlators/cluster/ec/src/ec-inode-read.c +++ b/xlators/cluster/ec/src/ec-inode-read.c @@ -109,7 +109,7 @@ ec_manager_access(ec_fop_data_t *fop, int32_t state) fop->cbks.access(fop->req_frame, fop, fop->xl, -1, fop->error, NULL); } - return -EC_STATE_LOCK_REUSE; + return EC_STATE_LOCK_REUSE; case -EC_STATE_LOCK_REUSE: case EC_STATE_LOCK_REUSE: |