From: "Tommaso Califano" <tcalifano@redhat.com>
To: "Klaus Jensen" <its@irrelevant.dk>,
"Tommaso Califano" <tcalifano@redhat.com>
Cc: <qemu-devel@nongnu.org>, "Keith Busch" <kbusch@kernel.org>,
"Jesper Devantier" <foss@defmacro.it>, <qemu-block@nongnu.org>,
"Klaus Jensen" <k.jensen@samsung.com>,
<qemu-devel-bounces+tcalifano=redhat.com@nongnu.org>
Subject: Re: [PATCH 1/3] hw/nvme: reject fdp.runs smaller than the lba data size
Date: Fri, 13 Nov 2026 14:22:16 +0100 [thread overview]
Message-ID: <DMU45BQNS73Y.PS91851Q47GH@redhat.com> (raw)
In-Reply-To: <avX_vc432uGe1kmf@AALNPWKJENSEN.aal.scsc.local>
On Thu Nov 12, 2026 at 7:04 PM CET, Klaus Jensen wrote:
> On Nov 12 14:13, Tommaso Califano wrote:
>> On Thu Nov 12, 2026 at 10:53 AM CET, Klaus Jensen wrote:
>> > From: Klaus Jensen <k.jensen@samsung.com>
>> >
>> > The reclaim unit accounting in nvme_do_write_fdp spins forever if RUNS
>> > is configured to be smaller than LBADS.
>> >
>> > Reject such an invalid configuration upfront.
>> >
>> > Link: https://gitlab.com/qemu-project/qemu/-/work_items/4520
>>
>> /s/Link/Resolves
>>
>
> Issue #4520 is a bulk report that unfortunately contains more than just
> one issue. I didn't want automation to close the issue. That's why I
> used Link instead of Resolves.
>
Makes sense!
>> > Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
>>
>> Maybe Fixes tag and QEMU stable in cc?
>
> Doh. Yes.
>
> Thanks!
Regards,
Tommaso
next prev parent reply other threads:[~2026-11-13 13:22 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-12 9:53 [PATCH 0/3] hw/nvme: misc fixes Klaus Jensen
2026-11-12 9:53 ` [PATCH 1/3] hw/nvme: reject fdp.runs smaller than the lba data size Klaus Jensen
2026-11-12 13:13 ` Tommaso Califano
2026-11-12 18:04 ` Klaus Jensen
2026-11-13 13:22 ` Tommaso Califano [this message]
2026-11-12 21:38 ` Jesper Wendel Devantier
2026-11-12 9:53 ` [PATCH 2/3] hw/nvme: fix fdp reclaim unit accounting Klaus Jensen
2026-11-12 13:15 ` Tommaso Califano
2026-11-12 16:08 ` Jesper Wendel Devantier
2026-11-13 8:17 ` Klaus Jensen
2026-11-12 9:53 ` [PATCH 3/3] hw/nvme: fix broadcast fdp events handling Klaus Jensen
2026-11-12 13:16 ` Tommaso Califano
2026-11-12 16:39 ` Jesper Wendel Devantier
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=DMU45BQNS73Y.PS91851Q47GH@redhat.com \
--to=tcalifano@redhat.com \
--cc=foss@defmacro.it \
--cc=its@irrelevant.dk \
--cc=k.jensen@samsung.com \
--cc=kbusch@kernel.org \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel-bounces+tcalifano=redhat.com@nongnu.org \
--cc=qemu-devel@nongnu.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.