790106 - Desktop B2G/FF crash due to misrouted UPower messages from WebBluetooth

Status: RESOLVED INVALID

(Core :: DOM: Device Interfaces, defect)

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

UPower uses dbus on desktop, but other comm services on the phone. Using bluetooth on the desktop will crash in debug after a while due to a failed main thread assert in upower handling. This is due to the fact that we pass on messages that aren't handled by bluetooth, and they're handled in the dbus signal catching thread, not the main thread.

Comment 1

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

This probably doesn't need to block basecamp since we aren't actively using Desktop B2G for much?

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

(In reply to Kyle Machulis [:kmachulis] [:qdot] from comment #1)
> This probably doesn't need to block basecamp since we aren't actively using
> Desktop B2G for much?

Dunno, I think we expect developers to use it (*I* certainly do :) ).

Comment 3

[Andrew Overholt [:overholt]]

While this seems very annoying, we can't block on desktop-only issues.

Comment 4

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

This is now a problem again now that we're trying to run bluetooth tests on b2g desktop. bent has been running into it for a couple of days.

Comment 5

[Andrew Overholt [:overholt]]

I can't say for sure whether or not it would be approved, but an uplift could be considered for a patch here.

Comment 6

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

I think once we get C++ tests in (which bent at least has started), we'll have better grounds for uplift.

Comment 7

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Fun fact and reason we haven't seen it in a while: This only happens on laptops running linux (either for real or in a VM). Desktops don't do battery queries. :)

Comment 8

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Attachment: Patch 1 (v1) - Either have UPower or bluetooth on for linux B2G Desktop, but not both

This is a quick fix. Not really something I expect to land as it means either bluetooth or battery works, but not both.

Comment 9

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Bluetooth hasn't worked on desktop in years. RIP.