Halium / projectmanagement

144 stars 33 forks source link

libhybris issues for Fairphone 2 (FP2) #10

Closed peat-psuwit closed 5 years ago

peat-psuwit commented 7 years ago

Works

Doesn't work

Broken/skipped tests

z3ntu commented 7 years ago

test_hwcomposer is probably the same issue I encountered while porting Plasma Mobile which is that something tries to continously open /system/build.prop (and also never closing it)... Attachment: strace I did for Plasma Mobile

Also:

Vincent R., [18.04.17 13:21] @z3ntu I had the same issue (blobs) on my OnePlus One. Couldn't figure out the problem.

Luca Weiss, [18.04.17 13:21] [In reply to Vincent R.] good to know. what android version did you test it on?

Vincent R., [18.04.17 13:22] Cm12.1 and AOSP 5.1

Another also:

Bhushan Shah, [27.01.17 14:03] it is at least doing eglSwapBuffers

Bhushan Shah, [27.01.17 14:03] which is partly good

Bhushan Shah, [27.01.17 14:08] it seems it is opening build.prop but then not closing it

Bhushan Shah, [27.01.17 14:08] which is awkward

bhush9 commented 7 years ago

I am not sure if is same issue, I believe for you test_hwcomposer used to crash in the end right? but still strace and logcat during test_hwcomposer is run would be lovely @peat-psuwit

z3ntu commented 7 years ago

(I used sudo EGL_PLATFORM=hwcomposer strace test_hwcomposer for Plasma Mobile btw 😄)

peat-psuwit commented 7 years ago

logcat: http://paste.ubuntu.com/24690594/ strace: http://paste.ubuntu.com/24704553/ Note that they're from a different run.

I spotted failing ioctl on fd 3 with EPERM, digging into kernel.......

z3ntu commented 7 years ago

test_hwcomposer works with the latest rootfs (test_hwcompower didn't support hwcomposer version 1.4 so a patch had to be applied to the test)

z3ntu commented 7 years ago
z3ntu commented 7 years ago

When ignoring the float issue with libhybris (=changing printf of floats to integers (casting)), the sensor test lists all sensors and then fails with *** Error in '/root/libhybris/hybris/tests/.libs/test_sensors': free(): invalid pointer: 0xb6bd90a8 ***

z3ntu commented 7 years ago

gps works too (after working around the printf bug)

z3ntu commented 7 years ago

EGL_PLATFORM=hwcomposer test_camera crashes at

#0  0xb3c19b48 in HWComposerNativeWindow::setup(gralloc_module_t*, alloc_device_t*) () from /usr/lib/arm-linux-gnueabihf/libhybris-hwcomposerwindow.so.1
#1  0xb3f11c6a in hwcomposerws_CreateWindow () from /usr/lib/arm-linux-gnueabihf/libhybris/eglplatform_hwcomposer.so
#2  0xb6fafbde in eglCreateWindowSurface () from /usr/lib/arm-linux-gnueabihf/libhybris-egl/libEGL.so.1
#3  0x00011bd8 in main (argc=<optimized out>, argv=<optimized out>) at test_camera.c:387

EGL_PLATFORM=hwcomposer test_glesv2 crashes at

#0  0xb66f8b48 in HWComposerNativeWindow::setup(gralloc_module_t*, alloc_device_t*) () from /usr/lib/arm-linux-gnueabihf/libhybris-hwcomposerwindow.so.1
#1  0xb6747c6a in hwcomposerws_CreateWindow () from /usr/lib/arm-linux-gnueabihf/libhybris/eglplatform_hwcomposer.so
#2  0xb6f37bde in eglCreateWindowSurface () from /usr/lib/arm-linux-gnueabihf/libhybris-egl/libEGL.so.1
#3  0x00010d4a in main (argc=1, argv=0xbefffd84) at test_glesv2.c:116
peat-psuwit commented 5 years ago

Superseded by #135.