New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 693208 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature



Sign in to add a comment

Log pidstat while running power_LoadTest to get more insight into C0 residency

Project Member Reported by ravisadineni@chromium.org, Feb 16 2017

Issue description

Currently if there is a increase in C0 residency while running PLT it is hard to find out what caused this increase. If we could log pidstat(debug log) every one sec, that would help us understand if there are unexpected processes that are running.

 

Comment 1 by derat@chromium.org, Feb 16 2017

Components: Tests
Summary: Log pidstat while running power_LoadTest to get more insight into C0 residency (was: Log pidstat while running PLT to get more insight into C0 residency)
Do humans look at the debug logs right now? I'm worried that this could make them so spammy as to be unreadable (if they aren't already). Is there some way to log this info to a new file that'll get bundled up into the test results?
Sure. I will log into a separate file into the results directory.

Comment 3 by tbroch@chromium.org, Feb 16 2017

May want to checkout psutil module in python if you haven't already instead of making calls to pidstat.

In particular psutil.process_iter to loop across the pids.
Here is the initial CL for the same.
https://chromium-review.googlesource.com/#/c/444965/

Please let me know incase you have any suggestion.

This seems to add around 0.7% of load on the cpu(watched manually). One suggestion that todd mentioned was to add the page loaded  at that time. 
 
 

Comment 5 by tbroch@chromium.org, Feb 22 2017

Discussed a bit w/ Ravi and was thinking initially we should put this feature behind an argument as to not impact existing control files and instead enable in a new control file.

Thoughts?

I'd think copying control.1hour -> control.1hour_pidstat and enabling there.

Would be interesting to validate tsproxy's impact ( crbug.com/653519 ) as one effort to weed out potential test related cpu load during PLT.

On that note, it would be useful to add a helper script or postprocessing method to aggregate the pidstat results into a hotlist ... especially for long running pids in a separate CL to that in #c4

Sign in to add a comment