Bug 59562 - new-run-webkit-tests should disable/enable spindump on mac
Summary: new-run-webkit-tests should disable/enable spindump on mac
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.6
: P2 Normal
Assignee: Nobody
URL:
Keywords: NRWT
Depends on:
Blocks: 64491
  Show dependency treegraph
 
Reported: 2011-04-26 16:59 PDT by Stephanie Lewis
Modified: 2012-07-13 17:10 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stephanie Lewis 2011-04-26 16:59:59 PDT
new-run-webkit-tests should disable/enable spindump on mac.  It can hog the available memory and cpu

to disable:
sudo launchctl unload -w /System/Library/LaunchDaemons/com.apple.spindump.plist
sudo launchctl unload -w /System/Library/LaunchDaemons/com.apple.spindump_symbolicator.plist 
sudo launchctl load -w /System/Library/LaunchDaemons/com.apple.spindump_starter.plist 

to enable:
sudo launchctl load -w /System/Library/LaunchDaemons/com.apple.spindump.plist
sudo launchctl load -w /System/Library/LaunchDaemons/com.apple.spindump_symbolicator.plist 
sudo launchctl load -w /System/Library/LaunchDaemons/com.apple.spindump_starter.plist
Comment 1 Dirk Pranke 2012-06-18 18:08:37 PDT
hm. Sorry for not looking at this before now ... 

This is a problematic idea for me, since sudo would prompt for passwords and hence presumably cause problems on the bots, unless the commands were specially treated in sudoers.

Unless there's another way to solve this, the best I can come up with is to add a --disable-spindump flag that would do this only on request, but I'm not sure if that's really better than just doing this yourself in a wrapper script around new-run-webkit-tests.

WDYT?
Comment 2 Stephanie Lewis 2012-06-19 15:23:43 PDT
For the bots it would probably just be easier for each bot to disable spindump permanently (there's a good chance we already do this on the mac bots).  

I was thinking more of people running the tests on their machines.  Spindump, when it gets going, can dramatically reduce performance particularly when symbolicating custom builds.  I haven' t tested it recently so I should get some numbers on cpu usage on a recent build.

I don't like the idea of a wrapper, it seems like it would add an extra layer of complexity for a small benefit.
Comment 3 Dirk Pranke 2012-06-19 15:45:40 PDT
maybe we should add a separate script for enabling/disabling spindump so people can run it when they want?
Comment 4 Dirk Pranke 2012-07-13 17:10:35 PDT
I'm going to close this as WONTFIX, since I don't see a good way of doing this without prompting for passwords, and I don't think that's a good idea. Let me know if you disagree.