Closed Bug 1122570 (mozSettings-perf) Opened 10 years ago Closed 8 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: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.