[PATCH 1/3] block: Fix sysfs queue freeze and limits lock order
Christoph Hellwig
hch at lst.de
Mon Jan 6 00:29:02 PST 2025
On Mon, Jan 06, 2025 at 11:31:43AM +0800, Ming Lei wrote:
> As I mentioned in another thread, freezing queue may not be needed in
> ->store(), so let's discuss and confirm if it is needed here first.
>
> https://lore.kernel.org/linux-block/Z3tHozKiUqWr7gjO@fedora/
We do need the freezing. What you're proposing is playing fast and loose
which is going to get us in trouble. While most (all?) limits are simple
scalars, you often can't update just one without the others without
having coherent state. Having coherent state was the entire point of
the atomic queue limit updates.
More information about the Linux-nvme
mailing list