1250117 - Timeouts if gralloc is disabled on b2g

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Nicolas Silva [:nical]]

If gralloc is disabled on b2g we still go through the AsyncTransactionWaiter code such as https://dxr.mozilla.org/mozilla-central/rev/af6356a3e8c56036b74ba097395356d9c6e6c5a3/gfx/layers/client/TiledContentClient.cpp#572 expecting to get a notification back from the compositor, but since BufferTextureHost does not expose fence objects, SendFenceHandleIfPresent on the other side will not send what the AsyncTransactionWaiter is waiting on.

We should not create an AsyncTransactionWaiter unless we know that the TextureClient/Host works with fence objects or some other thing that will send the notification back from the compositor.
We should also pass the information along in the transaction that something will wait for the compositor, in order to be extra certain the compositor will send the notification back, otherwise it is very easy to get into this kind of deadlock.

Comment 1

[Nicolas Silva [:nical]]

Attachment: centralize the gralloc specific txn tracking logic and make sure it only runs with gralloc.

The remaining manual AsyncTransactionWaiter/Tracker usage is in ImageClient/ImageBridgeChild but it is a bit different and is not specific to gralloc so I think it's best to leave it as it is for now.

Comment 2

[Nicolas Silva [:nical]]

Juan, when you have some time could you test this patch with the device you are interested running b2g on without gralloc? This should fix the deadlock you were having.

Comment 3

[Nicolas Silva [:nical]]

Attachment: Centralize the gralloc specific txn tracking logic and make sure it only runs with gralloc.

Sorry for the noise, I forgot to hg qrefresh before submitting the previous patch.

Comment 4

[Sotaro Ikeda [:sotaro]]

Comment on attachment 8723015 [details] [diff] [review]
Centralize the gralloc specific txn tracking logic and make sure it only runs with gralloc.

Looks good.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4c4f342da9f5

Comment 6

[Nicolas Silva [:nical]]

Attachment: build fix

Comment 7

[Juan Gomez [:_AtilA_] (CET/CEST)]

Ok, tried the fix, and I could build without errors :)

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3a55e7c48a53

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/4c4f342da9f5
https://hg.mozilla.org/mozilla-central/rev/3a55e7c48a53