Closed Bug 1722668 Opened 3 years ago Closed 3 years ago

Speed up temporary storage initialization by removing old origin directories

Categories

(Core :: Storage: Quota Manager, task, P1)

Product:
Core
Component:
Storage: Quota Manager
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
94 Branch
Tracking Flags:
Tracking Status
firefox94 --- fixed

People

(Reporter: janv, Assigned: janv)

Details

Attachments

(5 files)

Bug 1722668 - Speed up temporary storage initialization by archiving unaccessed origin directories; r=#dom-storage
(deleted), text/x-phabricator-request
Details
Bug 1722668 - Implement a workaround for the case when GetLastModifiedTime returns INT64_MIN; r=#dom-storage
(deleted), text/x-phabricator-request
Details
Bug 1722668 - Expose nsFileStream via the component manager; r=#dom-storage
(deleted), text/x-phabricator-request
Details
Bug 1722668 - Implement nsIQuotaManagerService::getFullOriginMetadata; r=#dom-storage
(deleted), text/x-phabricator-request
Details
Bug 1722668 - Add test_unsetLastAccessTime.js for the case when quota information is cached; r=#dom-storage
(deleted), text/x-phabricator-request
Details

Some users have a lot of free space on disk so origins are never evicted which leads to large set of directories and files after some time (years). This can be a problem on slower hardware, so we should reduce number of origin directories if the initialization takes longer than X seconds. Origins which were used in last 13 months shouldn't be removed.

Depends on D123613

Attachment #9236966 - Attachment description: Bug 1722668 - Speed up temporary storage initialization by removing old origin directories; r=#dom-storage → Bug 1722668 - Speed up temporary storage initialization by archiving unaccessed origin directories; r=#dom-storage
Keywords: leave-open
Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a7179b67eb1a Implement a workaround for the case when GetLastModifiedTime returns INT64_MIN; r=jari https://hg.mozilla.org/integration/autoland/rev/c9f100cb3f86 Expose nsFileStream via the component manager; r=necko-reviewers,dragana,jari https://hg.mozilla.org/integration/autoland/rev/db05822c64ff Speed up temporary storage initialization by archiving unaccessed origin directories; r=jari

Backed out for failures on test_unsetLastAccessTime.js

[task 2021-09-04T21:08:20.666Z] 21:08:20     INFO -  TEST-START | dom/quota/test/xpcshell/test_unsetLastAccessTime.js
[task 2021-09-04T21:08:20.937Z] 21:08:20  WARNING -  TEST-UNEXPECTED-FAIL | dom/quota/test/xpcshell/test_unsetLastAccessTime.js | xpcshell return code: 0
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  TEST-INFO took 270ms
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  >>>>>>>
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2021-09-04T21:08:20.938Z] 21:08:20     INFO -  TEST-PASS | dom/quota/test/xpcshell/test_unsetLastAccessTime.js | undefined assertion name - There should be a testSteps function - true == true
Flags: needinfo?(jvarga)
Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/48e005ee27b7 Implement a workaround for the case when GetLastModifiedTime returns INT64_MIN; r=jari https://hg.mozilla.org/integration/autoland/rev/126032a9c310 Expose nsFileStream via the component manager; r=necko-reviewers,dragana,jari https://hg.mozilla.org/integration/autoland/rev/144d07c1d155 Speed up temporary storage initialization by archiving unaccessed origin directories; r=jari
Flags: needinfo?(jvarga)
Keywords: leave-open
Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/af59d0fac304 Implement nsIQuotaManagerService::getFullOriginMetadata; r=dom-storage-reviewers,jari https://hg.mozilla.org/integration/autoland/rev/0d3c7f057415 Add test_unsetLastAccessTime.js for the case when quota information is cached; r=dom-storage-reviewers,jari
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: