From: "Tommaso Califano" <tcalifano@redhat.com>
To: "Klaus Jensen" <its@irrelevant.dk>, <qemu-devel@nongnu.org>,
"Keith Busch" <kbusch@kernel.org>,
"Jesper Devantier" <foss@defmacro.it>, <qemu-block@nongnu.org>
Cc: "Klaus Jensen" <k.jensen@samsung.com>,
<qemu-devel-bounces+tcalifano=redhat.com@nongnu.org>
Subject: Re: [PATCH 2/3] hw/nvme: fix fdp reclaim unit accounting
Date: Thu, 12 Nov 2026 14:15:03 +0100 [thread overview]
Message-ID: <DMT9D9ASJ4K4.33A9K51FM3PE2@redhat.com> (raw)
In-Reply-To: <20261112-nvme-for-rc-v1-2-69b3e5dcd5c8@samsung.com>
On Thu Nov 12, 2026 at 10:53 AM CET, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
>
> nvme_do_write_fdp falls back to placement handle zero for a write that
> has no Data Placement directive or where the placement identifier is
> invalid. However, it then passes the invalid placement identifier to
> nvme_update_ruh which results in no update to RUAMW for placement handle
> zero.
>
> Reset the placement identifier along with the handle in the fallback.
>
> Link: https://gitlab.com/qemu-project/qemu/-/work_items/4520
/s/Link/Resolves
> Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
Fixes tag and QEMU stable
> ---
> hw/nvme/ctrl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c
> index 685cbc2f340c..a0d22dacedfa 100644
> --- a/hw/nvme/ctrl.c
> +++ b/hw/nvme/ctrl.c
> @@ -3754,6 +3754,7 @@ static void nvme_do_write_fdp(NvmeCtrl *n, NvmeRequest *req, uint64_t slba,
> !nvme_parse_pid(ns, pid, &ph, &rg)) {
> ph = 0;
> rg = 0;
> + pid = 0;
> }
>
> ruhid = ns->fdp.phs[ph];
Thanks,
Tommaso
next prev parent reply other threads:[~2026-11-12 13:15 UTC|newest]
Thread overview: 11+ 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-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 [this message]
2026-11-12 16:08 ` Jesper Wendel Devantier
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=DMT9D9ASJ4K4.33A9K51FM3PE2@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.