On Fri, 2021-02-12 at 16:01 -0600, Eric W. Biederman wrote: > Joe Perches <j...@perches.com> writes: > > > Convert S_<FOO> permissions to the more readable octal. > > > > Done using: > > $ ./scripts/checkpatch.pl -f --fix-inplace --types=SYMBOLIC_PERMS > > fs/proc/*.[ch] > > > > No difference in generated .o files allyesconfig x86-64 > > > > Link: > > https://lore.kernel.org/lkml/ca+55afw5v23t-zvdzp-mmd_eyxf8wbafwwb59934fv7g21u...@mail.gmail.com/ > > > I will be frank. I don't know what 0644 means. I can never remember > which bit is read, write or execute. So I like symbolic constants. > > I don't see a compelling reason to change the existing code.
Did you read Linus' message in the Link: above? It was a reply to what Ingo Molnar suggested here: https://lore.kernel.org/lkml/20160803081140.ga7...@gmail.com/