*** ChanServ sets mode: +v T4 | 02:25 | |
Benclark006 | good morning, just wondering if anyone knew some common reasons that the ui isn't working (or some places to look) thanks :) | 06:54 |
---|---|---|
T4 | <adampigg> R0kk3rz,.hmm,.not thought about it.... | 07:21 |
T4 | <adampigg> @benclark006 test_hwcomposer, check egl plugins are correct etc, all in the faq | 07:25 |
T4 | <adampigg> Logcat, journal | 07:26 |
Benclark006 | 👍 | 07:27 |
*** ChanServ sets mode: +v T4 | 08:19 | |
sledges | PSA: (was already a while in kernel checker) https://together.jolla.com/question/195734/release-notes-301-sipoonkorpi/#195734-for-those-porting-sailfish-to-other-devices | 08:50 |
sledges | r0kk3rz: @adampigg: please create topics for community meetup: guides/decisions on NFC and fingerprints need time from sailors that work on them (better than 'its closed were not ready' answer :)) | 08:50 |
sledges | PSA: Sailfish OS community #mer-meeting <- /join (or connect via any other applicable means) this channel in 10mins | 08:50 |
T4 | <adampigg> sledges, ok | 09:07 |
T4 | <adampigg> thx for the merge too | 09:07 |
T4 | <adampigg> someone should /connect t4 to mer-meeting | 09:08 |
sledges | it's there | 09:13 |
krnlyng | !seen dcaliste | 09:41 |
merbot | krnlyng: dcaliste was last seen in #sailfishos-porters 3 weeks, 1 day, 0 hours, 52 minutes, and 21 seconds ago: <dcaliste> I'm trying to add some debug in /usr/lib/libhybris/eglplatform_wayland.so which is created by compiling libhybris with --with-default-egl-platform=wayland. I'm wondering if there is a race condition in finishSwap() call there. | 09:41 |
r0kk3rz | hes in mer-meeting right now | 09:45 |
T4 | <adampigg> channel link someone? | 10:04 |
T4 | <adampigg> actually, i cant join from depecher anyway ... next time! | 10:04 |
dcaliste | mal: may I ask? In your wayland MR in gitlab, you are building new wayland-egl package that was previously built by mesa. How do you plan to migrate? | 11:24 |
dcaliste | There should be a conflict with mesa < x.y.z I guess and an upgrade in mesa packaging? | 11:24 |
mal | dcaliste: hmm, not sure what you mean, I think we use wayland-egl from libhybris | 11:25 |
dcaliste | I try to install wayland-egl-devel in SDK to compile SDL with the new version and it said that file /usr/lib/libwayland-egl.so.1.0.0 was providing by mesa-llvmpipe-libwayland-egl-9.2.5+git3-1.1.7.jolla.armv7hl | 11:27 |
mal | dcaliste: uninstall mesa-llvmpipe-libwayland-egl ? | 11:27 |
mal | it should then install the wayland version | 11:28 |
dcaliste | Yes, I may do this, but in term of packaging, if package B is providing a file that was previously in package A, it should conflict with package A so package A can be removed. | 11:29 |
dcaliste | Well, it's not really important, just wondering if it may raise later an issue when Mer MR will be accepted. | 11:29 |
mal | dcaliste: good point, I'll fix the packaging of wayland | 11:29 |
mal | in the MR | 11:30 |
mal | dcaliste: need to figure out when we need mesa for building | 11:32 |
dcaliste | Otherwise, wayland upgrade and SDL upgrade compile well in SDK, will going to test them on device. | 11:32 |
dcaliste | Besides, this apply in SDK, the same library is provided on device by libhybris-libwayland-egl indeed, as you mentioned. | 11:35 |
dcaliste | I'm not packaging expert at all, on how to plan this migration on both SDK and device. May need to ask pvuorella. | 11:36 |
mal | we don't want to depend on libhybris for sdl so probably either mesa or wayland should be used for libwayland-egl.so, probably wayland is better | 11:37 |
dcaliste | Yes, I fully agree, but in my opinion, it's the migration that requires to be planned, because currently libwayland-egl is provided either by hybris (on device) or mesa (in SDK). | 11:38 |
dcaliste | And to make this a bit more tricky, libhybris-libwayland-egl on device is not only providing libwayland-egl but also the plugin for libhybris called eglplatform_wayland that we need to keep. | 11:40 |
mal | yes, need to check that everything is correctly done | 11:42 |
dcaliste | yeh, sorry for raising more work to be done... | 11:43 |
mal | dcaliste: no problem | 11:46 |
dcaliste | mal: I agree, the crash can be easily reproduced. Need to investigate now. | 12:08 |
mal | dcaliste: just a moment, I'll find the hack for it | 12:26 |
dcaliste | No hurry, it's fine. | 12:26 |
mal | dcaliste: commenting out this line seems to prevent crashing https://github.com/SDL-mirror/SDL/blob/master/src/video/wayland/SDL_waylandvideo.c#L264 | 12:32 |
mal | this is the commit that brought the change https://github.com/SDL-mirror/SDL/commit/eff9d04664263fd27cdc1e9732af1381c381a1d6 | 12:32 |
mal | dcaliste: my guess is that either the too old qtwayland is causing the issue or libhybris wayland code has a bug, backtrace showed sometimes things from libhybris | 12:37 |
dcaliste | Thanks for the pointers. I'm testing on device on which I've a modified wayland_window.cpp from libhybris to avoid race conditions in calling dequeue(). | 12:39 |
dcaliste | It's still crashing though, and gdb backtrace indicate an issue in wayland-client.c from wayland package with a double free. | 12:39 |
dcaliste | I'm trying to install Valgrind but have network issues from phone at the moment... | 12:40 |
dcaliste | Besides, I'm also looking at the modifications between wayland 1.6 and 1.15 in wayland-client.c:dispatch_event() which is calling the closure that double free. | 12:41 |
mal | dcaliste: I suspected some race condition in libhybris that would cause double free | 12:41 |
dcaliste | Interestingly the free of the closure has been moved out of the locked section, for valid reasons, see https://git.merproject.org/mirror/wayland/commit/9744de9f472ec1e3c0e5b3416c097255e3bfdf12 | 12:42 |
mal | dcaliste: libhybris indeed is missing some things, it doesn't use the existing functions in case cases which are available in newer libhybris | 12:42 |
dcaliste | So the bug may already exist before but not triggered thanks to the locked section in dispatch_event(). | 12:42 |
dcaliste | What do you mean newer libhybris ? I'm using the master from github, is it not the latest ? | 12:43 |
mal | dcaliste: I mean newer wayland | 12:46 |
dcaliste | Ah, I understand :) | 12:46 |
mal | dcaliste: some stuff that is implemented in libhybris wayland code has been upstreamed | 12:46 |
mal | dcaliste: and the code has changed | 12:47 |
mal | dcaliste: I have some stuff ready for it, need to create PRs | 12:47 |
mal | maybe tonight | 12:47 |
dcaliste | Fine, I'm continuing reading all this code, trying to figure out a flow somewhere before adding fprintf everywhere... or finally succeed in Valgrind installation. | 12:48 |
mal | dcaliste: I think at least this was in upstream https://github.com/libhybris/libhybris/blob/master/hybris/egl/platforms/wayland/wayland_window.cpp#L132 | 12:51 |
mal | need to check after work | 12:51 |
T4 | <adampigg> ah,/so emu uses mesa? i wonder if browser works ok there | 13:47 |
T4 | <eugenio_g7> Tried, it didn't | 13:50 |
T4 | <eugenio_g7> Not on the latest version of the SDK/emulator though | 13:50 |
T4 | <eugenio_g7> I rememember it working back in 1.x | 13:51 |
T4 | <eugenio_g7> https://together.jolla.com/question/107726/sailfish-browser-almost-impossible-to-navigate11928/?answer=107770#post-id-107770 <--- it could be due to this | 13:56 |
T4 | <eugenio_g7> (the TJC question is unrelated) | 13:57 |
Thaodan | when will the plattform sdk be updated to 3.0.0.8? | 14:34 |
T4 | <elros34> have you tried this one http://releases.sailfishos.org/sdk/installers/1.23/ | 14:43 |
*** ChanServ sets mode: +v T4 | 14:55 | |
Thaodan | @elros34 I looked inside http://releases.sailfishos.org/sdk/latest/ | 14:59 |
Thaodan | the symlink wasn't updated | 14:59 |
T4 | <adampigg> Theodan, you do an irc client? Do you have an i486 build? | 16:20 |
r0kk3rz | Thaodan: usually once the release goes out after EA | 16:21 |
Benclark006 | when i run EGL_PLATFORM=hwcomposer test_hwcomposer i get test_hwcomposer: test_hwcomposer.cpp:179: int main(int, char**): Assertion `err == 0' failed. | 17:40 |
Benclark006 | Aborted | 17:40 |
Benclark006 | does anyone know a fix for this? | 18:42 |
T4 | <elros34> I'm trying backported bluetooth drivers on kernel 3.4. When I modprobe hci_smd ift fails because of unknown symbols. Any ideas? https://pastebin.com/w68ksNxE | 18:42 |
T4 | <elros34> Benclark006: provide logs: dmesg, journalctl | 18:43 |
Benclark006 | @elros34 journalctl https://paste.gnome.org/ppmpec5cz dmesg (phone) https://paste.gnome.org/pjqvvgata dmesg (host) https://paste.gnome.org/prqnnlici | 19:03 |
T4 | <elros34> so confusing logs... I bet nobody will bother to read them. Reboot, wait a while and type: dmesg, and journalctl —no-page | 19:12 |
T4 | <elros34> amybe add -b0 to journal to get latest boot logs | 19:13 |
mal | @elros34 you used my code? | 19:14 |
Benclark006 | ok | 19:15 |
T4 | <elros34> mal: yes, instruction from faq | 19:16 |
Benclark006 | journalctl -b0 https://paste.gnome.org/pntgorpti | 19:18 |
mal | Benclark006: we never need host dmesg anymore | 19:18 |
Benclark006 | 👍 | 19:18 |
mal | @elros34 just to be sure you haven't run make hybris-hal or hybris-boot after make backports? which you never do, make backports is always last | 19:18 |
Benclark006 | dmesg https://paste.gnome.org/pdsm4ov5o | 19:19 |
mal | Benclark006: that is only when you don't have telnet | 19:19 |
Benclark006 | i see | 19:19 |
mal | Benclark006: you have masked droid-hal-init? | 19:20 |
Benclark006 | maybe? | 19:20 |
Benclark006 | how do you do that? | 19:20 |
mal | you shouldn't have it masked if you want graphics | 19:22 |
mal | so run systemctl unmask droid-hal-init on device and reboot | 19:22 |
T4 | <elros34> mal: I was aware of that, first hybris-hal next backports | 19:22 |
mal | @elros34 are you sure kernel config changes are ok? | 19:23 |
Benclark006 | mal: doesn't look like that fixed it :( | 19:25 |
T4 | <elros34> I checked kernel configs in .config and they looks ok | 19:30 |
mal | Benclark006: new logs | 19:41 |
mal | Benclark006: dmesg and journal | 19:41 |
Benclark006 | sure, give me a sec | 19:41 |
Benclark006 | journalctl https://paste.gnome.org/pxjjnpija | 19:42 |
Benclark006 | dmesg https://paste.gnome.org/p6guidaiz | 19:42 |
mal | Benclark006: systemctl unmask user@100000 and maybe also systemctl unmask ofono | 19:48 |
T4 | <elros34> not sure about: Warning: unable to open an initial console. but what mer-kernel-checker prints | 19:48 |
piggz | https://twitter.com/adampigg/status/1083453566189211648 | 20:02 |
Benclark006 | well the graphics still don't work but hwcomposer works, i think | 20:06 |
Benclark006 | is it supposed to give out any output when it's running? | 20:06 |
mal | Benclark006: test_hwcomposer should show things in display | 20:16 |
Benclark006 | then it just hangs D: | 20:17 |
mal | Benclark006: so more log | 20:18 |
mal | *logs | 20:18 |
Benclark006 | journals https://paste.gnome.org/p8tqgn2qm | 20:19 |
Benclark006 | dmesg https://paste.gnome.org/plaqbdbqt | 20:20 |
*** ChanServ sets mode: +v T4 | 20:20 | |
mal | Benclark006: edit /init.rc and comment out line with init_user0 | 20:27 |
Benclark006 | ok so now it just bootloops to recovery | 20:35 |
Benclark006 | D: | 20:36 |
Benclark006 | i'll see if i can very quickly get some logs | 20:38 |
Benclark006 | mal: journalctl: https://paste.gnome.org/pqejgwyhu | 20:41 |
Benclark006 | dmesg: https://paste.gnome.org/ptttnrapx | 20:41 |
Nokius | is there away to change to devel repos from testing image ? I don't recall the way | 21:12 |
r0kk3rz | sure | 21:16 |
r0kk3rz | ssu dr the testing repos | 21:16 |
r0kk3rz | ssu ar the devel repos | 21:16 |
Nokius | vi /usr/share/ssu/features.d/adaptation-community.ini and vi /usr/share/ssu/features.d/adaptation-community-common.ini is what I do now :-/ | 21:17 |
Benclark006 | hey, i'm getting Sailfish jolla-startupwizard-pre-user-session[2589]: libminisf is incompatible or missing. Can not possibly start the SurfaceFlinger service. If you're experiencing troubles with media try updating droidmedia (and/or this plugin).== hwcomposer module == | 21:18 |
Benclark006 | in my system journal, is there a fix for this? | 21:19 |
mal | that is normal, not an error | 21:22 |
Benclark006 | i see | 21:27 |
Nokius | r0kk3rz: thanks, cu soon mate :) | 21:27 |
Nokius | mi3/mi4 is on SFOS3 :D thanks to Mike | 22:06 |
T4 | <It_sMike> You're welcome :) Thanks to mal too :) | 22:06 |
Nokius | https://twitter.com/Nokius/status/1083480495663529984 | 22:07 |
Nokius | ^ OTA | 22:07 |
Nokius | yes mal thanks to you for all the great help! | 22:07 |
Nokius | <3 | 22:07 |
T4 | <It_sMike> :) <3 | 22:07 |
mal | @It_sMike this is just wrong https://github.com/CancroSailfishOS/geoclue-providers-hybris/commit/be8b19720717102df2498341aa38e045afbb23a4 | 22:48 |
mal | those files are generated at build time | 22:48 |
mal | also why did you even fork that? | 22:48 |
T4 | <It_sMike> just testing around :D | 22:49 |
T4 | <It_sMike> I'll remove that | 22:49 |
T4 | <It_sMike> so sorry ! | 22:49 |
mal | for OBS build you can specify the used commit so you can easily build the needed older version | 22:49 |
T4 | <It_sMike> yes I know.... | 22:50 |
mal | also you have a mistake in OBS | 22:50 |
T4 | <It_sMike> Hmmmm .... what ? | 22:50 |
mal | geoclue package | 22:51 |
T4 | <It_sMike> so ..? | 22:51 |
mal | https://build.merproject.org/package/show/nemo:devel:hw:xiaomi:cancro/geoclue-provider-hybris this should be named geoclue-providers-hybris | 22:51 |
mal | you are missing one s | 22:51 |
T4 | <It_sMike> guess this wasn't mine :) | 22:51 |
mal | it cannot build now because it cannot know which spec file to use | 22:51 |
mal | ok, I can fix it | 22:51 |
T4 | <It_sMike> yeah !!! | 22:52 |
T4 | <It_sMike> Thanks mal!!! | 22:52 |
T4 | <It_sMike> :) | 22:52 |
T4 | <It_sMike> mal the revision should be set on 38ef0f1d8a56b56db5bfe99fdb88b9a0e684ca6d | 22:54 |
mal | yes, just noticed that | 22:55 |
T4 | <It_sMike> :) | 22:55 |
mal | I copied the wrong one accidentially | 22:55 |
T4 | <It_sMike> no problem :) | 22:56 |
T4 | <It_sMike> finally :D finished building | 22:58 |
T4 | <It_sMike> Thanks a lot mal. and sorry for that mistake ( o-o ) | 22:59 |
mal | the mistake has probably been there a long time but it didn't break until we added android 8 support | 22:59 |
T4 | <It_sMike> O-o | 23:00 |
mal | android 8 support added a second spec file to that repo and OBS decides the spec to use based on package name but in that project the package name didn't match either of spec files and therefore failed | 23:00 |
T4 | <It_sMike> Got it .... | 23:00 |
mal | just so you know in future that package name should usually match the spec file just in case | 23:01 |
mal | especially when there are more than one spec files | 23:01 |
T4 | <It_sMike> yes... of course :) | 23:01 |
mal | if the repo has only one spec file the package name doesn't matter but it's safer to name it correctly | 23:04 |
mal | at some point the testing projects should be fixed also | 23:05 |
T4 | <It_sMike> I see :) , Should always respect the naming rules ! | 23:05 |
T4 | <It_sMike> Hmm... fixing what exactly? the sources? | 23:06 |
T4 | <It_sMike> I don't have any access to my device testing project, should ask saidinesh5 to give me that permission | 23:10 |
T4 | <It_sMike> :) | 23:11 |
T4 | <It_sMike> projects* | 23:11 |
T4 | <It_sMike> mal, I have to go now... will talk tomorrow :) | 23:17 |
T4 | <It_sMike> Thanks again :) good night porters <3 ---{-@ | 23:19 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!