Closed
Bug 938806
Opened 11 years ago
Closed 10 years ago
Handle errors better in GonkFMRadio
Categories
(Core :: Hardware Abstraction Layer (HAL), defect)
Tracking
()
RESOLVED
FIXED
mozilla33
People
(Reporter: mwu, Assigned: mwu)
Details
Attachments
(1 file, 1 obsolete file)
2.50 KB,
patch
|
dhylands
:
review+
|
Details | Diff | Splinter Review |
Right now, we don't necessarily report failures while setting up the FM radio. This needs to be fixed.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8439431 -
Flags: review?(dhylands)
Comment 2•10 years ago
|
||
Comment on attachment 8439431 [details] [diff] [review] Improve error handling in FM radio init path Review of attachment 8439431 [details] [diff] [review]: ----------------------------------------------------------------- I like it. ::: hal/gonk/GonkFMRadio.cpp @@ +300,5 @@ > > if (sMsmFMMode) { > sRadioFD = fd.forget(); > sMsmFMVersion = cap.version; > pthread_create(&sRadioThread, nullptr, runMsmFMRadio, nullptr); Should this code path also generate some type of notification? Right now it looks like all of the other ways of exiting this function will create a notification, except for this one.
Attachment #8439431 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 3•10 years ago
|
||
runMsmFMRadio is suppose to generate its own notification. I suppose if pthread_create fails, that won't happen. I'll add a check for that too.
Assignee | ||
Comment 4•10 years ago
|
||
This also ensures pthread_create succeeds.
Attachment #8439431 -
Attachment is obsolete: true
Attachment #8439549 -
Flags: review?(dhylands)
Comment 5•10 years ago
|
||
Comment on attachment 8439549 [details] [diff] [review] improve-fm-init-error-handling-v2 Review of attachment 8439549 [details] [diff] [review]: ----------------------------------------------------------------- Looks good.
Attachment #8439549 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/5bb301cd0e7c
https://hg.mozilla.org/mozilla-central/rev/5bb301cd0e7c
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in
before you can comment on or make changes to this bug.
Description
•