Adding psinfo #4

Merged
merged 1 commit into from Jul 9, 2012

Conversation

Projects
None yet
2 participants
Collaborator

bahamas10 commented Jul 7, 2012

  • Added /proc/pid/psinfo struct parser

  • Updated readme and CLI usage

  • Added illumos to keywords in packace.json

    dave @ [ dave-01 :: (SunOS) ] ~/dev/proc $ node bin/proc-cli.js psinfo | json

{
  "flag": 33554432,
  "nlwp": 3,
  "pid": 15171,
  "ppid": 19249,
  "pgid": 15171,
  "sid": 19249,
  "uid": 2011,
  "euid": 2011,
  "gid": 2300,
  "egid": 2300,
  "addr": 0,
  "size": 14536,
  "rssize": 9868,
  "ttydev": 1572868,
  "pctcpu": 9,
  "pctmem": 3,
  "start": 1341693552.6120477,
  "time": 0.152629749,
  "ctime": 0,
  "fname": "node",
  "psargs": "node bin/proc-cli.js psinfo",
  "wstat": 0,
  "argc": 3,
  "argv": 134511780,
  "envp": 134511796,
  "dmodel": "\u0001",
  "taskid": 96333,
  "projid": 3,
  "nzomb": 0,
  "poolid": 0,
  "zoneid": 29,
  "contract": 18586358
}
Adding psinfo
* Added /proc/x/psinfo struct parser
* Updated readme and CLI usage
* Added `illumos` to keywords in packace.json
Owner

dshaw commented Jul 7, 2012

Excellent.

Owner

dshaw commented Jul 7, 2012

psinfo was actually in the initial module, but we really needed prusage so we focused on that first.

Similar to prusage being exposed as "usage", I'd like to expose psinfo as "info". What do you think?

Collaborator

bahamas10 commented Jul 7, 2012

I was thinking about the naming, and I thought it would be best to just basename what is in /proc.

  • /proc/self/psinfo => psinfo
  • /proc/self/usage => usage
Owner

dshaw commented Jul 7, 2012

@bahamas10 +1 I'll buy that.

bahamas10 added a commit that referenced this pull request Jul 9, 2012

@bahamas10 bahamas10 merged commit 592b44e into master Jul 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment