|
 |
d1681e |
From fd0a8f99f853e923d485acfcdda556156105c8d1 Mon Sep 17 00:00:00 2001
|
|
|
d1681e |
From: Poornima G <pgurusid@redhat.com>
|
|
|
d1681e |
Date: Mon, 13 Nov 2017 12:55:06 +0530
|
|
|
d1681e |
Subject: [PATCH 150/180] dht: Fill first_up_subvol before use in dht_opendir
|
|
|
d1681e |
|
|
|
d1681e |
Reported by: Sam McLeod
|
|
|
d1681e |
|
|
|
d1681e |
> Upstream patch: https://review.gluster.org/#/c/18723/
|
|
|
d1681e |
|
|
|
d1681e |
Change-Id: Ic8f9b46b173796afd70aff1042834b03ac3e80b2
|
|
|
d1681e |
BUG: 1529072
|
|
|
d1681e |
Signed-off-by: Poornima G <pgurusid@redhat.com>
|
|
|
d1681e |
Reviewed-on: https://code.engineering.redhat.com/gerrit/130059
|
|
|
d1681e |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
d1681e |
Tested-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
d1681e |
---
|
|
|
d1681e |
tests/bugs/readdir-ahead/bug-1512437.t | 23 +++++++++++++++++++++++
|
|
|
d1681e |
xlators/cluster/dht/src/dht-common.c | 5 +++++
|
|
|
d1681e |
2 files changed, 28 insertions(+)
|
|
|
d1681e |
create mode 100755 tests/bugs/readdir-ahead/bug-1512437.t
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/tests/bugs/readdir-ahead/bug-1512437.t b/tests/bugs/readdir-ahead/bug-1512437.t
|
|
|
d1681e |
new file mode 100755
|
|
|
d1681e |
index 0000000..50eaa7d
|
|
|
d1681e |
--- /dev/null
|
|
|
d1681e |
+++ b/tests/bugs/readdir-ahead/bug-1512437.t
|
|
|
d1681e |
@@ -0,0 +1,23 @@
|
|
|
d1681e |
+#!/bin/bash
|
|
|
d1681e |
+
|
|
|
d1681e |
+. $(dirname $0)/../../include.rc
|
|
|
d1681e |
+. $(dirname $0)/../../volume.rc
|
|
|
d1681e |
+
|
|
|
d1681e |
+cleanup;
|
|
|
d1681e |
+
|
|
|
d1681e |
+TEST glusterd
|
|
|
d1681e |
+
|
|
|
d1681e |
+TEST $CLI volume create $V0 $H0:$B0/${V0}1
|
|
|
d1681e |
+TEST $CLI volume start $V0
|
|
|
d1681e |
+
|
|
|
d1681e |
+TEST $CLI volume set $V0 parallel-readdir on
|
|
|
d1681e |
+TEST $CLI volume set $V0 readdir-optimize on
|
|
|
d1681e |
+
|
|
|
d1681e |
+TEST glusterfs --volfile-server=$H0 --volfile-id=$V0 $M0
|
|
|
d1681e |
+TEST mkdir -p $M0/subdir1/subdir2;
|
|
|
d1681e |
+umount $M0
|
|
|
d1681e |
+TEST glusterfs --volfile-server=$H0 --volfile-id=$V0 $M0
|
|
|
d1681e |
+count=`ls -1 $M0/subdir1 | wc -l`
|
|
|
d1681e |
+TEST [ $count -eq 1 ]
|
|
|
d1681e |
+
|
|
|
d1681e |
+cleanup;
|
|
|
d1681e |
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c
|
|
|
d1681e |
index 727a47b..1262732 100644
|
|
|
d1681e |
--- a/xlators/cluster/dht/src/dht-common.c
|
|
|
d1681e |
+++ b/xlators/cluster/dht/src/dht-common.c
|
|
|
d1681e |
@@ -5043,6 +5043,7 @@ dht_opendir (call_frame_t *frame, xlator_t *this, loc_t *loc, fd_t *fd,
|
|
|
d1681e |
op_errno = ENOMEM;
|
|
|
d1681e |
goto err;
|
|
|
d1681e |
}
|
|
|
d1681e |
+ local->first_up_subvol = dht_first_up_subvol (this);
|
|
|
d1681e |
|
|
|
d1681e |
if (!xdata) {
|
|
|
d1681e |
xdata = dict_new ();
|
|
|
d1681e |
@@ -5070,6 +5071,10 @@ dht_opendir (call_frame_t *frame, xlator_t *this, loc_t *loc, fd_t *fd,
|
|
|
d1681e |
subvolumes = conf->local_subvols;
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
+ /* In case of parallel-readdir, the readdir-ahead will be loaded
|
|
|
d1681e |
+ * below dht, in this case, if we want to enable or disable SKIP_DIRs
|
|
|
d1681e |
+ * it has to be done in opendir, so that prefetching logic in
|
|
|
d1681e |
+ * readdir-ahead, honors it */
|
|
|
d1681e |
for (i = 0; i < call_count; i++) {
|
|
|
d1681e |
if (conf->readdir_optimize == _gf_true) {
|
|
|
d1681e |
if (subvolumes[i] != local->first_up_subvol) {
|
|
|
d1681e |
--
|
|
|
d1681e |
1.8.3.1
|
|
|
d1681e |
|