From: Chao Yu <chao@kernel.org>
To: Christoph Hellwig <hch@infradead.org>
Cc: chao@kernel.org, jaegeuk@kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org, linux-block@vger.kernel.org
Subject: Re: [PATCH v3] f2fs: introduce flush_policy sysfs entry
Date: Tue, 12 Aug 2025 16:22:45 +0800 [thread overview]
Message-ID: <16753b65-326d-48ba-90d2-476380455199@kernel.org> (raw)
In-Reply-To: <aJr0Tk36ZNiCg8mQ@infradead.org>
On 8/12/25 15:59, Christoph Hellwig wrote:
> On Tue, Aug 12, 2025 at 03:53:54PM +0800, Chao Yu wrote:
>>> What did you use before? At least for older qemu the default was
>>> buffered I/O, which can lead to very expensive fua or flush calls.
>>
>> Previously, I didn't use any cache= option, as manual described, it
>> should equal to cache=wrteback.
>
> Modern qemu actually split the cache option. You absolute want
> cache.direct=on. If you don't do simulated power fail testing by
Yes,
> killing qemu (or run real workloads for the matter, but who does that
> :)) it might make sense to just ignore the flushes with cache.no-flush=on
Yes, I don't care whether data can be persisted to host devices or not,
nor killing qemu for test, so cache.no-flush=on looks good to me as well.
> as well, which is what I do for my test VMs on the laptop.
Thanks for sharing this, it helps. :)
Thanks,
WARNING: multiple messages have this Message-ID (diff)
From: Chao Yu via Linux-f2fs-devel <linux-f2fs-devel@lists.sourceforge.net>
To: Christoph Hellwig <hch@infradead.org>
Cc: linux-block@vger.kernel.org, jaegeuk@kernel.org,
linux-kernel@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH v3] f2fs: introduce flush_policy sysfs entry
Date: Tue, 12 Aug 2025 16:22:45 +0800 [thread overview]
Message-ID: <16753b65-326d-48ba-90d2-476380455199@kernel.org> (raw)
In-Reply-To: <aJr0Tk36ZNiCg8mQ@infradead.org>
On 8/12/25 15:59, Christoph Hellwig wrote:
> On Tue, Aug 12, 2025 at 03:53:54PM +0800, Chao Yu wrote:
>>> What did you use before? At least for older qemu the default was
>>> buffered I/O, which can lead to very expensive fua or flush calls.
>>
>> Previously, I didn't use any cache= option, as manual described, it
>> should equal to cache=wrteback.
>
> Modern qemu actually split the cache option. You absolute want
> cache.direct=on. If you don't do simulated power fail testing by
Yes,
> killing qemu (or run real workloads for the matter, but who does that
> :)) it might make sense to just ignore the flushes with cache.no-flush=on
Yes, I don't care whether data can be persisted to host devices or not,
nor killing qemu for test, so cache.no-flush=on looks good to me as well.
> as well, which is what I do for my test VMs on the laptop.
Thanks for sharing this, it helps. :)
Thanks,
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
next prev parent reply other threads:[~2025-08-12 8:22 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-07 3:48 [f2fs-dev] [PATCH v3] f2fs: introduce flush_policy sysfs entry Chao Yu via Linux-f2fs-devel
2025-08-07 3:48 ` Chao Yu
2025-08-07 5:31 ` [f2fs-dev] " Zhiguo Niu
2025-08-07 5:31 ` Zhiguo Niu
2025-08-11 10:52 ` Christoph Hellwig
2025-08-11 10:52 ` [f2fs-dev] " Christoph Hellwig
2025-08-11 13:44 ` Bart Van Assche
2025-08-11 13:44 ` [f2fs-dev] " Bart Van Assche via Linux-f2fs-devel
2025-08-12 6:39 ` Chao Yu
2025-08-12 6:39 ` [f2fs-dev] " Chao Yu via Linux-f2fs-devel
2025-08-12 6:28 ` Chao Yu
2025-08-12 6:28 ` [f2fs-dev] " Chao Yu via Linux-f2fs-devel
2025-08-12 7:32 ` Christoph Hellwig
2025-08-12 7:32 ` [f2fs-dev] " Christoph Hellwig
2025-08-12 7:53 ` Chao Yu
2025-08-12 7:53 ` [f2fs-dev] " Chao Yu via Linux-f2fs-devel
2025-08-12 7:59 ` Christoph Hellwig
2025-08-12 7:59 ` [f2fs-dev] " Christoph Hellwig
2025-08-12 8:22 ` Chao Yu [this message]
2025-08-12 8:22 ` Chao Yu via Linux-f2fs-devel
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=16753b65-326d-48ba-90d2-476380455199@kernel.org \
--to=chao@kernel.org \
--cc=hch@infradead.org \
--cc=jaegeuk@kernel.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-kernel@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.