* [PATCH] statx.2: document STATX_DIO_READ_ALIGN [not found] <20250106151607.954940-1-hch@lst.de> @ 2025-01-06 15:19 ` Christoph Hellwig 2025-01-06 17:40 ` Darrick J. Wong 2025-01-06 22:01 ` Alejandro Colomar 0 siblings, 2 replies; 9+ messages in thread From: Christoph Hellwig @ 2025-01-06 15:19 UTC (permalink / raw) To: Alexander Viro, Christian Brauner Cc: Jan Kara, Chandan Babu R, Darrick J. Wong, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man Document the new STATX_DIO_READ_ALIGN flag and the new stx_dio_read_offset_align field guarded by it. Signed-off-by: Christoph Hellwig <hch@lst.de> --- man/man2/statx.2 | 27 ++++++++++++++++++++++++++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/man/man2/statx.2 b/man/man2/statx.2 index c5b5a28ec2f1..378bf363d93f 100644 --- a/man/man2/statx.2 +++ b/man/man2/statx.2 @@ -76,6 +76,9 @@ struct statx { __u32 stx_atomic_write_unit_min; __u32 stx_atomic_write_unit_max; __u32 stx_atomic_write_segments_max; + + /* File offset alignment for direct I/O reads */ + __u32 stx_dio_read_offset_align; }; .EE .in @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. It is deprecated and should not be used. STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. (since Linux 6.1; support varies by filesystem) STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) STATX_SUBVOL Want stx_subvol @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, stx_atomic_write_unit_max, and stx_atomic_write_segments_max. (since Linux 6.11; support varies by filesystem) +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. + (since Linux 6.14; support varies by filesystem) .TE .in .P @@ -467,6 +472,26 @@ This will only be nonzero if .I stx_dio_mem_align is nonzero, and vice versa. .TP +.I stx_dio_read_offset_align +The alignment (in bytes) required for file offsets and I/O segment lengths for +direct I/O reads +.RB ( O_DIRECT ) +on this file. If zero the limit in +.I +stx_dio_offset_align +applies for reads as well. If non-zero this value must be +smaller than +.I +stx_dio_offset_align +which must be provided by the file system. +This value does not affect the memory alignent in +.I stx_dio_mem_align . +.IP +.B STATX_DIO_READ_ALIGN +.I ( stx_dio_offset_align ) +support by filesystem; +it is supported by xfs since Linux 6.14. +.TP .I stx_subvol Subvolume number of the current file. .IP -- 2.45.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-06 15:19 ` [PATCH] statx.2: document STATX_DIO_READ_ALIGN Christoph Hellwig @ 2025-01-06 17:40 ` Darrick J. Wong 2025-01-06 18:09 ` Christoph Hellwig 2025-01-06 22:01 ` Alejandro Colomar 1 sibling, 1 reply; 9+ messages in thread From: Darrick J. Wong @ 2025-01-06 17:40 UTC (permalink / raw) To: Christoph Hellwig Cc: Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man On Mon, Jan 06, 2025 at 04:19:38PM +0100, Christoph Hellwig wrote: > Document the new STATX_DIO_READ_ALIGN flag and the new > stx_dio_read_offset_align field guarded by it. > > Signed-off-by: Christoph Hellwig <hch@lst.de> > --- > man/man2/statx.2 | 27 ++++++++++++++++++++++++++- > 1 file changed, 26 insertions(+), 1 deletion(-) > > diff --git a/man/man2/statx.2 b/man/man2/statx.2 > index c5b5a28ec2f1..378bf363d93f 100644 > --- a/man/man2/statx.2 > +++ b/man/man2/statx.2 > @@ -76,6 +76,9 @@ struct statx { > __u32 stx_atomic_write_unit_min; > __u32 stx_atomic_write_unit_max; > __u32 stx_atomic_write_segments_max; > + > + /* File offset alignment for direct I/O reads */ > + __u32 stx_dio_read_offset_align; > }; > .EE > .in > @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime > STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. > It is deprecated and should not be used. > STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) > -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align > +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. > (since Linux 6.1; support varies by filesystem) > STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) > STATX_SUBVOL Want stx_subvol > @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, > stx_atomic_write_unit_max, > and stx_atomic_write_segments_max. > (since Linux 6.11; support varies by filesystem) > +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. > + (since Linux 6.14; support varies by filesystem) > .TE > .in > .P > @@ -467,6 +472,26 @@ This will only be nonzero if > .I stx_dio_mem_align > is nonzero, and vice versa. > .TP > +.I stx_dio_read_offset_align > +The alignment (in bytes) required for file offsets and I/O segment lengths for > +direct I/O reads > +.RB ( O_DIRECT ) > +on this file. If zero the limit in manpage nit: new sentences should start on a new line. > +.I > +stx_dio_offset_align > +applies for reads as well. If non-zero this value must be Here too. > +smaller than > +.I > +stx_dio_offset_align > +which must be provided by the file system. I can't imagine a filesystem where dio_read_offset > dio_offset makes sense, but why do we need to put that in the manpage? vs. "If non-zero, the filesystem must also provide stx_dio_offset_align." > +This value does not affect the memory alignent in alignment > +.I stx_dio_mem_align . > +.IP > +.B STATX_DIO_READ_ALIGN > +.I ( stx_dio_offset_align ) > +support by filesystem; > +it is supported by xfs since Linux 6.14. Aside from those bits, this looks good to me. --D > +.TP > .I stx_subvol > Subvolume number of the current file. > .IP > -- > 2.45.2 > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-06 17:40 ` Darrick J. Wong @ 2025-01-06 18:09 ` Christoph Hellwig 2025-01-06 19:09 ` Darrick J. Wong 0 siblings, 1 reply; 9+ messages in thread From: Christoph Hellwig @ 2025-01-06 18:09 UTC (permalink / raw) To: Darrick J. Wong Cc: Christoph Hellwig, Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man On Mon, Jan 06, 2025 at 09:40:07AM -0800, Darrick J. Wong wrote: > > +stx_dio_offset_align > > +which must be provided by the file system. > > I can't imagine a filesystem where dio_read_offset > dio_offset makes > sense, but why do we need to put that in the manpage? Well, to be backwards compatible to older userspace the value put into stx_dio_offset_align also needs to work for reads. Given that there were questions about this in the RFC round I thought I'd mention it. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-06 18:09 ` Christoph Hellwig @ 2025-01-06 19:09 ` Darrick J. Wong 0 siblings, 0 replies; 9+ messages in thread From: Darrick J. Wong @ 2025-01-06 19:09 UTC (permalink / raw) To: Christoph Hellwig Cc: Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man On Mon, Jan 06, 2025 at 07:09:58PM +0100, Christoph Hellwig wrote: > On Mon, Jan 06, 2025 at 09:40:07AM -0800, Darrick J. Wong wrote: > > > +stx_dio_offset_align > > > +which must be provided by the file system. > > > > I can't imagine a filesystem where dio_read_offset > dio_offset makes > > sense, but why do we need to put that in the manpage? > > Well, to be backwards compatible to older userspace the value put into > stx_dio_offset_align also needs to work for reads. Given that there > were questions about this in the RFC round I thought I'd mention it. Ah ok then. :) With the other formatting nits fixed, Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --D ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-06 15:19 ` [PATCH] statx.2: document STATX_DIO_READ_ALIGN Christoph Hellwig 2025-01-06 17:40 ` Darrick J. Wong @ 2025-01-06 22:01 ` Alejandro Colomar 1 sibling, 0 replies; 9+ messages in thread From: Alejandro Colomar @ 2025-01-06 22:01 UTC (permalink / raw) To: Christoph Hellwig Cc: Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Darrick J. Wong, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man [-- Attachment #1: Type: text/plain, Size: 3375 bytes --] Hi Christoph, On Mon, Jan 06, 2025 at 04:19:38PM +0100, Christoph Hellwig wrote: > Document the new STATX_DIO_READ_ALIGN flag and the new > stx_dio_read_offset_align field guarded by it. > > Signed-off-by: Christoph Hellwig <hch@lst.de> Thanks for the patch! Please see some minor comments below. Have a lovely night! Alex > --- > man/man2/statx.2 | 27 ++++++++++++++++++++++++++- > 1 file changed, 26 insertions(+), 1 deletion(-) > > diff --git a/man/man2/statx.2 b/man/man2/statx.2 > index c5b5a28ec2f1..378bf363d93f 100644 > --- a/man/man2/statx.2 > +++ b/man/man2/statx.2 > @@ -76,6 +76,9 @@ struct statx { > __u32 stx_atomic_write_unit_min; > __u32 stx_atomic_write_unit_max; > __u32 stx_atomic_write_segments_max; > + > + /* File offset alignment for direct I/O reads */ > + __u32 stx_dio_read_offset_align; > }; > .EE > .in > @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime > STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. > It is deprecated and should not be used. > STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) > -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align > +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. > (since Linux 6.1; support varies by filesystem) > STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) > STATX_SUBVOL Want stx_subvol > @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, > stx_atomic_write_unit_max, > and stx_atomic_write_segments_max. > (since Linux 6.11; support varies by filesystem) > +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. > + (since Linux 6.14; support varies by filesystem) > .TE > .in > .P > @@ -467,6 +472,26 @@ This will only be nonzero if > .I stx_dio_mem_align > is nonzero, and vice versa. > .TP > +.I stx_dio_read_offset_align > +The alignment (in bytes) required for file offsets and I/O segment lengths for > +direct I/O reads > +.RB ( O_DIRECT ) > +on this file. If zero the limit in Please write poems, not prose. :) In other words, new sentence, new line. See man-pages(7). $ MANWIDTH=72 man man-pages | sed -n '/Use semantic newlines/,/^$/p' Use semantic newlines In the source of a manual page, new sentences should be started on new lines, long sentences should be split into lines at clause breaks (commas, semicolons, colons, and so on), and long clauses should be split at phrase boundaries. This convention, sometimes known as "semantic newlines", makes it easier to see the effect of patches, which often operate at the level of individual sentences, clauses, or phrases. > +.I > +stx_dio_offset_align We put the italics word in the same line as the .I. > +applies for reads as well. If non-zero this value must be > +smaller than > +.I > +stx_dio_offset_align > +which must be provided by the file system. > +This value does not affect the memory alignent in > +.I stx_dio_mem_align . > +.IP > +.B STATX_DIO_READ_ALIGN > +.I ( stx_dio_offset_align ) You probably meant .RI (roman-italics alternating). > +support by filesystem; > +it is supported by xfs since Linux 6.14. > +.TP > .I stx_subvol > Subvolume number of the current file. > .IP > -- > 2.45.2 > > -- <https://www.alejandro-colomar.es/> [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <20250108085549.1296733-1-hch@lst.de>]
* [PATCH] statx.2: document STATX_DIO_READ_ALIGN [not found] <20250108085549.1296733-1-hch@lst.de> @ 2025-01-08 8:59 ` Christoph Hellwig 2025-01-08 17:27 ` Darrick J. Wong 0 siblings, 1 reply; 9+ messages in thread From: Christoph Hellwig @ 2025-01-08 8:59 UTC (permalink / raw) To: Alexander Viro, Christian Brauner Cc: Jan Kara, Chandan Babu R, Darrick J. Wong, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man Document the new STATX_DIO_READ_ALIGN flag and the new stx_dio_read_offset_align field guarded by it. Signed-off-by: Christoph Hellwig <hch@lst.de> --- man/man2/statx.2 | 26 +++++++++++++++++++++++++- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/man/man2/statx.2 b/man/man2/statx.2 index c5b5a28ec2f1..8ef6a1cfb1c0 100644 --- a/man/man2/statx.2 +++ b/man/man2/statx.2 @@ -76,6 +76,9 @@ struct statx { __u32 stx_atomic_write_unit_min; __u32 stx_atomic_write_unit_max; __u32 stx_atomic_write_segments_max; + + /* File offset alignment for direct I/O reads */ + __u32 stx_dio_read_offset_align; }; .EE .in @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. It is deprecated and should not be used. STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. (since Linux 6.1; support varies by filesystem) STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) STATX_SUBVOL Want stx_subvol @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, stx_atomic_write_unit_max, and stx_atomic_write_segments_max. (since Linux 6.11; support varies by filesystem) +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. + (since Linux 6.14; support varies by filesystem) .TE .in .P @@ -467,6 +472,25 @@ This will only be nonzero if .I stx_dio_mem_align is nonzero, and vice versa. .TP +.I stx_dio_read_offset_align +The alignment (in bytes) required for file offsets and I/O segment lengths for +direct I/O reads +.RB ( O_DIRECT ) +on this file. +If zero the limit in +.I stx_dio_offset_align +applies for reads as well. +If non-zero this value must be smaller than +.I stx_dio_offset_align +which must be provided by the file system. +The memory alignment in +.I stx_dio_mem_align +is not affected by this value. +.IP +.B STATX_DIO_READ_ALIGN +.RI ( stx_dio_offset_align ) +is supported by xfs on regular files since Linux 6.14. +.TP .I stx_subvol Subvolume number of the current file. .IP -- 2.45.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-08 8:59 ` Christoph Hellwig @ 2025-01-08 17:27 ` Darrick J. Wong 0 siblings, 0 replies; 9+ messages in thread From: Darrick J. Wong @ 2025-01-08 17:27 UTC (permalink / raw) To: Christoph Hellwig Cc: Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man On Wed, Jan 08, 2025 at 09:59:00AM +0100, Christoph Hellwig wrote: > Document the new STATX_DIO_READ_ALIGN flag and the new > stx_dio_read_offset_align field guarded by it. > > Signed-off-by: Christoph Hellwig <hch@lst.de> > --- > man/man2/statx.2 | 26 +++++++++++++++++++++++++- > 1 file changed, 25 insertions(+), 1 deletion(-) > > diff --git a/man/man2/statx.2 b/man/man2/statx.2 > index c5b5a28ec2f1..8ef6a1cfb1c0 100644 > --- a/man/man2/statx.2 > +++ b/man/man2/statx.2 > @@ -76,6 +76,9 @@ struct statx { > __u32 stx_atomic_write_unit_min; > __u32 stx_atomic_write_unit_max; > __u32 stx_atomic_write_segments_max; > + > + /* File offset alignment for direct I/O reads */ > + __u32 stx_dio_read_offset_align; > }; > .EE > .in > @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime > STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. > It is deprecated and should not be used. > STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) > -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align > +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. > (since Linux 6.1; support varies by filesystem) > STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) > STATX_SUBVOL Want stx_subvol > @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, > stx_atomic_write_unit_max, > and stx_atomic_write_segments_max. > (since Linux 6.11; support varies by filesystem) > +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. > + (since Linux 6.14; support varies by filesystem) > .TE > .in > .P > @@ -467,6 +472,25 @@ This will only be nonzero if > .I stx_dio_mem_align > is nonzero, and vice versa. > .TP > +.I stx_dio_read_offset_align > +The alignment (in bytes) required for file offsets and I/O segment lengths for > +direct I/O reads > +.RB ( O_DIRECT ) > +on this file. > +If zero the limit in nit: add a comma here (really a dependent clause) to make it clearer that 'zero' isn't being used as a verb here: "If zero, the limit in..." > +.I stx_dio_offset_align > +applies for reads as well. > +If non-zero this value must be smaller than Same here. "If non-zero, this value..." With that fixed, Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --D > +.I stx_dio_offset_align > +which must be provided by the file system. > +The memory alignment in > +.I stx_dio_mem_align > +is not affected by this value. > +.IP > +.B STATX_DIO_READ_ALIGN > +.RI ( stx_dio_offset_align ) > +is supported by xfs on regular files since Linux 6.14. > +.TP > .I stx_subvol > Subvolume number of the current file. > .IP > -- > 2.45.2 > > ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <20250109083109.1441561-1-hch@lst.de>]
* [PATCH] statx.2: document STATX_DIO_READ_ALIGN [not found] <20250109083109.1441561-1-hch@lst.de> @ 2025-01-09 8:32 ` Christoph Hellwig 2025-01-09 13:21 ` Alejandro Colomar 0 siblings, 1 reply; 9+ messages in thread From: Christoph Hellwig @ 2025-01-09 8:32 UTC (permalink / raw) To: Alexander Viro, Christian Brauner Cc: Jan Kara, Chandan Babu R, Darrick J. Wong, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man Document the new STATX_DIO_READ_ALIGN flag and the new stx_dio_read_offset_align field guarded by it. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --- man/man2/statx.2 | 26 +++++++++++++++++++++++++- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/man/man2/statx.2 b/man/man2/statx.2 index c5b5a28ec2f1..7ad9c219a51d 100644 --- a/man/man2/statx.2 +++ b/man/man2/statx.2 @@ -76,6 +76,9 @@ struct statx { __u32 stx_atomic_write_unit_min; __u32 stx_atomic_write_unit_max; __u32 stx_atomic_write_segments_max; + + /* File offset alignment for direct I/O reads */ + __u32 stx_dio_read_offset_align; }; .EE .in @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. It is deprecated and should not be used. STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. (since Linux 6.1; support varies by filesystem) STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) STATX_SUBVOL Want stx_subvol @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, stx_atomic_write_unit_max, and stx_atomic_write_segments_max. (since Linux 6.11; support varies by filesystem) +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. + (since Linux 6.14; support varies by filesystem) .TE .in .P @@ -467,6 +472,25 @@ This will only be nonzero if .I stx_dio_mem_align is nonzero, and vice versa. .TP +.I stx_dio_read_offset_align +The alignment (in bytes) required for file offsets and I/O segment lengths for +direct I/O reads +.RB ( O_DIRECT ) +on this file. +If zero, the limit in +.I stx_dio_offset_align +applies for reads as well. +If non-zero, this value must be smaller than or equal to +.I stx_dio_offset_align +which must be provided by the file system if requested by the application. +The memory alignment in +.I stx_dio_mem_align +is not affected by this value. +.IP +.B STATX_DIO_READ_ALIGN +.RI ( stx_dio_offset_align ) +is supported by xfs on regular files since Linux 6.14. +.TP .I stx_subvol Subvolume number of the current file. .IP -- 2.45.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN 2025-01-09 8:32 ` Christoph Hellwig @ 2025-01-09 13:21 ` Alejandro Colomar 0 siblings, 0 replies; 9+ messages in thread From: Alejandro Colomar @ 2025-01-09 13:21 UTC (permalink / raw) To: Christoph Hellwig Cc: Alexander Viro, Christian Brauner, Jan Kara, Chandan Babu R, Darrick J. Wong, Hongbo Li, Ryusuke Konishi, linux-nilfs, linux-fsdevel, linux-xfs, linux-man [-- Attachment #1: Type: text/plain, Size: 3361 bytes --] Hi Christoph, On Thu, Jan 09, 2025 at 09:32:26AM +0100, Christoph Hellwig wrote: > Document the new STATX_DIO_READ_ALIGN flag and the new > stx_dio_read_offset_align field guarded by it. > > Signed-off-by: Christoph Hellwig <hch@lst.de> > Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> Thanks for the patch! I've applied it. <https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=3eb8ef31cb5295b5eaaaf319796ea6279b7f7002> A few minor comments: Please add the CCd people as Cc: in the commit message. I'll do that anyway, so it avoids me pasting them, and will probably make it easier for you to send with git-send-email(1) (or whatever you use). > Subject: Re: [PATCH] statx.2: document STATX_DIO_READ_ALIGN I changed commit subjects to use the full path to the manual page. Also please start with uppercase after the ':'. Also, please use version numbers for patches (v2, v3, ...). > --- > man/man2/statx.2 | 26 +++++++++++++++++++++++++- > 1 file changed, 25 insertions(+), 1 deletion(-) > > diff --git a/man/man2/statx.2 b/man/man2/statx.2 > index c5b5a28ec2f1..7ad9c219a51d 100644 > --- a/man/man2/statx.2 > +++ b/man/man2/statx.2 > @@ -76,6 +76,9 @@ struct statx { > __u32 stx_atomic_write_unit_min; > __u32 stx_atomic_write_unit_max; > __u32 stx_atomic_write_segments_max; > + I didn't realize this needed a \&. The CI reminded me. I've amended that. Have a lovely day! Alex > + /* File offset alignment for direct I/O reads */ > + __u32 stx_dio_read_offset_align; > }; > .EE > .in > @@ -261,7 +264,7 @@ STATX_BTIME Want stx_btime > STATX_ALL The same as STATX_BASIC_STATS | STATX_BTIME. > It is deprecated and should not be used. > STATX_MNT_ID Want stx_mnt_id (since Linux 5.8) > -STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align > +STATX_DIOALIGN Want stx_dio_mem_align and stx_dio_offset_align. > (since Linux 6.1; support varies by filesystem) > STATX_MNT_ID_UNIQUE Want unique stx_mnt_id (since Linux 6.8) > STATX_SUBVOL Want stx_subvol > @@ -270,6 +273,8 @@ STATX_WRITE_ATOMIC Want stx_atomic_write_unit_min, > stx_atomic_write_unit_max, > and stx_atomic_write_segments_max. > (since Linux 6.11; support varies by filesystem) > +STATX_DIO_READ_ALIGN Want stx_dio_read_offset_align. > + (since Linux 6.14; support varies by filesystem) > .TE > .in > .P > @@ -467,6 +472,25 @@ This will only be nonzero if > .I stx_dio_mem_align > is nonzero, and vice versa. > .TP > +.I stx_dio_read_offset_align > +The alignment (in bytes) required for file offsets and I/O segment lengths for > +direct I/O reads > +.RB ( O_DIRECT ) > +on this file. > +If zero, the limit in > +.I stx_dio_offset_align > +applies for reads as well. > +If non-zero, this value must be smaller than or equal to > +.I stx_dio_offset_align > +which must be provided by the file system if requested by the application. > +The memory alignment in > +.I stx_dio_mem_align > +is not affected by this value. > +.IP > +.B STATX_DIO_READ_ALIGN > +.RI ( stx_dio_offset_align ) > +is supported by xfs on regular files since Linux 6.14. > +.TP > .I stx_subvol > Subvolume number of the current file. > .IP > -- > 2.45.2 > > -- <https://www.alejandro-colomar.es/> [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2025-01-09 13:21 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20250106151607.954940-1-hch@lst.de>
2025-01-06 15:19 ` [PATCH] statx.2: document STATX_DIO_READ_ALIGN Christoph Hellwig
2025-01-06 17:40 ` Darrick J. Wong
2025-01-06 18:09 ` Christoph Hellwig
2025-01-06 19:09 ` Darrick J. Wong
2025-01-06 22:01 ` Alejandro Colomar
[not found] <20250108085549.1296733-1-hch@lst.de>
2025-01-08 8:59 ` Christoph Hellwig
2025-01-08 17:27 ` Darrick J. Wong
[not found] <20250109083109.1441561-1-hch@lst.de>
2025-01-09 8:32 ` Christoph Hellwig
2025-01-09 13:21 ` Alejandro Colomar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox