Bug 102227

Summary: Commit 26aee6f4d5 causes crash-loop on android-x86 (surfaceflinger to exit with status 1)
Product: Mesa Reporter: Mauro Rossi <issor.oruam>
Component: OtherAssignee: mesa-dev
Status: RESOLVED MOVED QA Contact: mesa-dev
Severity: blocker    
Priority: high    
Version: 17.2   
Hardware: x86-64 (AMD64)   
OS: other   
Whiteboard:
i915 platform: i915 features:
Attachments: logcat
dmesg showing surfaceflinger is not working
Screenshot capture of mesa 17.2.0rc4 working on AMD Turks after reverting the two commits
Screenshot capture of mesa 17.3.0devel working on GT610 after revert of the two commits

Description Mauro Rossi 2017-08-15 02:21:31 UTC
Created attachment 133516 [details]
logcat

In attachement a logcat/dmesg of not working build on AMD Turks, with
surfaceflinger not liking the egl+gallium_dri combination
and exiting at init in an infinite loop.

[   29.182766] init: Starting service 'surfaceflinger'...
[   29.280027] init: Service 'surfaceflinger' (pid 2773) exited with status 1
[   29.282251] init: Service 'surfaceflinger' (pid 2773) killing any
children in process group
[   29.284439] init: Service 'zygote' is being killed...

The first bad commit is:

26aee6f4d5a28350e1f6becfc485ce929c4126f9 Android: rework LLVM build support

As a proposal the following two commits can be reverted in mesa 17.2.0rc4

git revert d31a2b4d49274ca362097a9707d9e9a48ea58063
[detached HEAD 455978c029] Revert "Android: Add LLVM support for Android O"
 1 file changed, 1 insertion(+), 4 deletions(-)

git revert 26aee6f4d5a28350e1f6becfc485ce929c4126f9
[detached HEAD 70f3377e4e] Revert "Android: rework LLVM build support"
 8 files changed, 23 insertions(+), 27 deletions(-)

This will give a working build for nouveau, r600 and all other gallium_dri drivers.

Tested with nougat-x86 on AMD Turks (HD76xx) and Nvidia NVD9 (GT610)
Kind regards

Mauro Rossi
android-x86 team
Comment 1 Mauro Rossi 2017-08-15 02:22:24 UTC
Created attachment 133517 [details]
dmesg showing surfaceflinger is not working
Comment 2 Mauro Rossi 2017-08-15 02:23:55 UTC
Created attachment 133518 [details]
Screenshot capture of mesa 17.2.0rc4 working on AMD Turks
after reverting the two commits
Comment 3 Mauro Rossi 2017-08-15 02:26:24 UTC
Created attachment 133519 [details]
Screenshot capture of mesa 17.3.0devel working on GT610 after revert of the two commits
Comment 4 Dainius Masiliƫnas 2018-06-28 15:04:35 UTC
I'm experiencing this as well, on an AMD Brazos machine.
Comment 5 Emil Velikov 2018-06-28 17:24:14 UTC
A couple of things come to mind:
 - HAVE_LLVM is not defined at some point
Temporary reinstate only the required LOCAL_CFLAGS line in Android.common.mk 

 - building some parts with c++11 causes problems
Move the "LOCAL_CPPFLAGS += -std=c++11" within the MESA_ENABLE_LLVM block

If either of these work, don't be shy to send a patch to mesa-dev@ ;-)
Comment 6 Pokemon 2019-04-11 05:45:42 UTC Comment hidden (spam)
Comment 7 GitLab Migration User 2019-09-18 20:18:06 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/918.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.