Split procfs_attr into multiple functions

[eadler]

Test Plan

install and reboot; test various parts of /proc

[kib]

sys/fs/procfs/procfs.c

105

'__*' namespace not be used for regular code.

Look at the style(9) how correct function body looks like.

115

same style isue.

116

return ();
blank line before first statement.

175

Style, binary ops require space from both side of the sign. Since you changing the line, it must be fixed.

[eadler]

Address @kib's comments

[kib]

sys/fs/procfs/procfs.c

102

'{' must be on the new line.

116

What 'global' means there ?

[eadler]

sys/fs/procfs/procfs.c

116

0555 instead of 0500 like the others. Any suggestions on a better name? Maybe everyone_readexec or ugo for user-group-other ?

[domagoj.stolfa_gmail.com]

sys/fs/procfs/procfs.c

124

Somewhat unrelated to this patch, but is there a good reason why this isn't something like:

return (procfs_attr(td, p, pn, vap, (RW << OWNER));

instead of magic numbers like 0600, 0555, ...?

[kib]

Also look at MAINTAINERS file.

sys/fs/procfs/procfs.c

116

I.e. your 'global' is usually spelled as 'all'. I would also use 'rw', 're' etc instead of spelling out the whole words.

[eadler]

@kib's comments

[des]

sys/fs/procfs/procfs.c

116

Is re here supposed to mean read + execute? If so, it should be spelled rx, cf. chgrp(1), chgrp(2) etc.

124

Anyone with enough Unix experience to be reading this code will immediately know what the numbers mean. The macros take longer to read and understand and become unwieldy in more complex cases (e.g. 0755).

[des]

Sorry, mislabeled. See comment about function name.

[eadler]

@des's comments