Closed Bug 1638920 Opened 4 years ago Closed 4 years ago

Replace the dedicated Rust `cert_storage` thread with a background task queue

Categories

(Core :: Security: PSM, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: lina, Assigned: keeler)

References

(Blocks 1 open bug)

Details

(Whiteboard: [psm-assigned])

Attachments

(1 file)

do_construct_cert_storage currently spins up its own thread for async persistence. We'd like to move away from one-off threads toward background task queues, and moz_task now exposes create_background_task_queue with a similar API.

The new a-s Rust components already use them for background I/O, and I think we can give cert storage the same treatment without changing too much code! 🎉

Assignee: nobody → dkeeler
Priority: -- → P1
Whiteboard: [psm-assigned]
Pushed by dkeeler@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6ee922ee7cc0 use a background task queue in cert_storage rather than a dedicated thread r=lina
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: