Re: [PATCH v6 05/10] block: Add core atomic write support

From: John Garry
Date: Thu Apr 11 2024 - 04:16:08 EST


On 11/04/2024 00:34, Luis Chamberlain wrote:
On 3/26/24 06:38, John Garry wrote:
diff --git a/Documentation/ABI/stable/sysfs-block b/Documentation/ABI/stable/sysfs-block
index 1fe9a553c37b..4c775f4bdefe 100644
--- a/Documentation/ABI/stable/sysfs-block
+++ b/Documentation/ABI/stable/sysfs-block
+What: /sys/block/<disk>/atomic_write_boundary_bytes
+Date: February 2024
+Contact: Himanshu Madhani<himanshu.madhani@xxxxxxxxxx>
+Description:
+ [RO] A device may need to internally split I/Os which
+ straddle a given logical block address boundary. In that
+ case a single atomic write operation will be processed as
+ one of more sub-operations which each complete atomically.
or
If*or* was meant, wouldn't it be better just to say one or more
operations may be processed as one atomically in this situation?

"Or" was meant (thanks Randy).

I think that we just need to say that the write operation will not complete atomically if it straddles the boundary. Whether the separate parts of the write operation which straddle the boundary complete atomically is undefined and irrelevant.

Thanks,
John