install and reboot; test various parts of /proc
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 16244 Build 16194: arc lint + arc unit
Event Timeline
| sys/fs/procfs/procfs.c | ||
|---|---|---|
| 102 | '__*' namespace not be used for regular code. Look at the style(9) how correct function body looks like. | |
| 114 | same style isue. | |
| 115 | return (); | |
| 168 | Style, binary ops require space from both side of the sign. Since you changing the line, it must be fixed. | |
| sys/fs/procfs/procfs.c | ||
|---|---|---|
| 115 | 0555 instead of 0500 like the others. Any suggestions on a better name? Maybe everyone_readexec or ugo for user-group-other ? | |
| sys/fs/procfs/procfs.c | ||
|---|---|---|
| 123 | 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, ...? | |
Comment Actions
Also look at MAINTAINERS file.
| sys/fs/procfs/procfs.c | ||
|---|---|---|
| 115 | I.e. your 'global' is usually spelled as 'all'. I would also use 'rw', 're' etc instead of spelling out the whole words. | |
| sys/fs/procfs/procfs.c | ||
|---|---|---|
| 115 | Is re here supposed to mean read + execute? If so, it should be spelled rx, cf. chgrp(1), chgrp(2) etc. | |
| 123 | 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). | |