Closed Bug 1239292 Opened 7 years ago Closed 3 years ago

[meta] Multi-threaded job scheduler


(Core :: Graphics, defect, P3)




Tracking Status
firefox78 --- fixed


(Reporter: nical, Assigned: nical)


(Depends on 1 open bug)


(Keywords: meta)


(2 files)

Tracking bug for the job scheduler.
Assignee: nobody → nical.bugzilla
Depends on: 1239288, 1210669, 1239290
Blocks: 1083101
Depends on: 1209039
Depends on: 1263212
Depends on: 1264301
Depends on: 1264305
Depends on: 1266991
Keywords: meta
We dropped the idea of multi-threaded tile rendering so we don't have a use case for this right now, there are bitrotted patches pending on reviews for a year, and more importantly several seconds spent running the scheduler tests in each push, which is a waste of infra since we don't have plans to use this in the foreseeable future.

If we look into something like this again in the future, rust would be a much better place for building this type of system.
Attachment #8892910 - Flags: review?(jmuizelaar)
Attachment #8892910 - Flags: review?(jmuizelaar) → review+
Pushed by
Remove the gfx job scheduler. r=jrmuizel
Backed out for build bustage at StyleSheetInlines.h(77):

Push with failures:
Failure log:

09:10:04     INFO -  Unified_cpp_gfx_tests_gtest0.cpp
09:10:04     INFO -  z:\build\build\src\obj-firefox\dist\include\mozilla/StyleSheetInlines.h(77): error C2287: 'mozilla::StyleSheet': inheritance representation: 'single_inheritance' is less general than the required 'multiple_inheritance'
09:10:04     INFO -  z:\build\build\src\obj-firefox\dist\include\mozilla/StyleSheetInfo.h(17): note: see declaration of 'mozilla::StyleSheet'
09:10:04     INFO -  z:\build\build\src\obj-firefox\dist\include\mozilla/StyleSheetInlines.h(77): fatal error C1903: unable to recover from previous error(s); stopping compilation
09:10:04     INFO -  z:/build/build/src/config/ recipe for target 'Unified_cpp_gfx_tests_gtest0.obj' failed
09:10:04     INFO -  mozmake.EXE[5]: *** [Unified_cpp_gfx_tests_gtest0.obj] Error 2
Flags: needinfo?(nical.bugzilla)
Flags: needinfo?(nical.bugzilla)
We should reland this.
Flags: needinfo?(nical.bugzilla)
Pushed this to try again today and the windows builds are still failing on some mysterious and unrelated code, due to unified build hazards I presume.

This is pretty far down my list, so I don't think I'll investigate it in the short term. We could comment out the test as a short term thing if we want to save some infra.
Flags: needinfo?(nical.bugzilla)
Pushed by
Remove the multi-threaded job scheduler. r=jrmuizel
Pushed by
Remove the multi-threaded job scheduler. r=jrmuizel

The rs bustages are actually caused by an infra issue with autograph that started appearing coincidentally on your push,
I've relanded your changeset:

Sorry for the inconvenience!

Flags: needinfo?(nical.bugzilla)
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.