Closed Bug 1122570 (mozSettings-perf) Opened 9 years ago Closed 7 years ago

[meta] mozSettings API Performances

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

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.
Depends on: 1110063
Depends on: 1107982
Blocks: 1107982, 1110063
No longer depends on: 1107982, 1110063
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.
(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.
Depends on: 1122649
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.
Depends on: 1124102
Depends on: 1090810
Depends on: 1124622
Depends on: 1128489
Alias: mozSettings-perf
tracking-b2g: --- → +
Keywords: perf
tracking-b2g: + → ---
Whiteboard: [perf-wanted]
[Tracking Requested - why for this release]:
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.