|
 |
cead9d |
From ac5b1b38e705bd0e4c00cc50580a71dfaa4d3b5f Mon Sep 17 00:00:00 2001
|
|
|
cead9d |
From: Krutika Dhananjay <kdhananj@redhat.com>
|
|
|
cead9d |
Date: Wed, 7 Aug 2019 12:12:43 +0530
|
|
|
cead9d |
Subject: [PATCH 375/375] features/shard: Send correct size when reads are sent
|
|
|
cead9d |
beyond file size
|
|
|
cead9d |
|
|
|
cead9d |
Backport of:
|
|
|
cead9d |
> https://review.gluster.org/c/glusterfs/+/23175
|
|
|
cead9d |
> Change-Id: I0cebaaf55c09eb1fb77a274268ff564e871b743b
|
|
|
cead9d |
> fixes bz#1738419
|
|
|
cead9d |
> Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
|
|
|
cead9d |
|
|
|
cead9d |
Change-Id: I0cebaaf55c09eb1fb77a274268ff564e871b743b
|
|
|
cead9d |
BUG: 1802013
|
|
|
cead9d |
Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
|
|
|
cead9d |
Reviewed-on: https://code.engineering.redhat.com/gerrit/199570
|
|
|
cead9d |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
cead9d |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
cead9d |
---
|
|
|
cead9d |
tests/bugs/shard/bug-1738419.t | 29 +++++++++++++++++++++++++++++
|
|
|
cead9d |
xlators/features/shard/src/shard.c | 2 ++
|
|
|
cead9d |
2 files changed, 31 insertions(+)
|
|
|
cead9d |
create mode 100644 tests/bugs/shard/bug-1738419.t
|
|
|
cead9d |
|
|
|
cead9d |
diff --git a/tests/bugs/shard/bug-1738419.t b/tests/bugs/shard/bug-1738419.t
|
|
|
cead9d |
new file mode 100644
|
|
|
cead9d |
index 0000000..8d0a31d
|
|
|
cead9d |
--- /dev/null
|
|
|
cead9d |
+++ b/tests/bugs/shard/bug-1738419.t
|
|
|
cead9d |
@@ -0,0 +1,29 @@
|
|
|
cead9d |
+#!/bin/bash
|
|
|
cead9d |
+
|
|
|
cead9d |
+. $(dirname $0)/../../include.rc
|
|
|
cead9d |
+. $(dirname $0)/../../volume.rc
|
|
|
cead9d |
+
|
|
|
cead9d |
+cleanup
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST glusterd
|
|
|
cead9d |
+TEST pidof glusterd
|
|
|
cead9d |
+TEST $CLI volume create $V0 replica 3 $H0:$B0/${V0}{0,1,2}
|
|
|
cead9d |
+TEST $CLI volume set $V0 features.shard on
|
|
|
cead9d |
+TEST $CLI volume set $V0 network.remote-dio off
|
|
|
cead9d |
+TEST $CLI volume set $V0 performance.io-cache off
|
|
|
cead9d |
+TEST $CLI volume set $V0 performance.quick-read off
|
|
|
cead9d |
+TEST $CLI volume set $V0 performance.read-ahead off
|
|
|
cead9d |
+TEST $CLI volume set $V0 performance.strict-o-direct on
|
|
|
cead9d |
+TEST $CLI volume start $V0
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST $GFS --volfile-id=$V0 --volfile-server=$H0 $M0
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST dd if=/dev/zero of=$M0/metadata bs=501 count=1
|
|
|
cead9d |
+
|
|
|
cead9d |
+EXPECT "501" echo $("dd" if=$M0/metadata bs=4096 count=1 of=/dev/null iflag=direct 2>&1 | awk '/bytes/ {print $1}')
|
|
|
cead9d |
+
|
|
|
cead9d |
+EXPECT_WITHIN $UMOUNT_TIMEOUT "Y" force_umount $M0
|
|
|
cead9d |
+TEST $CLI volume stop $V0
|
|
|
cead9d |
+TEST $CLI volume delete $V0
|
|
|
cead9d |
+
|
|
|
cead9d |
+cleanup
|
|
|
cead9d |
diff --git a/xlators/features/shard/src/shard.c b/xlators/features/shard/src/shard.c
|
|
|
cead9d |
index b224abd..9ed597b 100644
|
|
|
cead9d |
--- a/xlators/features/shard/src/shard.c
|
|
|
cead9d |
+++ b/xlators/features/shard/src/shard.c
|
|
|
cead9d |
@@ -4433,6 +4433,8 @@ out:
|
|
|
cead9d |
if (xdata)
|
|
|
cead9d |
local->xattr_rsp = dict_ref(xdata);
|
|
|
cead9d |
vec.iov_base = local->iobuf->ptr;
|
|
|
cead9d |
+ if (local->offset + local->req_size > local->prebuf.ia_size)
|
|
|
cead9d |
+ local->total_size = local->prebuf.ia_size - local->offset;
|
|
|
cead9d |
vec.iov_len = local->total_size;
|
|
|
cead9d |
local->op_ret = local->total_size;
|
|
|
cead9d |
SHARD_STACK_UNWIND(readv, frame, local->op_ret, local->op_errno, &vec, 1,
|
|
|
cead9d |
--
|
|
|
cead9d |
1.8.3.1
|
|
|
cead9d |
|