From: Niklas Cassel <Niklas.Cassel@wdc.com>
To: "axboe@kernel.dk" <axboe@kernel.dk>
Cc: "fio@vger.kernel.org" <fio@vger.kernel.org>,
Damien Le Moal <Damien.LeMoal@wdc.com>,
Niklas Cassel <Niklas.Cassel@wdc.com>
Subject: [PATCH 0/4] Improve fio max open zones handling
Date: Wed, 12 May 2021 22:36:32 +0000 [thread overview]
Message-ID: <20210512223615.17239-1-Niklas.Cassel@wdc.com> (raw)
From: Niklas Cassel <niklas.cassel@wdc.com>
Improve max open zones handling by introducing a new IO engine operation,
which can be implemented by ioengines.
Also provide a default implementation for Linux, which parses sysfs for
the max open zones value.
Update libzbc ioengine to utilize the new callback.
Having this callback has the two following advantages:
1) If the user forgets to specify a --max_open_zones value, we will
automatically use the maximum the drive supports.
(Instead of getting errors because fio tries to open too many zones.)
2) If the user specifies a --max_open_zones value that is too big,
we will return an error and fio will exit with a proper error message.
(Instead of getting errors because fio tries to open too many zones.)
Kind regards,
Niklas
Niklas Cassel (4):
zbd: only put an upper limit on max open zones once
oslib/linux-blkzoned: move sysfs reading into its own function
ioengines: add get_max_open_zones zoned block device operation
engines/libzbc: add support for the get_max_open_zones io op
engines/libzbc.c | 21 +++++++++
engines/skeleton_external.c | 13 ++++++
ioengines.h | 4 +-
oslib/blkzoned.h | 7 +++
oslib/linux-blkzoned.c | 82 ++++++++++++++++++++++++----------
zbd.c | 88 ++++++++++++++++++++++++++++++++++---
6 files changed, 186 insertions(+), 29 deletions(-)
--
2.25.1
next reply other threads:[~2021-05-12 22:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-12 22:36 Niklas Cassel [this message]
2021-05-12 22:36 ` [PATCH 1/4] zbd: only put an upper limit on max open zones once Niklas Cassel
2021-05-13 0:08 ` Damien Le Moal
2021-05-12 22:36 ` [PATCH 2/4] oslib/linux-blkzoned: move sysfs reading into its own function Niklas Cassel
2021-05-13 0:14 ` Damien Le Moal
2021-05-12 22:37 ` [PATCH 3/4] ioengines: add get_max_open_zones zoned block device operation Niklas Cassel
2021-05-13 0:23 ` Damien Le Moal
2021-05-14 12:05 ` Niklas Cassel
2021-05-15 1:16 ` Damien Le Moal
2021-05-17 8:43 ` Niklas Cassel
2021-05-12 22:37 ` [PATCH 4/4] engines/libzbc: add support for the get_max_open_zones io op Niklas Cassel
2021-05-13 0:24 ` Damien Le Moal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210512223615.17239-1-Niklas.Cassel@wdc.com \
--to=niklas.cassel@wdc.com \
--cc=Damien.LeMoal@wdc.com \
--cc=axboe@kernel.dk \
--cc=fio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.