Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

Side by Side Diff: scripts/slave/compile.py

Issue 2513063002: Adding Findit high-end trybots to machines that can use higher -j value for compile (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """A tool to build chrome, executed by buildbot. 6 """A tool to build chrome, executed by buildbot.
7 7
8 When this is run, the current directory (cwd) should be the outer build 8 When this is run, the current directory (cwd) should be the outer build
9 directory (e.g., chrome-release/build/). 9 directory (e.g., chrome-release/build/).
10 10
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
260 260
261 assert number_of_processors > 0 261 assert number_of_processors > 0
262 262
263 # When goma is used, 10 * number_of_processors is basically good in 263 # When goma is used, 10 * number_of_processors is basically good in
264 # various situations according to our measurement. Build speed won't 264 # various situations according to our measurement. Build speed won't
265 # be improved if -j is larger than that. 265 # be improved if -j is larger than that.
266 # 266 #
267 # Since Mac had process number limitation before, we had to set 267 # Since Mac had process number limitation before, we had to set
268 # the upper limit to 50. Now that the process number limitation is 2000, 268 # the upper limit to 50. Now that the process number limitation is 2000,
269 # so we would be able to use 10 * number_of_processors. 269 # so we would be able to use 10 * number_of_processors.
270 # For the safety, we'd like to set the upper limit to 200. 270 # For safety, we'd like to set the upper limit to 200.
271 # 271 #
272 # Note that currently most try-bot build slaves have 8 processors. 272 # Note that currently most try-bot build slaves have 8 processors.
273 if chromium_utils.IsMac() or chromium_utils.IsWindows(): 273 if chromium_utils.IsMac() or chromium_utils.IsWindows():
274 return min(10 * number_of_processors, 200) 274 return min(10 * number_of_processors, 200)
275 275
276 # For Linux, we also would like to use 10 * cpu. However, not sure 276 # For Linux, we also would like to use 10 * cpu. However, not sure
277 # backend resource is enough, so let me set Linux and Linux x64 builder 277 # backend resource is enough, so let me set Linux and Linux x64 builder
278 # only for now. 278 # only for now.
279 hostname = goma_utils.GetShortHostname() 279 hostname = goma_utils.GetShortHostname()
280 if hostname in ( 280 if hostname in (
281 ['build14-m1', 'build48-m1'] + 281 ['build14-m1', 'build48-m1'] +
282 # Also increasing cpus for v8/blink trybots. 282 # Also increasing cpus for v8/blink trybots.
283 ['build%d-m4' % x for x in xrange(45, 48)] + 283 ['build%d-m4' % x for x in xrange(45, 48)] +
284 # Also increasing cpus for LTO buildbots. 284 # Also increasing cpus for LTO buildbots.
285 ['slave%d-c1' % x for x in [20, 33] + range(78, 108)]): 285 ['slave%d-c1' % x for x in [20, 33] + range(78, 108)] +
286 # Also increasing cpus for Findit trybots.
287 ['slave%d-c4' % x for x in [799] + range(873, 878)]):
286 return min(10 * number_of_processors, 200) 288 return min(10 * number_of_processors, 200)
287 289
288 return 50 290 return 50
289 291
290 292
291 def main_ninja(options, args, env): 293 def main_ninja(options, args, env):
292 """This function calls ninja. 294 """This function calls ninja.
293 295
294 Args: 296 Args:
295 options (Option): options for ninja command. 297 options (Option): options for ninja command.
(...skipping 195 matching lines...) Expand 10 before | Expand all | Expand 10 after
491 exit_status = main_ninja(options, args, env) 493 exit_status = main_ninja(options, args, env)
492 494
493 # stop goma 495 # stop goma
494 goma_teardown(options, env, exit_status, goma_cloudtail) 496 goma_teardown(options, env, exit_status, goma_cloudtail)
495 497
496 return exit_status 498 return exit_status
497 499
498 500
499 if '__main__' == __name__: 501 if '__main__' == __name__:
500 sys.exit(real_main()) 502 sys.exit(real_main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698