Closed
Bug 1122570
(mozSettings-perf)
Opened 10 years ago
Closed 8 years ago
[meta] mozSettings API Performances
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(tracking-b2g:backlog)
RESOLVED
WONTFIX
tracking-b2g | backlog |
People
(Reporter: gerard-majax, Unassigned)
References
Details
(Keywords: perf, Whiteboard: [perf-wanted])
People are reporting "Settings API is slow". Let's document and investigate, with this meta bug to keep an overview.
Reporter | ||
Updated•10 years ago
|
Comment 1•10 years ago
|
||
Note that I'm especially concerned about the synchronous performance of calling `mozSettings.createLock().get()`.
I don't really mind if it's slow to call asynchronous callback.
But the synchronous performance is essential because we have a lot of libraries that are simply running this code when being merely inserted and executed, and this hurts application launch performance.
Comment 2•10 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #1)
> Note that I'm especially concerned about the synchronous performance of
> calling `mozSettings.createLock().get()`.
>
> I don't really mind if it's slow to call asynchronous callback.
>
> But the synchronous performance is essential because we have a lot of
> libraries that are simply running this code when being merely inserted and
> executed, and this hurts application launch performance.
Do you have a profile where this shows up? This shouldn't be a performance bottleneck at all. We just create an object and put it into a queue. Everything happens async behind the scenes.
Comment 3•10 years ago
|
||
I don't trust our polling profiles, too much noise, too random things, we can't see anything.
See bug 1122649: I posted there a debug patch I used to find this issue. Let's move this discussion there.
Updated•10 years ago
|
Alias: mozSettings-perf
Updated•10 years ago
|
tracking-b2g:
--- → +
Keywords: perf
Updated•10 years ago
|
tracking-b2g:
+ → ---
Whiteboard: [perf-wanted]
Reporter | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•