Messages in this thread | | | Date | Fri, 6 Aug 2004 14:01:23 +0200 | From | Roger Luethi <> | Subject | Re: [proc.txt] Fix /proc/pid/statm documentation |
| |
[ fixed linux-mm address ]
On Fri, 06 Aug 2004 03:46:30 -0700, William Lee Irwin III wrote: > On Fri, Aug 06, 2004 at 11:40:37AM +0200, Roger Luethi wrote: > > I discussed this very issue with wli on linux-mm about a year ago. proc > > file and documentation are still broken. So what's wrong with doing > > something about it? > > So now what, you want me to do yet another forward port of > linux-2.4.9-statm-B1.diff?
Your call, obviously -- do you think it's worthwhile? I didn't CC you on my initial posting because I wanted to avoid the impression that I am trying to make this your problem somehow. Priorities as I see them are:
- Document statm content somewhere. I posted a patch to document the current state. It could be complemented with a description of what it is supposed to do.
- Come to some agreement on what the proper values should be and change kernels accordingly. I'm inclined to favor keeping the first two (albeit redundant) fields and setting the rest to 0, simply because for them too many different de-facto semantics live in exisiting kernels.
A year ago, the first field was broken in 2.4 as well (not sure if/when it got fixed), but I can see why it is useful to keep around until top has found a better source. Same for the second field, the only one that has always been correct AFAIK.
- Provide additional information in proc files other than statm.
The problems with undocumented records are evident, but /proc/pid/status may be getting too heavy for frequent parsing. It's not realistic to redesign proc at this point, but it would be nice to have some documented understanding about the direction of proc evolution.
Roger - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |