|
 |
ca3909 |
From 147cff762b307bf60519bae4cdefc62f655119a7 Mon Sep 17 00:00:00 2001
|
|
|
ca3909 |
From: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
ca3909 |
Date: Wed, 30 Oct 2019 10:47:17 +0530
|
|
|
ca3909 |
Subject: [PATCH 331/335] features/locks: Do special handling for op-version <
|
|
|
ca3909 |
3.12.0
|
|
|
ca3909 |
|
|
|
ca3909 |
Problem:
|
|
|
ca3909 |
Patch https://code.engineering.redhat.com/gerrit/#/c/140080/ diverges from
|
|
|
ca3909 |
its upstream patch(https://review.gluster.org/c/glusterfs/+/20031) in op-version.
|
|
|
ca3909 |
On upstream special-handling happens for version < 3.10.0 whereas for downstream
|
|
|
ca3909 |
special-handling happens for version < 3.12.0.
|
|
|
ca3909 |
When rebase happened for 3.5.0 from upstream, this downstream specific change
|
|
|
ca3909 |
is missed as there was no special downstream-only patch tracking this difference.
|
|
|
ca3909 |
This leads to I/O errors on upgrade from 3.3.1->3.5.0
|
|
|
ca3909 |
|
|
|
ca3909 |
Fix:
|
|
|
ca3909 |
Do special handling for op-version < 3.12.0 as in 3.4.x
|
|
|
ca3909 |
|
|
|
ca3909 |
Change-Id: I72fec058bdfb3cd30d017d205c90aa61aec86c5d
|
|
|
ca3909 |
Label: DOWNSTREAM ONLY
|
|
|
ca3909 |
BUG: 1766640
|
|
|
ca3909 |
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
ca3909 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/185835
|
|
|
ca3909 |
Reviewed-by: Xavi Hernandez Juan <xhernandez@redhat.com>
|
|
|
ca3909 |
---
|
|
|
ca3909 |
xlators/features/locks/src/posix.c | 2 +-
|
|
|
ca3909 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
ca3909 |
|
|
|
ca3909 |
diff --git a/xlators/features/locks/src/posix.c b/xlators/features/locks/src/posix.c
|
|
|
ca3909 |
index 9db5ac6..4592240 100644
|
|
|
ca3909 |
--- a/xlators/features/locks/src/posix.c
|
|
|
ca3909 |
+++ b/xlators/features/locks/src/posix.c
|
|
|
ca3909 |
@@ -57,7 +57,7 @@ fetch_pathinfo(xlator_t *, inode_t *, int32_t *, char **);
|
|
|
ca3909 |
do { \
|
|
|
ca3909 |
pl_local_t *__local = NULL; \
|
|
|
ca3909 |
if (frame->root->client && \
|
|
|
ca3909 |
- (frame->root->client->opversion < GD_OP_VERSION_3_10_0)) { \
|
|
|
ca3909 |
+ (frame->root->client->opversion < GD_OP_VERSION_3_12_0)) { \
|
|
|
ca3909 |
__local = frame->local; \
|
|
|
ca3909 |
PL_STACK_UNWIND_AND_FREE(__local, fop, frame, op_ret, params); \
|
|
|
ca3909 |
} else { \
|
|
|
ca3909 |
--
|
|
|
ca3909 |
1.8.3.1
|
|
|
ca3909 |
|