Distinguish (program) and (idle) in the time profiles

[aalexand]

@ofrobots @nolanmar511

Currently the time profiles contain a pseudo-node named "(program)" for samples which do not have any active Node.js code running. I find this name misleading. It would be clearer if it would be named "(idle)".

[aalexand]

We should confirm that (program) is used when and only when the Node.js thread is idle. E.g. that there are no other usages of this term for some additional rare states of the thread.

[aalexand]

@nolanmar511 Did we say during the discussion last time that you are going to look at the profiler source code to confirm that (program) only has the (idle) meaning?

[nolanmar511]

Yes, we did say that -- I do need to look at the source code to confirm that (program) only has the (idle) meaning. Thank you for reminding me of this.

[nolanmar511]

It does look like the v8 differentiates between "(program)" and "(idle)", given that both strings are defined here and idle and program appear to be used differently here.

I've not yet found any precise documentation on what "(program)" includes, but this Stack Overflow question referenced this bug from 2012, which suggests "(program)" (at one point and when using --inspect) referenced a mix of things, including idle time and native code execution.

Although there seems to be some concept of "(idle)" in v8-profilers, I've not seen a profile with that string. So, I'm guessing that idle time is encompassed in "(program)", but I'm don't think everything in program is idle.

[aalexand]

Closing this for now. We'll just use the naming the runtime uses, without any modifications.

[aalexand]

Re-opening to take a look and at least document somewhere the exact meaning of (idle) and (program).

[aalexand]

/cc @ofrobots