From: Ryusuke Konishi <konishi.ryusuke@gmail.com>
To: Randy Dunlap <rdunlap@infradead.org>
Cc: linux-fsdevel@vger.kernel.org,
Viacheslav Dubeyko <slava@dubeyko.com>,
linux-nilfs@vger.kernel.org
Subject: Re: [PATCH] nilfs2: convert nilfs_super_block to kernel-doc
Date: Fri, 28 Nov 2025 14:33:53 +0900 [thread overview]
Message-ID: <CAKFNMokjF6RdQLn35fxzaVBkfv_dkwLfTEnHg9aBOG7RH-aA0A@mail.gmail.com> (raw)
In-Reply-To: <20251128045541.672456-1-rdunlap@infradead.org>
On Fri, Nov 28, 2025 at 1:55 PM Randy Dunlap wrote:
>
> Eliminate 40+ kernel-doc warnings in nilfs2_ondisk.h by converting
> all of the struct member comments to kernel-doc comments.
>
> Fix one misnamed struct member in nilfs_direct_node.
>
> Object files before and after are the same size and content.
>
> Examples of warnings:
> Warning: include/uapi/linux/nilfs2_ondisk.h:202 struct member 's_rev_level'
> not described in 'nilfs_super_block'
> Warning: include/uapi/linux/nilfs2_ondisk.h:202 struct member
> 's_minor_rev_level' not described in 'nilfs_super_block'
> Warning: include/uapi/linux/nilfs2_ondisk.h:202 struct member 's_magic'
> not described in 'nilfs_super_block'
> Warning: include/uapi/linux/nilfs2_ondisk.h:202 struct member 's_bytes'
> not described in 'nilfs_super_block'
> Warning: include/uapi/linux/nilfs2_ondisk.h:202 struct member 's_flags'
> not described in 'nilfs_super_block'
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Cc: Ryusuke Konishi <konishi.ryusuke@gmail.com>
> Cc: Viacheslav Dubeyko <slava@dubeyko.com>
> Cc: linux-nilfs@vger.kernel.org
I didn't notice that. I'll grab this. Thank you!
Ryusuke Konishi
> ---
> include/uapi/linux/nilfs2_ondisk.h | 163 ++++++++++++++++-----------
> 1 file changed, 97 insertions(+), 66 deletions(-)
>
> --- linux-next-20251127.orig/include/uapi/linux/nilfs2_ondisk.h
> +++ linux-next-20251127/include/uapi/linux/nilfs2_ondisk.h
> @@ -133,73 +133,104 @@ struct nilfs_super_root {
>
> /**
> * struct nilfs_super_block - structure of super block on disk
> + * @s_rev_level: Revision level
> + * @s_minor_rev_level: minor revision level
> + * @s_magic: Magic signature
> + * @s_bytes: Bytes count of CRC calculation for
> + * this structure. s_reserved is excluded.
> + * @s_flags: flags
> + * @s_crc_seed: Seed value of CRC calculation
> + * @s_sum: Check sum of super block
> + * @s_log_block_size: Block size represented as follows:
> + * blocksize = 1 << (s_log_block_size + 10)
> + * @s_nsegments: Number of segments in filesystem
> + * @s_dev_size: block device size in bytes
> + * @s_first_data_block: 1st seg disk block number
> + * @s_blocks_per_segment: number of blocks per full segment
> + * @s_r_segments_percentage: Reserved segments percentage
> + * @s_last_cno: Last checkpoint number
> + * @s_last_pseg: disk block addr pseg written last
> + * @s_last_seq: seq. number of seg written last
> + * @s_free_blocks_count: Free blocks count
> + * @s_ctime: Creation time (execution time of newfs)
> + * @s_mtime: Mount time
> + * @s_wtime: Write time
> + * @s_mnt_count: Mount count
> + * @s_max_mnt_count: Maximal mount count
> + * @s_state: File system state
> + * @s_errors: Behaviour when detecting errors
> + * @s_lastcheck: time of last check
> + * @s_checkinterval: max. time between checks
> + * @s_creator_os: OS
> + * @s_def_resuid: Default uid for reserved blocks
> + * @s_def_resgid: Default gid for reserved blocks
> + * @s_first_ino: First non-reserved inode
> + * @s_inode_size: Size of an inode
> + * @s_dat_entry_size: Size of a dat entry
> + * @s_checkpoint_size: Size of a checkpoint
> + * @s_segment_usage_size: Size of a segment usage
> + * @s_uuid: 128-bit uuid for volume
> + * @s_volume_name: volume name
> + * @s_c_interval: Commit interval of segment
> + * @s_c_block_max: Threshold of data amount for the
> + * segment construction
> + * @s_feature_compat: Compatible feature set
> + * @s_feature_compat_ro: Read-only compatible feature set
> + * @s_feature_incompat: Incompatible feature set
> + * @s_reserved: padding to the end of the block
> */
> struct nilfs_super_block {
> -/*00*/ __le32 s_rev_level; /* Revision level */
> - __le16 s_minor_rev_level; /* minor revision level */
> - __le16 s_magic; /* Magic signature */
> -
> - __le16 s_bytes; /*
> - * Bytes count of CRC calculation
> - * for this structure. s_reserved
> - * is excluded.
> - */
> - __le16 s_flags; /* flags */
> - __le32 s_crc_seed; /* Seed value of CRC calculation */
> -/*10*/ __le32 s_sum; /* Check sum of super block */
> -
> - __le32 s_log_block_size; /*
> - * Block size represented as follows
> - * blocksize =
> - * 1 << (s_log_block_size + 10)
> - */
> - __le64 s_nsegments; /* Number of segments in filesystem */
> -/*20*/ __le64 s_dev_size; /* block device size in bytes */
> - __le64 s_first_data_block; /* 1st seg disk block number */
> -/*30*/ __le32 s_blocks_per_segment; /* number of blocks per full segment */
> - __le32 s_r_segments_percentage; /* Reserved segments percentage */
> -
> - __le64 s_last_cno; /* Last checkpoint number */
> -/*40*/ __le64 s_last_pseg; /* disk block addr pseg written last */
> - __le64 s_last_seq; /* seq. number of seg written last */
> -/*50*/ __le64 s_free_blocks_count; /* Free blocks count */
> -
> - __le64 s_ctime; /*
> - * Creation time (execution time of
> - * newfs)
> - */
> -/*60*/ __le64 s_mtime; /* Mount time */
> - __le64 s_wtime; /* Write time */
> -/*70*/ __le16 s_mnt_count; /* Mount count */
> - __le16 s_max_mnt_count; /* Maximal mount count */
> - __le16 s_state; /* File system state */
> - __le16 s_errors; /* Behaviour when detecting errors */
> - __le64 s_lastcheck; /* time of last check */
> -
> -/*80*/ __le32 s_checkinterval; /* max. time between checks */
> - __le32 s_creator_os; /* OS */
> - __le16 s_def_resuid; /* Default uid for reserved blocks */
> - __le16 s_def_resgid; /* Default gid for reserved blocks */
> - __le32 s_first_ino; /* First non-reserved inode */
> -
> -/*90*/ __le16 s_inode_size; /* Size of an inode */
> - __le16 s_dat_entry_size; /* Size of a dat entry */
> - __le16 s_checkpoint_size; /* Size of a checkpoint */
> - __le16 s_segment_usage_size; /* Size of a segment usage */
> -
> -/*98*/ __u8 s_uuid[16]; /* 128-bit uuid for volume */
> -/*A8*/ char s_volume_name[80] /* volume name */
> - __kernel_nonstring;
> -
> -/*F8*/ __le32 s_c_interval; /* Commit interval of segment */
> - __le32 s_c_block_max; /*
> - * Threshold of data amount for
> - * the segment construction
> - */
> -/*100*/ __le64 s_feature_compat; /* Compatible feature set */
> - __le64 s_feature_compat_ro; /* Read-only compatible feature set */
> - __le64 s_feature_incompat; /* Incompatible feature set */
> - __u32 s_reserved[186]; /* padding to the end of the block */
> +/*00*/ __le32 s_rev_level;
> + __le16 s_minor_rev_level;
> + __le16 s_magic;
> +
> + __le16 s_bytes;
> + __le16 s_flags;
> + __le32 s_crc_seed;
> +/*10*/ __le32 s_sum;
> +
> + __le32 s_log_block_size;
> + __le64 s_nsegments;
> +/*20*/ __le64 s_dev_size;
> + __le64 s_first_data_block;
> +/*30*/ __le32 s_blocks_per_segment;
> + __le32 s_r_segments_percentage;
> +
> + __le64 s_last_cno;
> +/*40*/ __le64 s_last_pseg;
> + __le64 s_last_seq;
> +/*50*/ __le64 s_free_blocks_count;
> +
> + __le64 s_ctime;
> +/*60*/ __le64 s_mtime;
> + __le64 s_wtime;
> +/*70*/ __le16 s_mnt_count;
> + __le16 s_max_mnt_count;
> + __le16 s_state;
> + __le16 s_errors;
> + __le64 s_lastcheck;
> +
> +/*80*/ __le32 s_checkinterval;
> + __le32 s_creator_os;
> + __le16 s_def_resuid;
> + __le16 s_def_resgid;
> + __le32 s_first_ino;
> +
> +/*90*/ __le16 s_inode_size;
> + __le16 s_dat_entry_size;
> + __le16 s_checkpoint_size;
> + __le16 s_segment_usage_size;
> +
> +/*98*/ __u8 s_uuid[16];
> +/*A8*/ char s_volume_name[80] __kernel_nonstring;
> +
> +/*F8*/ __le32 s_c_interval;
> + __le32 s_c_block_max;
> +
> +/*100*/ __le64 s_feature_compat;
> + __le64 s_feature_compat_ro;
> + __le64 s_feature_incompat;
> + __u32 s_reserved[186];
> };
>
> /*
> @@ -449,7 +480,7 @@ struct nilfs_btree_node {
> /**
> * struct nilfs_direct_node - header of built-in bmap array
> * @dn_flags: flags
> - * @dn_pad: padding
> + * @pad: padding
> */
> struct nilfs_direct_node {
> __u8 dn_flags;
prev parent reply other threads:[~2025-11-28 5:34 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-28 4:55 [PATCH] nilfs2: convert nilfs_super_block to kernel-doc Randy Dunlap
2025-11-28 5:33 ` Ryusuke Konishi [this message]
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=CAKFNMokjF6RdQLn35fxzaVBkfv_dkwLfTEnHg9aBOG7RH-aA0A@mail.gmail.com \
--to=konishi.ryusuke@gmail.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-nilfs@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=slava@dubeyko.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).