Re: [PATCH v2] Documentation: checkpatch: Add SYMBOLIC_PERMS message
From: Dwaipayan Ray <dwaipayanray1@gmail.com>
Date: 2021-08-03 21:49:58
On Wed, Aug 4, 2021 at 3:07 AM Utkarsh Verma [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Add a new message type SYMBOLIC_PERMS under the 'Permissions' subsection. Checkpatch documentation that recommends the user to use the octal permission bits instead of their symbolic macro names. Suggested-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> Signed-off-by: Utkarsh Verma <redacted> --- Documentation/dev-tools/checkpatch.rst | 6 ++++++ 1 file changed, 6 insertions(+)diff --git a/Documentation/dev-tools/checkpatch.rst b/Documentation/dev-tools/checkpatch.rst index f0956e9ea..0d60f8e5e 100644 --- a/Documentation/dev-tools/checkpatch.rst +++ b/Documentation/dev-tools/checkpatch.rst@@ -957,6 +957,12 @@ Permissions Permission bits should use 4 digit octal permissions (like 0700 or 0444). Avoid using any other base like decimal. + **SYMBOLIC_PERMS** + Permission bits in the octal form are more readable and easier to + understand than their symbolic counterparts. + + See: https://lwn.net/Articles/696229/
Please use lore.kernel.org links wherever possible. That article should be also there in the list. Cheers, Dwaipayan. _______________________________________________ Linux-kernel-mentees mailing list Linux-kernel-mentees@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees