Bug 102227 - Commit 26aee6f4d5 causes crash-loop on android-x86 (surfaceflinger to exit with status 1)
Summary: Commit 26aee6f4d5 causes crash-loop on android-x86 (surfaceflinger to exit wi...
Status: NEW
Alias: None
Product: Mesa
Classification: Unclassified
Component: Other (show other bugs)
Version: 17.2
Hardware: x86-64 (AMD64) other
: high blocker
Assignee: mesa-dev
QA Contact: mesa-dev
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-15 02:21 UTC by Mauro Rossi
Modified: 2019-04-11 05:45 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
logcat (84.04 KB, text/plain)
2017-08-15 02:21 UTC, Mauro Rossi
Details
dmesg showing surfaceflinger is not working (73.14 KB, text/plain)
2017-08-15 02:22 UTC, Mauro Rossi
Details
Screenshot capture of mesa 17.2.0rc4 working on AMD Turks after reverting the two commits (61.60 KB, image/png)
2017-08-15 02:23 UTC, Mauro Rossi
Details
Screenshot capture of mesa 17.3.0devel working on GT610 after revert of the two commits (49.12 KB, image/png)
2017-08-15 02:26 UTC, Mauro Rossi
Details

Note You need to log in before you can comment on or make changes to this bug.
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)


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.