975974 - [Flatfish] Cannot boot up successfully and screen is black

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Danny Liang [:dliang]]

Flatfish cannot boot up successfully and screen is black.
In this case, we found below error message and it might be related to HWC.

02-24 15:42:14.960 I/HWComposer( 2032): Creating new instance
02-24 15:42:15.750 I/ServiceManager( 1263): service 'media.resource_manager' died
02-24 15:42:15.750 I/ServiceManager( 1263): service 'permission' died
02-24 15:42:17.240 I/ServiceManager( 1263): service 'media.audio_flinger' died
02-24 15:42:17.240 I/ServiceManager( 1263): service 'media.player' died
02-24 15:42:17.240 I/ServiceManager( 1263): service 'media.camera' died
02-24 15:42:17.240 I/ServiceManager( 1263): service 'media.audio_policy' died

With revert the commit 1816a1dc11a8595b54ce945181c39dce36ed2b7b, it works normal.

Build information:
Gecko: 23792bbdb46f25a00cf15ff07f808d8f4644d973
Gaia: f903eb45eed032a585ec3d4faac7fa38bf958ce8

Comment 1

[Danny Liang [:dliang]]

Attachment: flatfish_black_screen.log

Comment 2

[Danny Liang [:dliang]]

Update the backtrace by GDB:
(gdb) f 1
#1  0x412bf436 in mozilla::HwcComposer2D::Init (this=0x4452ee20, dpy=0x1, sur=0x4640d3c0) at ../../../gecko/widget/gonk/HwcComposer2D.cpp:100
100	    if (mHwc->query(mHwc, HwcUtils::HWC_COLOR_FILL, &supported) == NO_ERROR) {
(gdb) p mHwc->query
$1 = (int (*)(hwc_composer_device_1 *, int, int *)) 0

In widget/gonk/HwcComposer2D.cpp
 98 #if ANDROID_VERSION >= 17
 99     int supported = 0;
100     if (mHwc->query(mHwc, HwcUtils::HWC_COLOR_FILL, &supported) == NO_ERROR) {
101         mColorFill = supported ? true : false;
102     }
103     if (mHwc->query(mHwc, HwcUtils::HWC_FORMAT_RB_SWAP, &supported) == NO_ERROR) {
104         mRBSwapSupport = supported ? true : false;
105     }
106 #else

Comment 3

[Jeff Chuang]

hwcomposer is released in binary and I can't trace the sources. I am checking with the vendor about (1)whether the function query() is not implemented? (2)If it is not implemented, the possibility of adding this function or other suggestion. Waiting for the feedback.
Another suggestion is: Could we check whether mHwc->query is null before invoking it? If it is null, we could set mColorFill and mRBSwapSupport as false directly instead of invoking query().

Comment 4

[Jerry Shih[:jerry] (UTC+8) (inactive)]

(In reply to Jeff Chuang from comment #3)
> hwcomposer is released in binary and I can't trace the sources. I am
> checking with the vendor about (1)whether the function query() is not
> implemented? (2)If it is not implemented, the possibility of adding this
> function or other suggestion. Waiting for the feedback.
> Another suggestion is: Could we check whether mHwc->query is null before
> invoking it? If it is null, we could set mColorFill and mRBSwapSupport as
> false directly instead of invoking query().

If we check the query function before calling, I can boot my flatfish device.
I also set mColorFill and mRBSwapSupport to false.
With this setting, the hwc is only for FBSurface post.

Comment 5

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Attachment: hwc_fix.patch

check "query" interface before using

Comment 6

[Jeff Chuang]

The vendor confirmed that they do not implement the interface query(). Since the interface query() is not using by Android, they do not plan to add it at this moment.

Comment 7

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Hi Michael,

Could you review this patch?

The flatfish's hwc doesn't implement the "query" interface.
Thus, we can't use it in HwcComposer2D.cpp.
http://dxr.mozilla.org/mozilla-central/source/widget/gonk/HwcComposer2D.cpp?from=hwccomposer2d.cpp&case=true#102

This patch checks the query function before calling.
It also set the mColorFill and mRBSwapSupport to false if we can't use query interface.
In this case, the hwc is only used for swap buffer.
Currently, we have no plan to use hwc on flatfish. I think this patch is enough for our using.

Comment 8

[Michael Wu [:mwu]]

Comment on attachment 8381273 [details] [diff] [review]
hwc_fix.patch

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

r=me with nits fixed.

::: widget/gonk/HwcComposer2D.cpp
@@ +96,5 @@
>      int supported = 0;
> +
> +    if (mHwc->query) {
> +      if (mHwc->query(mHwc, HwcUtils::HWC_COLOR_FILL, &supported) == NO_ERROR) {
> +          mColorFill = supported ? true : false;

I know the code was like this before, but while we're in here, can you change the use of "? true : false" to "!!" ?

@@ +105,2 @@
>      }
> +    else { //hwc doesn't support query interface

else needs to be on the same line as }. No need for a comment - it is obvious enough.

Comment 9

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Attachment: hwc_fix.patch v2, r=mwu

update patch for comment 8

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/fe0f587afa4d

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fe0f587afa4d