59562 2011-04-26 16:59:59 -0700 new-run-webkit-tests should disable/enable spindump on mac 2012-07-13 17:10:35 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) PC OS X 10.6 RESOLVED FIXED NRWT P2 Normal --- 64491 1 slewis webkit-unassigned dpranke slewis oldest_to_newest 393106 0 slewis 2011-04-26 16:59:59 -0700 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 651942 1 dpranke 2012-06-18 18:08:37 -0700 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? 652880 2 slewis 2012-06-19 15:23:43 -0700 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. 652895 3 dpranke 2012-06-19 15:45:40 -0700 maybe we should add a separate script for enabling/disabling spindump so people can run it when they want? 669606 4 dpranke 2012-07-13 17:10:35 -0700 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.