<div dir="ltr"><div dir="ltr">Hi Lepton,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 15, 2019 at 9:58 PM Lepton Wu <<a href="mailto:lepton@chromium.org">lepton@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">That's interesting. The android frame work will hard coded to use<br>
RGBA_8888 and set it as window format here:<br>
<br>
<a href="https://android.googlesource.com/platform/frameworks/native/+/refs/heads/master/opengl/libs/EGL/eglApi.cpp#738" rel="noreferrer" target="_blank">https://android.googlesource.com/platform/frameworks/native/+/refs/heads/master/opengl/libs/EGL/eglApi.cpp#738</a><br>
<br>
Do you have a custom version of android which do different code here?<br></blockquote><div><br></div><div>Affirmative, you may check [1] for reference</div><div><br></div><div>[1] <a href="http://git.osdn.net/view?p=android-x86/frameworks-native.git;a=commit;h=792c8dc009bd3a0c44eb39e757a95e099c03b54c">http://git.osdn.net/view?p=android-x86/frameworks-native.git;a=commit;h=792c8dc009bd3a0c44eb39e757a95e099c03b54c</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
For other GPU, like intel, if we choose a EGL config with BGRA ,<br>
finally we got a wrong color on screen.<br></blockquote><div><br></div><div>android-x86 implementation selects BGRA_8888 only when RGBA_8888 not available,</div><div>in a similar way to a pre-existing Workaround 10194508 that was removed from AOSP code</div><div><br></div><div>Removing HAL_PIXEL_FORMAT_BGRA_8888 support from egl/android </div><div>SurfaceFlinger will just crash with SIGABRT on nouveau</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Thu, Aug 15, 2019 at 12:49 PM Mauro Rossi <<a href="mailto:issor.oruam@gmail.com" target="_blank">issor.oruam@gmail.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> sorry if I'm communicating in dedicated thread, but I don't know how to reply to existing one.<br>
><br>
> The proposed patch is based on the wrong assumption that all drivers used with Android have RGBA_8888, which is not correct, for example nouveau does not support RGBA on primary planes, infact we have a fallback to simpler EGL query in android-x86 and we use BGRA_8888 (pixel format = 5)<br>
><br>
> If patch does not solve a specific problem and it will cause a regression at least in nouveau, it is recommended to abandon the patch<br>
><br>
> Thanks for your feedbacks<br>
><br>
> Mauro<br>
</blockquote></div></div>