From: Kevin Wolf <kwolf@redhat.com>
To: Alberto Faria <afaria@redhat.com>
Cc: qemu-devel@nongnu.org, "Eduardo Habkost" <eduardo@habkost.net>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
qemu-block@nongnu.org, "Fam Zheng" <fam@euphon.net>,
"Yanan Wang" <wangyanan55@huawei.com>,
"Zhao Liu" <zhao1.liu@intel.com>
Subject: Re: [PATCH v2 2/2] scsi-disk: Advertise FUA support by default
Date: Fri, 25 Apr 2025 17:05:24 +0200 [thread overview]
Message-ID: <aAuktDpM_FIZhXOH@redhat.com> (raw)
In-Reply-To: <20250411113036.786841-3-afaria@redhat.com>
Am 11.04.2025 um 13:30 hat Alberto Faria geschrieben:
> Allow the guest to submit FUA requests directly, instead of forcing it
> to emulate them using a regular flush.
>
> Signed-off-by: Alberto Faria <afaria@redhat.com>
> ---
> hw/core/machine.c | 1 +
> hw/scsi/scsi-disk.c | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/core/machine.c b/hw/core/machine.c
> index 63c6ef93d2..e4e6474a4e 100644
> --- a/hw/core/machine.c
> +++ b/hw/core/machine.c
> @@ -46,6 +46,7 @@ GlobalProperty hw_compat_9_2[] = {
> { "migration", "multifd-clean-tls-termination", "false" },
> { "migration", "send-switchover-start", "off"},
> { "vfio-pci", "x-migration-multifd-transfer", "off" },
> + { "scsi-disk", "dpofua", "off" },
> };
> const size_t hw_compat_9_2_len = G_N_ELEMENTS(hw_compat_9_2);
This needs to go to hw_compat_10_0 now.
And shouldn't it be "scsi-hd" rather than "scsi-disk"? Did you test that
the property is disabled when you use an older machine type?
Kevin
next prev parent reply other threads:[~2025-04-25 15:06 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-11 11:30 [PATCH v2 0/2] scsi-disk: Add FUA write support Alberto Faria
2025-04-11 11:30 ` [PATCH v2 1/2] scsi-disk: Add native " Alberto Faria
2025-04-25 15:02 ` Kevin Wolf
2025-04-11 11:30 ` [PATCH v2 2/2] scsi-disk: Advertise FUA support by default Alberto Faria
2025-04-25 15:05 ` Kevin Wolf [this message]
2025-05-02 12:12 ` Alberto Faria
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=aAuktDpM_FIZhXOH@redhat.com \
--to=kwolf@redhat.com \
--cc=afaria@redhat.com \
--cc=eduardo@habkost.net \
--cc=fam@euphon.net \
--cc=marcel.apfelbaum@gmail.com \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=wangyanan55@huawei.com \
--cc=zhao1.liu@intel.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.