Closed Bug 868533 Opened 12 years ago Closed 12 years ago

Throttle or pause messages to and/or from processes when a call is incoming

Categories

(Firefox OS Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: justin.lebar+bug, Unassigned)

References

Details

Attachments

(3 files)

This is one (somewhat heavy-handed) approach we've considered for bug 847592. At the very least, if this approach solves our problem, it would help us understand what's going wrong. I have a wip patch for this that seems to work...
This patch is based on a previous patch where I changed CPU priorities, so the language still speaks about lowering a process's CPU priority. Read that as "stop sending and receiving messages from it".
Do we want to do this? It feels to me like this cure may be worse than the disease, because it will cause the phone to completely freeze for a second or two until we handle the call, but it probably would fix bug 847592 in many cases. We're still not protected from the case when the browser process spins the CPU e.g. in a JS loop. With a simple testcase that spins the CPU (http://bit.ly/mozloop), I can get a ~3s delay. I suspect that if I was doing graphics-intensive stuff which put a burden on the main-process compositor, I could get a longer delay. I'm not sure who should be making this UX call, particularly in light of the time-frame we have (I don't even know what the precise timeframe is, to be honest). jst, can you please advise as to who decides whether this is an acceptable solution?
Flags: needinfo?(jst)
I think this decision ultimately comes down to a combination of sicking, bent, and release drivers as to whether we want to take the risk in making a change like this at this late a time. Personally I'd be ok with taking this change, but I'm not exactly an expert in this field.
Flags: needinfo?(jst)
I think we can wontfix this; it looks like what we have in bug 870480 was the missing piece here.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: