Closed Bug 884134 Opened 6 years ago Closed 6 years ago

Messages to vold need sequence number in gonk-JB

Categories

(Core :: DOM: Device Interfaces, defect)

All
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: mwu, Assigned: mwu)

References

Details

Attachments

(1 file, 1 obsolete file)

Messages to vold now require a sequence number at the beginning.
false on this line [1] will get vold working again with no gecko changes.  

[1] https://www.codeaurora.org/cgit/quic/la/platform/system/vold/tree/CommandListener.cpp?h=jb_mr1#n46
So does unconditionally passing 0 as a sequence number. Not sure this stuff actually does much for what we use it for..
Na, seems to be just for multiple clients and we don't have that use case atm.   netd fix is the same (0 or false)
I don't have a JB device with internal or external SD cards, so it would be nice to get confirmation if this completely fixes volume handling or not.
Assignee: nobody → mwu
Attachment #764923 - Flags: review?(dhylands)
Attachment #764923 - Flags: feedback?(mvines)
Comment on attachment 764923 [details] [diff] [review]
Prefix a sequence number on vold commands on gonk-JB

The sequence number ("0") is also returned back from vold, so needs to be skipped over at http://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/VolumeManager.cpp#154
Attachment #764923 - Flags: feedback?(mvines)
This patch also strips sequence numbers from incoming messages. We continue to assume the sequence number is zero.
Attachment #764923 - Attachment is obsolete: true
Attachment #764923 - Flags: review?(dhylands)
Attachment #767905 - Flags: review?(dhylands)
Attachment #767905 - Flags: feedback?(mvines)
Attachment #767905 - Flags: feedback?(mvines) → feedback+
Comment on attachment 767905 [details] [diff] [review]
Handle sequence numbers on vold commands on gonk-JB

Review of attachment 767905 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #767905 - Flags: review?(dhylands) → review+
https://hg.mozilla.org/mozilla-central/rev/764b9c296e69
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.