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 | ||
https://hg.mozilla.org/mozilla-central/rev/67b54d48aad2
You need to log in
before you can comment on or make changes to this bug.
Description
•