L10nRegistry Shared Memory & IPC
Categories
(Core :: Internationalization, enhancement, P3)
Tracking
()
Fission Milestone | Future |
People
(Reporter: zbraniecki, Unassigned)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [overhead:200k])
Comment 1•4 years ago
|
||
jimm says this L10n change is expected to save about 200 KB per content process. Work is planned for 2020 H2.
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
Comment 2•4 years ago
|
||
Moving this out to Fission-MVP as it's still waiting for L10 rewriting in Rust and will not be done in time to do this shared memory work by M7.
Updated•3 years ago
|
Reporter | ||
Comment 3•3 years ago
|
||
This is now unblocked! The cache that is currently stored per-process but should be shared between processes is here: https://github.com/mozilla/l10nregistry-rs/blob/master/src/registry/mod.rs#L26
My naive understanding of what has to happen here is:
- make it possible to share the
Vec<FileSource>
between processes - add logic to allow parent process instance to modify that cache
- add logic to allow content process instance to read from that cache
- add IPC methods for parent process to emit events when changes happen, and for content process to listen to those events
- Add IPC methods for content process to request changes that parent process can execute
:kmag - does the above plan look reasonable to you?
Also, can you take a look at the current importance of that for Fission? How much memory would we save? We also should take into account that we'll move another bunch of strings to Fluent in H2 2021 (optimistically completing M1 and M2 of https://www.arewefluentyet.com/).
Reporter | ||
Comment 4•3 years ago
|
||
Kris - this is now unblocked and we're making progress at moving more of DTD and .properties to Fluent.
Can you suggest how to reason about the priority of this? What kind of savings can we hope to get if we share memory between processes after moving all .properties to Fluent?
Comment 5•3 years ago
|
||
I'm removing the needinfo. If we or the Fission team want to prioritize this work, we can seek to get more input on Zibi's plan.
Updated•2 years ago
|
Description
•