From: Niklas Cassel <cassel@kernel.org>
To: Sam Li <faithilikerun@gmail.com>
Cc: qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>,
Kevin Wolf <kwolf@redhat.com>,
dlemoal@kernel.org, qemu-block@nongnu.org,
"Michael S. Tsirkin" <mst@redhat.com>,
Stefan Hajnoczi <stefanha@redhat.com>,
Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>,
Eric Blake <eblake@redhat.com>, Hanna Reitz <hreitz@redhat.com>
Subject: Re: [PATCH v13 2/6] block: widen BlockLimits.zone_size to uint64_t
Date: Fri, 26 Jun 2026 15:39:14 +0200 [thread overview]
Message-ID: <aj6BAs_a7aMb1qUO@ryzen> (raw)
In-Reply-To: <20260626131555.459792-3-faithilikerun@gmail.com>
On Fri, Jun 26, 2026 at 03:15:51PM +0200, Sam Li wrote:
> The zone-size field in BlockLimits is currently uint32_t, capping
> expressible zone sizes at 4 GiB. Real zoned-device protocols
> like NVMe ZNS allow larger zones. Widen BlockLimits.zone_size to
> uint64_t to match.
>
> Signed-off-by: Sam Li <faithilikerun@gmail.com>
> ---
> block/file-posix.c | 2 +-
> include/block/block_int-common.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/file-posix.c b/block/file-posix.c
> index 3c985da94f..ddb159c58b 100644
> --- a/block/file-posix.c
> +++ b/block/file-posix.c
> @@ -3600,7 +3600,7 @@ raw_co_zone_append(BlockDriverState *bs,
>
> if (*offset & zone_size_mask) {
> error_report("sector offset %" PRId64 " is not aligned to zone size "
> - "%" PRId32 "", *offset / 512, bs->bl.zone_size / 512);
> + "%" PRId64 "", *offset / 512, bs->bl.zone_size / 512);
> return -EINVAL;
> }
>
> diff --git a/include/block/block_int-common.h b/include/block/block_int-common.h
> index 147c08155f..7571ed9968 100644
> --- a/include/block/block_int-common.h
> +++ b/include/block/block_int-common.h
> @@ -901,7 +901,7 @@ typedef struct BlockLimits {
> BlockZoneModel zoned;
>
> /* zone size expressed in bytes */
> - uint32_t zone_size;
> + uint64_t zone_size;
>
> /* total number of zones */
> uint32_t nr_zones;
> --
> 2.53.0
>
Reviewed-by: Niklas Cassel <cassel@kernel.org>
next prev parent reply other threads:[~2026-06-26 13:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-26 13:15 [PATCH v13 0/6] Add full zoned storage emulation to the qcow2 driver Sam Li
2026-06-26 13:15 ` [PATCH v13 1/6] docs/qcow2: add the zoned format feature Sam Li
2026-06-26 13:39 ` Niklas Cassel
2026-06-26 13:15 ` [PATCH v13 2/6] block: widen BlockLimits.zone_size to uint64_t Sam Li
2026-06-26 13:39 ` Niklas Cassel [this message]
2026-06-26 13:15 ` [PATCH v13 3/6] qcow2: add configurations for zoned format extension Sam Li
2026-06-26 13:15 ` [PATCH v13 4/6] virtio-blk: do not merge writes across a zone boundary Sam Li
2026-06-26 13:38 ` Niklas Cassel
2026-06-26 13:15 ` [PATCH v13 5/6] qcow2: add zoned emulation capability Sam Li
2026-06-26 13:15 ` [PATCH v13 6/6] iotests: test the zoned format feature for qcow2 file Sam Li
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=aj6BAs_a7aMb1qUO@ryzen \
--to=cassel@kernel.org \
--cc=armbru@redhat.com \
--cc=dlemoal@kernel.org \
--cc=eblake@redhat.com \
--cc=faithilikerun@gmail.com \
--cc=hreitz@redhat.com \
--cc=kwolf@redhat.com \
--cc=mst@redhat.com \
--cc=pierrick.bouvier@oss.qualcomm.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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.