Hi, > Date: 2026-10-16 19:31:19+0200 > From: Günther Noack > > In the kernel's , udiag_vfs_ino comes before > udiag_vfs_dev, and it has been that way since the structure was added > in linux.git 5f7b0569460b ("unix_diag: Unix inode info NLA"). Both Please include the commit date. $ git ref 5f7b0569460b 5f7b0569460b (2011-12-16; "unix_diag: Unix inode info NLA") > members are __u32, so code written after the manual page would silently > mix up the inode and the device number. > > Also reorder the member descriptions to match. > > Fixes: 4f6a0a4a (2016-12-07; "sock_diag.7: New page documenting NETLINK_SOCK_DIAG interface") Please include 12 characters of the hash. $ git ref 4f6a0a4a 4f6a0a4a909e (2016-12-07; "sock_diag.7: New page documenting NETLINK_SOCK_DIAG interface") Worth CCing the authors of the patch. Cc: Dmitry V. Levin Cc: Pavel Emelyanov Cc: Michael Kerrisk > Assisted-by: LLM > Signed-off-by: Günther Noack Reviewed-by: Alejandro Colomar Cheers, Alex > --- > > Notes: > Compared against unix_diag.h header, > where the struct field order is different. > > man/man7/sock_diag.7 | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/man/man7/sock_diag.7 b/man/man7/sock_diag.7 > index d4e78c46cc8d..35a6d5a80ffc 100644 > --- a/man/man7/sock_diag.7 > +++ b/man/man7/sock_diag.7 > @@ -180,8 +180,8 @@ structure: > .in +4n > .EX > struct unix_diag_vfs { > - __u32 udiag_vfs_dev; > __u32 udiag_vfs_ino; > + __u32 udiag_vfs_dev; > }; > .EE > .in > @@ -189,11 +189,11 @@ struct unix_diag_vfs { > The fields of this structure are as follows: > .RS > .TP > -.I udiag_vfs_dev > -The device number of the corresponding on-disk socket inode. > -.TP > .I udiag_vfs_ino > The inode number of the corresponding on-disk socket inode. > +.TP > +.I udiag_vfs_dev > +The device number of the corresponding on-disk socket inode. > .RE > .TP > .B UDIAG_SHOW_PEER > -- > 2.56.0 > --