* [PATCH v3] zbd: Fix test scripts
@ 2018-09-05 8:02 Damien Le Moal
2018-09-05 14:21 ` Bart Van Assche
2018-09-05 14:36 ` Jens Axboe
0 siblings, 2 replies; 6+ messages in thread
From: Damien Le Moal @ 2018-09-05 8:02 UTC (permalink / raw)
To: fio, Jens Axboe; +Cc: Bart Van Assche
Fix handling of the path to the test-zbd-support script to avoid an
error when the tests are run directly when in fio/t/zbd using the
command "./run-against...".
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
---
Changes from v2:
* Avoid shellcheck warnings as suggested by Bart.
Changes from v1:
* Use a scriptdir variable as suggested by Bart.
t/zbd/run-tests-against-regular-nullb | 4 +++-
t/zbd/run-tests-against-zoned-nullb | 4 +++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/t/zbd/run-tests-against-regular-nullb b/t/zbd/run-tests-against-regular-nullb
index 133c7c41..0f6e4b66 100755
--- a/t/zbd/run-tests-against-regular-nullb
+++ b/t/zbd/run-tests-against-regular-nullb
@@ -4,6 +4,8 @@
#
# This file is released under the GPL.
+scriptdir="$(cd "$(dirname "$0")" && pwd)"
+
for d in /sys/kernel/config/nullb/*; do [ -d "$d" ] && rmdir "$d"; done
modprobe -r null_blk
modprobe null_blk nr_devices=0 || return $?
@@ -22,4 +24,4 @@ modprobe null_blk nr_devices=0 &&
echo 1 > memory_backed &&
echo 1 > power
-"$(dirname "$0")"/test-zbd-support "$@" /dev/nullb0
+"${scriptdir}"/test-zbd-support "$@" /dev/nullb0
diff --git a/t/zbd/run-tests-against-zoned-nullb b/t/zbd/run-tests-against-zoned-nullb
index 7d9eb439..9336716d 100755
--- a/t/zbd/run-tests-against-zoned-nullb
+++ b/t/zbd/run-tests-against-zoned-nullb
@@ -4,6 +4,8 @@
#
# This file is released under the GPL.
+scriptdir="$(cd "$(dirname "$0")" && pwd)"
+
for d in /sys/kernel/config/nullb/*; do [ -d "$d" ] && rmdir "$d"; done
modprobe -r null_blk
modprobe null_blk nr_devices=0 || return $?
@@ -24,4 +26,4 @@ modprobe null_blk nr_devices=0 &&
echo 1 > memory_backed &&
echo 1 > power
-"$(dirname "$0")"/test-zbd-support "$@" /dev/nullb0
+"${scriptdir}"/test-zbd-support "$@" /dev/nullb0
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] zbd: Fix test scripts
2018-09-05 8:02 [PATCH v3] zbd: Fix test scripts Damien Le Moal
@ 2018-09-05 14:21 ` Bart Van Assche
2018-09-05 14:36 ` Jens Axboe
1 sibling, 0 replies; 6+ messages in thread
From: Bart Van Assche @ 2018-09-05 14:21 UTC (permalink / raw)
To: Damien Le Moal, fio, Jens Axboe
On 09/05/18 01:02, Damien Le Moal wrote:
> Fix handling of the path to the test-zbd-support script to avoid an
> error when the tests are run directly when in fio/t/zbd using the
> command "./run-against...".
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] zbd: Fix test scripts
2018-09-05 8:02 [PATCH v3] zbd: Fix test scripts Damien Le Moal
2018-09-05 14:21 ` Bart Van Assche
@ 2018-09-05 14:36 ` Jens Axboe
2018-09-06 1:08 ` Damien Le Moal
1 sibling, 1 reply; 6+ messages in thread
From: Jens Axboe @ 2018-09-05 14:36 UTC (permalink / raw)
To: Damien Le Moal, fio; +Cc: Bart Van Assche
On 9/5/18 2:02 AM, Damien Le Moal wrote:
> Fix handling of the path to the test-zbd-support script to avoid an
> error when the tests are run directly when in fio/t/zbd using the
> command "./run-against...".
Thanks, applied.
Where are we on the other zbd fixes? I want to cut a new version
fairly soon, but I don't want to do that with fixes still pending.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] zbd: Fix test scripts
2018-09-05 14:36 ` Jens Axboe
@ 2018-09-06 1:08 ` Damien Le Moal
2018-09-06 1:12 ` Damien Le Moal
0 siblings, 1 reply; 6+ messages in thread
From: Damien Le Moal @ 2018-09-06 1:08 UTC (permalink / raw)
To: fio, axboe; +Cc: bvanassche
Jens,
On Wed, 2018-09-05 at 08:36 -0600, Jens Axboe wrote:
> On 9/5/18 2:02 AM, Damien Le Moal wrote:
> > Fix handling of the path to the test-zbd-support script to avoid an
> > error when the tests are run directly when in fio/t/zbd using the
> > command "./run-against...".
>
> Thanks, applied.
>
> Where are we on the other zbd fixes? I want to cut a new version
> fairly soon, but I don't want to do that with fixes still pending.
The v3 of the 3 patch series I sent was heavily tested and reviewed by Bart.
So I think the changes are OK to go.
I updated my pull request in github with the latest version of all patches:
* The 3 patches of the v3 "ZBD support improvements and cleanup":
[PATCH v3 1/3] zbd: Improve read randomness
[PATCH v3 2/3] zbd: Remove inexistent functions declaration
[PATCH v3 3/3] zbd: Fix zbd_zone_idx()
* This test script fix patch
* A patch that cleans up the code using the fio regular byte unit for zone
information:
[PATCH] zbd: Use bytes unit
Thanks !
--
Damien Le Moal
Western Digital
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] zbd: Fix test scripts
2018-09-06 1:08 ` Damien Le Moal
@ 2018-09-06 1:12 ` Damien Le Moal
2018-09-06 1:26 ` Jens Axboe
0 siblings, 1 reply; 6+ messages in thread
From: Damien Le Moal @ 2018-09-06 1:12 UTC (permalink / raw)
To: fio, axboe; +Cc: bvanassche
On Thu, 2018-09-06 at 01:08 +0000, Damien Le Moal wrote:
> Jens,
>
> On Wed, 2018-09-05 at 08:36 -0600, Jens Axboe wrote:
> > On 9/5/18 2:02 AM, Damien Le Moal wrote:
> > > Fix handling of the path to the test-zbd-support script to avoid an
> > > error when the tests are run directly when in fio/t/zbd using the
> > > command "./run-against...".
> >
> > Thanks, applied.
> >
> > Where are we on the other zbd fixes? I want to cut a new version
> > fairly soon, but I don't want to do that with fixes still pending.
>
> The v3 of the 3 patch series I sent was heavily tested and reviewed by Bart.
> So I think the changes are OK to go.
>
> I updated my pull request in github with the latest version of all patches:
> * The 3 patches of the v3 "ZBD support improvements and cleanup":
> [PATCH v3 1/3] zbd: Improve read randomness
> [PATCH v3 2/3] zbd: Remove inexistent functions declaration
> [PATCH v3 3/3] zbd: Fix zbd_zone_idx()
> * This test script fix patch
> * A patch that cleans up the code using the fio regular byte unit for zone
> information:
> [PATCH] zbd: Use bytes unit
Correction: I rebased the pull request. Since the test script patch was
applied, only 4 patches in the pull request now.
Thanks !
--
Damien Le Moal
Western Digital
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] zbd: Fix test scripts
2018-09-06 1:12 ` Damien Le Moal
@ 2018-09-06 1:26 ` Jens Axboe
0 siblings, 0 replies; 6+ messages in thread
From: Jens Axboe @ 2018-09-06 1:26 UTC (permalink / raw)
To: Damien Le Moal, fio; +Cc: bvanassche
On 9/5/18 7:12 PM, Damien Le Moal wrote:
> On Thu, 2018-09-06 at 01:08 +0000, Damien Le Moal wrote:
>> Jens,
>>
>> On Wed, 2018-09-05 at 08:36 -0600, Jens Axboe wrote:
>>> On 9/5/18 2:02 AM, Damien Le Moal wrote:
>>>> Fix handling of the path to the test-zbd-support script to avoid an
>>>> error when the tests are run directly when in fio/t/zbd using the
>>>> command "./run-against...".
>>>
>>> Thanks, applied.
>>>
>>> Where are we on the other zbd fixes? I want to cut a new version
>>> fairly soon, but I don't want to do that with fixes still pending.
>>
>> The v3 of the 3 patch series I sent was heavily tested and reviewed by Bart.
>> So I think the changes are OK to go.
>>
>> I updated my pull request in github with the latest version of all patches:
>> * The 3 patches of the v3 "ZBD support improvements and cleanup":
>> [PATCH v3 1/3] zbd: Improve read randomness
>> [PATCH v3 2/3] zbd: Remove inexistent functions declaration
>> [PATCH v3 3/3] zbd: Fix zbd_zone_idx()
>> * This test script fix patch
>> * A patch that cleans up the code using the fio regular byte unit for zone
>> information:
>> [PATCH] zbd: Use bytes unit
>
> Correction: I rebased the pull request. Since the test script patch was
> applied, only 4 patches in the pull request now.
Great thanks, I'll pull them in.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-09-06 1:26 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-05 8:02 [PATCH v3] zbd: Fix test scripts Damien Le Moal
2018-09-05 14:21 ` Bart Van Assche
2018-09-05 14:36 ` Jens Axboe
2018-09-06 1:08 ` Damien Le Moal
2018-09-06 1:12 ` Damien Le Moal
2018-09-06 1:26 ` Jens Axboe
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.