Rik van Riel wrote: > Laurent Vivier wrote: >> [PATCH 1/4] as recent CPUs introduce a third running state, after >> "user" and >> "system", we need a new field, "guest", in cpustat to store the time >> used by >> the CPU to run virtual CPU. Modify /proc/stat to display this new field. > > I think it would be good to always print out that number, even if > it is zero without the CONFIG option set. > > Changing the format of /proc/stat depending on config options is > probably worse than changing it when moving kernel versions. > > I guess reusing the "steal" field is not a good idea, since your > new "guest time" has different semantics than the steal time field. > I agree with all. Laurent -- ------------- [email protected] -------------- "Software is hard" - Donald Knuth
Attachment:
signature.asc
Description: OpenPGP digital signature
- References:
- [RESEND][PATCH 1/4] Introduce a new field "guest" in cpustat
- From: Laurent Vivier <[email protected]>
- Re: [RESEND][PATCH 1/4] Introduce a new field "guest" in cpustat
- From: Rik van Riel <[email protected]>
- [RESEND][PATCH 1/4] Introduce a new field "guest" in cpustat
- Prev by Date: [PATCH 6/7] NFS: if ATTR_KILL_S*ID bits are set, then skip mode change
- Next by Date: Re: [RESEND][PATCH] dir_index: error out instead of BUG on corrupt hash dir limit
- Previous by thread: Re: [RESEND][PATCH 1/4] Introduce a new field "guest" in cpustat
- Next by thread: [RESEND]PATCH 2/4] Introduce a new fields "gtime" and "cgtime" in task_struct and signal_struct
- Index(es):
 |