Closed
Bug 1389171
Opened 7 years ago
Closed 7 years ago
DLL readahead should use normal priority
Categories
(Core :: Widget: Win32, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: agashlin, Assigned: agashlin)
References
Details
(Whiteboard: tpi:+)
Attachments
(1 file)
Following the discussion in bug 1367416 comment 33 and on, it will be more likely for the readahead to succeed in time if we use normal priority on the thread instead of low priority.
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Priority: -- → P2
Whiteboard: tpi:+
Assignee | ||
Comment 2•7 years ago
|
||
try with ts_paint shows no significant change in startup time, looks safe.
https://treeherder.mozilla.org/perf.html#/compare?originalProject=mozilla-central&newProject=try&newRevision=26eaaf35120d32a46f50441f9d852786b9e80a9a&framework=1&filter=ts_paint&showOnlyImportant=0&selectedTimeRange=172800
Assignee | ||
Updated•7 years ago
|
Attachment #8896051 -
Flags: review?(jmathies)
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8896051 [details]
Bug 1389171 - normal priority DLL readahead thread
https://reviewboard.mozilla.org/r/167320/#review172950
Attachment #8896051 -
Flags: review+
Updated•7 years ago
|
Attachment #8896051 -
Flags: review?(jmathies)
Pushed by aklotz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/67b54d48aad2
normal priority DLL readahead thread r=aklotz
Comment 5•7 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•