Firefox should fall back if staging fails with a memory error
Categories
(Toolkit :: Application Update, defect, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox111 | --- | fixed |
People
(Reporter: bytesized, Assigned: bytesized)
References
Details
(Whiteboard: [fidedi-ope])
Attachments
(2 files)
The updater needs to allocate memory in order to stage an update. We have several specific error codes for when staging fails due to inability to allocate memory. It looks like we currently react to these errors the way that we react to most errors: by clearing out the update and attempting to download another one.
But there is something that we should keep in mind when this happens. Staging means installing an update specifically while Firefox is running, and Firefox uses a lot of memory. If we failed to allocate sufficient memory to update when Firefox was running, we should try again when Firefox is not running. This is to say that when we get a memory error while staging, we should fall back to updating without staging.
|
||
Updated•1 year ago
|
|
||
Comment 1•1 year ago
|
||
But there is something that we should keep in mind when this happens. Staging means installing an update specifically while Firefox is running, and Firefox uses a lot of memory. If we failed to allocate sufficient memory to update when Firefox was running, we should try again when Firefox is not running. This is to say that when we get a memory error while staging, we should fall back to updating without staging.
This would just fall back to not staging for this particular update, correct? And then the next update cycle would respect the existing in-Gecko preferences?
|
Assignee | |
Comment 2•1 year ago
|
||
(In reply to Nick Alexander :nalexander [he/him] from comment #1)
This would just fall back to not staging for this particular update, correct? And then the next update cycle would respect the existing in-Gecko preferences?
That's correct.
|
|
Assignee | |
Comment 3•1 year ago
|
||
|
|
Assignee | |
Comment 4•1 year ago
|
||
Pushed by ksteuber@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f96a4c0174ee Fall back from memory errors during staging r=nalexander https://hg.mozilla.org/integration/autoland/rev/6c309593a0f9 Test for fallback on memory allocation error r=nalexander
|
||
Comment 6•1 year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/f96a4c0174ee
https://hg.mozilla.org/mozilla-central/rev/6c309593a0f9
Description
•