* [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
@ 2026-04-24 9:06 Baolin Liu
2026-04-24 21:12 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Baolin Liu @ 2026-04-24 9:06 UTC (permalink / raw)
To: corbet, skhan, cyphar, vbabka, surenb, rdunlap
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
From: Baolin Liu <liubaolin@kylinos.cn>
Commit e24ccaaf7ec4 ("block: remove last remaining traces of IDE
documentation") removed the IDE section but left its table of
contents entry behind.
Fix the stale entry and renumber the following sections.
Fixes: e24ccaaf7ec4 ("block: remove last remaining traces of IDE documentation")
Signed-off-by: Baolin Liu <liubaolin@kylinos.cn>
---
Documentation/filesystems/proc.rst | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/Documentation/filesystems/proc.rst b/Documentation/filesystems/proc.rst
index 7ce02573a3d9..70db35987ee1 100644
--- a/Documentation/filesystems/proc.rst
+++ b/Documentation/filesystems/proc.rst
@@ -23,13 +23,13 @@ fixes/update part 1.1 Stefani Seibold <stefani@seibold.net> June 9 2009
1 Collecting System Information
1.1 Process-Specific Subdirectories
1.2 Kernel data
- 1.3 IDE devices in /proc/ide
- 1.4 Networking info in /proc/net
- 1.5 SCSI info
- 1.6 Parallel port info in /proc/parport
- 1.7 TTY info in /proc/tty
- 1.8 Miscellaneous kernel statistics in /proc/stat
- 1.9 Ext4 file system parameters
+ 1.3 Networking info in /proc/net
+ 1.4 SCSI info
+ 1.5 Parallel port info in /proc/parport
+ 1.6 TTY info in /proc/tty
+ 1.7 Miscellaneous kernel statistics in /proc/stat
+ 1.8 Ext4 file system parameters
+ 1.9 /proc/consoles - Shows registered system consoles
2 Modifying System Parameters
--
2.51.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
2026-04-24 9:06 [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents Baolin Liu
@ 2026-04-24 21:12 ` Randy Dunlap
2026-04-27 9:55 ` Jonathan Corbet
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2026-04-24 21:12 UTC (permalink / raw)
To: Baolin Liu, corbet, skhan, cyphar, vbabka, surenb
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
On 4/24/26 2:06 AM, Baolin Liu wrote:
> From: Baolin Liu <liubaolin@kylinos.cn>
>
> Commit e24ccaaf7ec4 ("block: remove last remaining traces of IDE
> documentation") removed the IDE section but left its table of
> contents entry behind.
> Fix the stale entry and renumber the following sections.
>
> Fixes: e24ccaaf7ec4 ("block: remove last remaining traces of IDE documentation")
> Signed-off-by: Baolin Liu <liubaolin@kylinos.cn>
LGTM.
Acked-by: Randy Dunlap <rdunlap@infradead.org>
I note that the html-generated TOC is already correct;
e.g., https://docs.kernel.org/filesystems/proc.html#networking-info-in-proc-net
is already section 1.3.
AFAICT, this TOC is for users/readers who use an editor or pager
to read this file, but the patch is useful in that setting.
> ---
> Documentation/filesystems/proc.rst | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/Documentation/filesystems/proc.rst b/Documentation/filesystems/proc.rst
> index 7ce02573a3d9..70db35987ee1 100644
> --- a/Documentation/filesystems/proc.rst
> +++ b/Documentation/filesystems/proc.rst
> @@ -23,13 +23,13 @@ fixes/update part 1.1 Stefani Seibold <stefani@seibold.net> June 9 2009
> 1 Collecting System Information
> 1.1 Process-Specific Subdirectories
> 1.2 Kernel data
> - 1.3 IDE devices in /proc/ide
> - 1.4 Networking info in /proc/net
> - 1.5 SCSI info
> - 1.6 Parallel port info in /proc/parport
> - 1.7 TTY info in /proc/tty
> - 1.8 Miscellaneous kernel statistics in /proc/stat
> - 1.9 Ext4 file system parameters
> + 1.3 Networking info in /proc/net
> + 1.4 SCSI info
> + 1.5 Parallel port info in /proc/parport
> + 1.6 TTY info in /proc/tty
> + 1.7 Miscellaneous kernel statistics in /proc/stat
> + 1.8 Ext4 file system parameters
> + 1.9 /proc/consoles - Shows registered system consoles
>
> 2 Modifying System Parameters
>
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
2026-04-24 21:12 ` Randy Dunlap
@ 2026-04-27 9:55 ` Jonathan Corbet
2026-04-27 17:00 ` Randy Dunlap
[not found] ` <9455d5d6-80b7-4c0a-a569-fef3bd9f1529@163.com>
0 siblings, 2 replies; 6+ messages in thread
From: Jonathan Corbet @ 2026-04-27 9:55 UTC (permalink / raw)
To: Randy Dunlap, Baolin Liu, skhan, cyphar, vbabka, surenb
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
Randy Dunlap <rdunlap@infradead.org> writes:
> On 4/24/26 2:06 AM, Baolin Liu wrote:
>> From: Baolin Liu <liubaolin@kylinos.cn>
>>
>> Commit e24ccaaf7ec4 ("block: remove last remaining traces of IDE
>> documentation") removed the IDE section but left its table of
>> contents entry behind.
>> Fix the stale entry and renumber the following sections.
>>
>> Fixes: e24ccaaf7ec4 ("block: remove last remaining traces of IDE documentation")
>> Signed-off-by: Baolin Liu <liubaolin@kylinos.cn>
>
> LGTM.
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>
> I note that the html-generated TOC is already correct;
> e.g., https://docs.kernel.org/filesystems/proc.html#networking-info-in-proc-net
> is already section 1.3.
There are advantages to having a computer do a job like that :)
I do really question the value of these manually maintained TOCs; they
are always going to be out of date. But ...
> AFAICT, this TOC is for users/readers who use an editor or pager
> to read this file, but the patch is useful in that setting.
Others seem to disagree, so we keep them.
I've applied the patch, thanks.
jon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
2026-04-27 9:55 ` Jonathan Corbet
@ 2026-04-27 17:00 ` Randy Dunlap
[not found] ` <9455d5d6-80b7-4c0a-a569-fef3bd9f1529@163.com>
1 sibling, 0 replies; 6+ messages in thread
From: Randy Dunlap @ 2026-04-27 17:00 UTC (permalink / raw)
To: Jonathan Corbet, Baolin Liu, skhan, cyphar, vbabka, surenb
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
On 4/27/26 2:55 AM, Jonathan Corbet wrote:
> Randy Dunlap <rdunlap@infradead.org> writes:
>
>> On 4/24/26 2:06 AM, Baolin Liu wrote:
>>> From: Baolin Liu <liubaolin@kylinos.cn>
>>>
>>> Commit e24ccaaf7ec4 ("block: remove last remaining traces of IDE
>>> documentation") removed the IDE section but left its table of
>>> contents entry behind.
>>> Fix the stale entry and renumber the following sections.
>>>
>>> Fixes: e24ccaaf7ec4 ("block: remove last remaining traces of IDE documentation")
>>> Signed-off-by: Baolin Liu <liubaolin@kylinos.cn>
>>
>> LGTM.
>> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>>
>> I note that the html-generated TOC is already correct;
>> e.g., https://docs.kernel.org/filesystems/proc.html#networking-info-in-proc-net
>> is already section 1.3.
>
> There are advantages to having a computer do a job like that :)
>
> I do really question the value of these manually maintained TOCs; they
> are always going to be out of date. But ...
>
>> AFAICT, this TOC is for users/readers who use an editor or pager
>> to read this file, but the patch is useful in that setting.
>
> Others seem to disagree, so we keep them.
>
> I've applied the patch, thanks.
Yes, others do disagree, but my comment was not meant to be an endorsement
for keeping it there. I'm fine without it.
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
[not found] ` <9455d5d6-80b7-4c0a-a569-fef3bd9f1529@163.com>
@ 2026-04-28 8:57 ` Jonathan Corbet
2026-04-28 10:36 ` liubaolin
0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Corbet @ 2026-04-28 8:57 UTC (permalink / raw)
To: liubaolin, Randy Dunlap, skhan, cyphar, vbabka, surenb
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
Your use of HTML email almost certainly prevents your messages from
making it onto the lists; you need to stop doing that.
liubaolin <liubaolin12138@163.com> writes:
> I mainly work on ext4 kernel development. Recently, I've been
> implementing a feature that changes the usage of the ext4 proc
> parameter mb_stats, which requires updating the ext4 proc parameter
> documentation. Currently, the ext4 proc parameter documentation is
> explained in two files: "Documentation/admin-guide/ext4.rst" and
> "Documentation/filesystems/proc.rst".
>
> However, I found that only the ext4 file system describes the
> proc parameters in proc.rst, while other file systems such as btrfs
> and xfs do not describe the proc parameters they use in the file.
> Personally, I also feel that it is unnecessary to explain the ext4
> proc parameters in proc.rst.
>
> Therefore, I want to make a change by deleting the "Ext4 file
> system parameters" section from the proc.rst file. After
> modification, the usage and explanation of ext4 proc parameters will
> only be provided in Documentation/admin-guide/ext4.rst.
Documenting things twice is obviously not ideal; readers will wonder
about the truth when they inevitably go out of sync.
> If you agree, I will submit a patch to remove the "Ext4 file
> system parameters" section. If you disagree, my subsequent
> modifications to the ext4 proc parameters will be made simultaneously
> to the two documents: Documentation/admin-guide/ext4.rst and
> Documentation/filesystems/proc.rst. I look forward to your comments
> and replies.
I think that having a single location for that documentation makes
sense. Whether it's in the individual filesystem docs or centrally
located is up to the filesystem maintainers in the end. If you move
that documentation out of Documentation/filesystems/proc.rst, it would
be good to add links to the filesystem-specific parameters in its place,
though.
Longer term, it seems to me that much of what's in that file really
belongs in the admin guide, though.
Thanks,
jon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents
2026-04-28 8:57 ` Jonathan Corbet
@ 2026-04-28 10:36 ` liubaolin
0 siblings, 0 replies; 6+ messages in thread
From: liubaolin @ 2026-04-28 10:36 UTC (permalink / raw)
To: Jonathan Corbet, Randy Dunlap, skhan, cyphar, vbabka, surenb
Cc: linux-kernel, linux-fsdevel, linux-doc, Baolin Liu
在 2026/4/28 16:57, Jonathan Corbet 写道:
> Your use of HTML email almost certainly prevents your messages from
> making it onto the lists; you need to stop doing that.
>
I apologize, there was a problem with my newly set up email environment
that caused the HTML email to be sent. I will pay attention to this in
the future.
> liubaolin <liubaolin12138@163.com> writes:
>
>> I mainly work on ext4 kernel development. Recently, I've been
>> implementing a feature that changes the usage of the ext4 proc
>> parameter mb_stats, which requires updating the ext4 proc parameter
>> documentation. Currently, the ext4 proc parameter documentation is
>> explained in two files: "Documentation/admin-guide/ext4.rst" and
>> "Documentation/filesystems/proc.rst".
>>
>> However, I found that only the ext4 file system describes the
>> proc parameters in proc.rst, while other file systems such as btrfs
>> and xfs do not describe the proc parameters they use in the file.
>> Personally, I also feel that it is unnecessary to explain the ext4
>> proc parameters in proc.rst.
>>
>> Therefore, I want to make a change by deleting the "Ext4 file
>> system parameters" section from the proc.rst file. After
>> modification, the usage and explanation of ext4 proc parameters will
>> only be provided in Documentation/admin-guide/ext4.rst.
>
> Documenting things twice is obviously not ideal; readers will wonder
> about the truth when they inevitably go out of sync.
>
>> If you agree, I will submit a patch to remove the "Ext4 file
>> system parameters" section. If you disagree, my subsequent
>> modifications to the ext4 proc parameters will be made simultaneously
>> to the two documents: Documentation/admin-guide/ext4.rst and
>> Documentation/filesystems/proc.rst. I look forward to your comments
>> and replies.
>
> I think that having a single location for that documentation makes
> sense. Whether it's in the individual filesystem docs or centrally
> located is up to the filesystem maintainers in the end. If you move
> that documentation out of Documentation/filesystems/proc.rst, it would
> be good to add links to the filesystem-specific parameters in its place,
> though.
>
> Longer term, it seems to me that much of what's in that file really
> belongs in the admin guide, though.
>
> Thanks,
>
> jon
Okay, thank you for your reply. I will ask the ext4 maintainers, and if
they agree, I will submit a patch as you suggested.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-04-28 10:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-24 9:06 [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents Baolin Liu
2026-04-24 21:12 ` Randy Dunlap
2026-04-27 9:55 ` Jonathan Corbet
2026-04-27 17:00 ` Randy Dunlap
[not found] ` <9455d5d6-80b7-4c0a-a569-fef3bd9f1529@163.com>
2026-04-28 8:57 ` Jonathan Corbet
2026-04-28 10:36 ` liubaolin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox