libera/#maemo-leste/ Thursday, 2021-09-02

uvosso the adress book depends on a specific email client which depends on a calandar widget which is a plugin for a program that depends on hildon-desktop and in turn allmost everything hildon which also depends on the calendar backend00:32
uvosclearly this is not sane and a cut needs to be made in some (or better several places) here00:33
freemangordonuvos: Well, the dependency is not that tight, it is more or less like calling xdg stuff: https://pastebin.com/CiWmK7F407:47
freemangordonalso, email client *does not* depend on calendar widget, but on a library that provides common ui elements07:49
freemangordonit just happens that this library is part of calendar-ui, which I agree is a bad design07:49
freemangordonhowever, maemo *is* hildon, like it or not, once we dehildonize it, it will be no more maemo but something else07:50
freemangordonWTF is 'Pip' supposed to mean in PipCalendarDialog, PipCalendarColor and PipColorPicker?!?08:33
freemangordonuvos: also, modest conditionally use color picker widget from calendar, it is #ifdef-ed09:00
freemangordonthis 'pip' drives me nuts, it is all over the place in calendar-ui, without a single hint what its meaning could be09:12
freemangordonnone of https://acronyms.thefreedictionary.com/PIP make any sense to me in the context. Anyways, I am going to  rename it09:19
tmlindso looks like weston-simple-dmabuf-egl works ok with current wlroots and gles2_texture_from_dmabuf_buffer works ok09:43
tmlindlooks like weston-simple-dmabuf-egl --drm-render-node=/dev/dri/card0 option is needed, otherwise it won't work. also needed for wlroots-0.14.009:44
tmlindso only weston-simple-egl is broken with current wlroots head, but works with wlroots-0.14.009:44
tmlindmaybe fd confusion between drm cards as the renderer is a separate card09:45
tmlindto me it looks like drmPrimeHandleToFD is not getting the right fd for weston-simple-dmabuf-egl09:46
tmlindfreemangordon, uvos: also glamor could get confused with the fd to use, worth checking09:47
uvosglamor uses prvk dri device only12:32
uvosor rather only the device you tell it to use12:32
uvosif you specify omapdrm it works fine (in software rendering ofc)12:33
uvosfreemangordon: if its just calling it via dbus thats fine maybe, still a bit annoying for packaging (it should maybe just use modest if available at compile time) the problem for me with this being dependancies that for instance would later force us to port everything to gtk3 at the same time before anything works. that kind of thing is not acceptable12:35
uvoslike if to port the calendar ui or watever to gtk3 i have to also port modest which means i have to port abook and hildon-home and hildon-desktop and so pushing us into a corner where we have to port everthing at the same time. this makes mantaining these things nigh impossible since you cant test any componant untill you port everything, and the dependancie on hildonized gtk2 also makes incoperating these things in other distros12:38
uvosimpossible.12:38
uvosso this kind of dependacy must be avoided imo12:39
uvosas it would be taking us imo further away from maintainablity and also what would be my personal eventual goal that leste could cease to exist as a distrobution and simply be a project like kde or whatever that provides a ui/telephony/etc solution for distros to package along side all the other normal DE they pacakge.12:40
Wizzupuvos: yes the connui-cellular port is not ready yet for primetime14:37
Wizzupuvos: libgofono was a bit hard to fit into the connui-cellular API14:37
uvosWizzup: i know, im just reporting14:39
uvosWizzup: the architecture of the status menu is simply bad, but thats another topic14:40
Wizzupmhm14:40
WizzupI think it's not too bad tbh, but ok :p14:40
Wizzupit's optimised for low resource usage14:40
uvosyeah it was sane on n900 maybe14:40
Wizzupuvos: you had more luck with another library for sfone, right?14:40
uvosbut i dont think its sane anymore14:40
uvosWizzup: anyhow maybe (if its not lots additional work) it might be worth it to ditch libgofono14:41
uvossince we have to use something else anyways for all the interfaces libgofono dosent cover14:41
uvosWizzup: well intels libofono is feature compleate14:41
uvosWizzup: but i dident end up using it (rather i just used ofono directly) because it dident fit well with the way sphone worked14:42
uvosso that was just less work14:42
WizzupI did quite some stuff but it wasn't ready yet https://github.com/maemo-leste/connui-cellular/commits/ofono-port14:42
uvosok14:42
uvosusing ofno directly is pretty sane, so is libofono or writing our own even14:42
uvoswe could also extend libgofono14:43
WizzupI need to look at what the exact problems were again, but I think I had to like call some glib event iteration functions to get the info from libgofono, since it took a while for it to process changes14:43
uvosbut libofono is very close in function (and glib usage)14:43
Wizzupthat kind of stuff14:43
uvosok14:43
uvosi havent used libofono at all but its internals looked very sane (i lifted code for sphone)14:44
uvosif you want to try that14:44
Wizzupthis kind of stuff: https://github.com/maemo-leste/connui-cellular/commit/cf74cda887e2d09a87635a528856caaf044e1d9c14:45
Wizzup(this is for the pin entry iirc)14:45
uvosok14:45
Wizzupyou can tell by the commit msgs I was getting a bit annoyed14:45
Wizzuphowever I currently have a strong hangover and I can't reason clearly about it :D14:46
WizzupI think once tor/wireguard is done (deadline in ~10 days), I will get back to audio, and then probably telepathy+cellular makes sense14:46
Wizzupthe commit msgs at least show how much had to be ported14:46
Wizzupbut basically it's currently full of hacks14:47
uvosanyone having the problem that the leste kernel dosent shutdown properly?16:00
Wizzupwhat device?16:00
uvosi just booted it by accident16:00
uvosd416:00
uvosmy private dev kernel shuts down fine16:01
uvosthe leste one dose not16:01
WizzupI don't think so, could it be related to the new dsme?16:01
uvos(5.11)16:01
uvosnew dsme isent tagged out or?16:01
Wizzupit's in -devel afaik16:01
uvoshm maybe the swtich in kernel came at the same time so yeah16:02
uvoshave to check16:02
uvoshmm no its something with the kernel16:08
WizzupI don't think we upgraded the kernel recently did we?16:10
uvosno but i havent used your kernel untill now16:12
uvosi was just swiching from one i built to the vanilla leste one16:12
Wizzupso what do you do, type 'sudo poweroff' and then it resets for you?16:13
uvosit just hangs sudo power off hangs at a blank screen with just a _16:13
uvospower off via dsme hangs with just the mce light on forever16:14
Wizzuplet me upgrade to latest sw and try16:14
uvosworks fine with the other kenrel16:16
uvos(should be same thing except i have some wip cpcap patches and a call audio hack applied)16:16
Wizzupcould also potentially be related to new mce perhaps16:17
Wizzuplet's see16:17
uvosi gues but i have been using that for a long time16:17
uvos(with dev kernel ofc)16:17
Wizzupat least before upgrade to latest mce and dsme 'poweroff' from terminal worked fine16:19
Wizzupbooting again16:19
Wizzupuvos: not seeing any problems here16:40
uvoshmm wierd16:49
freemangordonuvos: I agree, more or less, however, it is nopt now the time to start decoupling, lets first have something that works17:02
freemangordonotherwise we'll never end chasing our tail17:02
uvoswe need to create a ddk1.17 package17:43
uvosfor pure ddk17:43
uvoshttps://github.com/IMbackK/pvr-omap417:43
uvosthis works for chomeos mesa17:43
uvosbut im trying to setup pure pvr again rn and the dependacy mess is maddening17:44
tmlindyup.. so the dmabuf issues seem to be related to buggy mesa modifiers handling.. trying to fix17:54
tmlindthe chromeos mesa patches as in jonathan's tree do advertise modifiers, but it's buggy so currently disabled17:55
tmlindthe ddk-1.17 blobs don't even advertise modifiers which can cause issues guessing the format17:56
tmlinduvos: yup that pvrsrvinit.c totally does the trick :)17:56
tmlindoh then i did get weston-simple-egl running too with a temporary test hack of sudo ln -s /dev/dri/card0 /dev/dri/renderD128 after starting sway..18:00
Wizzupsounds like progressa18:01
tmlindthere's a bug somewhere where /dev/dri/card0 should be returned instead of the render device for dmabuf related ioctls..18:01
tmlindyup getting there slowly18:01
uvostmlind: nice19:27
uvosi cant get the pure blobs to work atm with our glbc for some reason (dosent want to load dri_pvr.so19:28
uvos)19:28
uvosWizzup: found out why it was locking up with the mobile conui19:28
uvosWizzup: was not your or its fault19:28
uvosbut my fault and a wierd interaction19:28
Wizzupuvos: ok19:35
lelIMbackK opened a pull request: https://github.com/maemo-leste/hildon-desktop/pull/15 (avoid regesitering presence more than once to avoid lockup)19:52
uvosfreemangordon: ^^^^ is required 95ec1becd1a413a77609629ab4ce73f8c51db11a but not 758d2685c788b19cf71d17fc25d65cba025f7ddd otherwise causes a lockup in certain situations19:52
konaMy n900 arrived! Time to party like it's 2009!20:32
Wizzup:)20:36
Wizzupkona: let me know if you'd like a d4 as well, I could ship you one in a few weeks20:36
konai haven't had any luck sourcing a d4. so maybe?20:59
konait's always fun to see what people leave on used devices.21:16
konain this case, email from 2010.21:16
freemangordonuvos: not sure I understand the issue (no PR or commit description)21:31
uvoshd_enumerate_input_devices can get called multiple times21:34
freemangordonah21:34
uvosXSelectExtensionEvent multiple times has interesting effects21:34
uvos(xorg hangs effectively)21:34
freemangordonwel,, why is not that in the commit message? :(21:35
uvos"avoid regesitering presence more than once to avoid lockup"21:35
uvosi gues "avoid regesitering for presence event more than once to avoid lockup"21:36
uvoswould have been enough21:36
uvosthe fact that you can use XSelectExtensionEvent to wedge the xserver seams like a bug tho21:37
uvosshould maybe report it21:37
freemangordonmaybe, but please, for the future, please leave a sentence or 2 to explain what the issue is and how the commit fixes it, ok? example: https://github.com/maemo-leste/dsme/commit/d866deb122101cffec00620947843fea1396568f . See, I am not nitpicking, it is really way easier to review if you know what it is all about, without trying to guess.21:38
freemangordonI think there is no use to raise a bug agains xorg21:40
freemangordonthey will silently ignore, most probably21:40
uvosnot if it affects xwayland :P21:40
freemangordonhmm, right :)21:40
freemangordondoes it?21:40
freemangordonBTW, what are the chances to run h-d on xwayland?21:41
freemangordonin theory it should run, no?21:41
uvosshould exist21:41
uvosthe same extension exists21:41
uvosyeah should run21:41
freemangordonwe shall try to start it someday21:41
uvosbut accelerated xwayland on d4 is hard/impossible21:41
freemangordonglamor?21:41
freemangordonisn't it the same as with ordinary x?21:42
uvosthere where other problems tmlind was investigating21:42
uvosask him21:42
freemangordonmaybe we shall follow the stream and focus on xwayland instead of trying to resurrect yet another dead man (xserver)21:43
freemangordon(just thinking out loudly)21:43
uvosmaybe21:44
uvosultimately we need to replace h-d really21:44
uvosbut thats nigh impossible manpower wise ofc21:44
freemangordonkona: BTW, I am on holiday starting tomorrow for a week21:45
freemangordonso far you have wpeditor in the repo21:45
freemangordonso you can install it21:45
konayes, it's installed for me :)21:46
freemangordonI am going to fulfill another modest dependency when I am back21:46
freemangordoncalendar-widgets that is21:46
freemangordonbut for now you may simply remove it temporarily while I finish the REing21:46
konaok!21:47
freemangordonif remove it from debian/control that is, configure.ac is already smart enough to deal with that21:47
freemangordonbasically, if there is no  calendar-ui-widgets pkgconfig package, modest will use HildonColorPicker instead of PipColorPicker (pip_color_picker_select_color())21:49
freemangordonNot that I know the difference :)21:49
freemangordonalso, it seems modest-providers-data is included as dependency, but not really used21:50
freemangordonkona: one question - do you rewrite tinymail gnomevfs module with gio or do a copy of it to be ported to gio21:51
freemangordon?21:51
lelfreemangordon closed a pull request: https://github.com/maemo-leste/hildon-desktop/pull/15 (avoid regesitering presence more than once to avoid lockup)22:04
uvosfreemangordon: great22:05
konaRewrite was my plan, do you have a beret idea?22:05
uvosfreemangordon: please coordinate with parazyd on any release of h-d (as the prs require removals and changes to the meta packages)22:06
konas/beret/better/22:06
konaWell, rename to libtinymail-gio22:07
konaI don't plan to rename the tinyvfs apis, though22:08
freemangordonkona: what about introduce a copy (libtinymail-gio) and to keep the current gnomevfs plugin22:09
freemangordonIt should be the same, no?22:09
freemangordonbesides that we'll keep it compatible with fremantle22:10
freemangordonand yes, no tinyvfs rename shall be done22:10
freemangordonuvos: I will, but I want to fix anotehr issue I just saw22:10
uvosnot that there is a reason not to but compatability with fremantle is fairly unimportant no?22:11
uvosfreemangordon: no rush22:11
uvosfreemangordon: im just saying that the old method of rotation needs to go at the same time on user devices :)22:11
freemangordonuvos: https://github.com/maemo-leste/hildon-desktop/blob/6eba1573905ac7b8e36a18ce52776811ef3c456d/src/util/hd-xinput.c#L1422:11
freemangordonwhy are those not static?22:11
uvosoversight22:11
freemangordonor rather - is there any particular reason?22:11
uvosno22:11
freemangordonok, going to fix22:11
freemangordonuvos: re tinymail - it is unimportant, but is more or less free22:12
uvosok22:12
uvosyeah was just asking it as a policy question22:12
freemangordonwe just don;t build gnomevfs module for leste22:12
uvos(ie is fremantle backward compatibilty a goal)22:12
freemangordonbesides fremantle, there are 4 or 5 more platforms that are supported22:13
freemangordonincludeing gnome desktop22:13
freemangordonso we either remove them all and leave leste only or add leste as another platform22:13
freemangordonI'd say - add leste as another platform given that wit costs no additional overhead22:14
freemangordon*it costs22:14
uvossure im not opposed22:14
freemangordonkona: what do you think?22:14
freemangordonI have already added mamo-leste platform22:14
freemangordon*maemo-leste22:14
freemangordonuvos: back to h-d: don;t we have the same issue with   DeviceMotionNotify() ?22:15
konaDoes tinymail even build for the others now?22:16
freemangordonfor sure it builds for fremantle22:16
freemangordonand I see no reason to not build for others22:17
konaAh, hmm.22:17
konaOk, no problem :)22:17
uvosfreemangordon: no because XCloseDevice unregisters all notfiers on the device afaik22:17
freemangordonthough I am not sure we shall keep it backward compatible after leste is fully functional22:18
freemangordonbut until then...22:18
freemangordonuvos: I mean here https://github.com/maemo-leste/hildon-desktop/blob/6eba1573905ac7b8e36a18ce52776811ef3c456d/src/util/hd-xinput.c#L8822:19
uvosyeah thats fine22:19
uvosno way that runs twice for the same device22:19
uvosas https://github.com/maemo-leste/hildon-desktop/blob/6eba1573905ac7b8e36a18ce52776811ef3c456d/src/util/hd-xinput.c#L5322:19
uvosguards it with22:19
uvoshd_close_input_devices22:20
uvosand that calls XCloseDevice on every device22:20
freemangordonbasically we get (possibly) different  xi_motion_ev_type value for every call, no?22:20
uvossure yeah you get it for the last device more or less22:21
uvosthats "fine"22:21
freemangordonok, if you say so I am not going to argue, but it looks suspicious to me, like, why event id for the last device is the same as for the others?22:22
freemangordonit is the same because the class is the same?22:23
freemangordon*is it22:23
uvosnot sure if its defined to be so22:23
uvosbut it is in practice22:23
uvosim reading docs rn to see what i can find22:23
freemangordonok22:23
freemangordongoing to fix those static vars in the meanwile22:24
freemangordonuvos: have a look at FindTypeAndClass macro22:26
uvoshttps://github.com/D-Programming-Deimos/libX11/blob/master/c/X11/XInput.h22:28
freemangordonit seems it just happens to return the same values for different devices22:28
uvosits allways the same22:28
uvostype =  _ip->event_type_base + offset22:28
uvosoffset is #define _deviceMotionNotify022:28
uvosand  _ip->event_type_base  is the same if device class is the same22:28
freemangordonso, _ip is the same for every device?22:29
uvoslooks like yeah for same input_class22:30
uvoslet me check22:30
freemangordonthanks22:30
uvosfreemangordon: https://gitlab.freedesktop.org/xorg/xserver/-/blob/master/Xi/opendev.c22:34
uvosit comes from a global here22:34
uvosthis really is an implementation detail of xorg22:34
uvosbut yeah it looks like its ok22:34
freemangordonyup22:34
uvosi gues a perfectly spec adhering x11 server could do something else22:35
freemangordonok, going to do a release22:35
Wizzupthanks for being thorough guys22:38
freemangordonuvos: will you have some spare time soon to look at matchboz focus (or whatever it was) issue?22:45
freemangordon*patchbox22:45
freemangordonaaah22:45
freemangordonMATCHBOX22:45
freemangordon:)22:45
uvosfreemangordon: sure22:52
uvosrn im still trying to figure out why the pure pvr setup dosent work for me anymore22:52
uvosyeah i dont get it so it goes:23:08
uvosMESA-LOADER: failed to open pvr (search paths /usr/lib/dri)23:09
uvosfailed to load driver: pvr23:09
uvosi can LD_PRELOAD=/usr/lib/dri/pvr_dri.so23:09
uvosand then it works23:09
uvosif i strace it goes openat(AT_FDCWD, "/usr/lib/dri/pvr_dri.so", O_RDONLY|O_LARGEFILE|O_CLOEXEC) = 423:10
uvosso23:10
uvosnot sure whats up with that23:10
uvosfreemangordon: any ideas?23:12
uvosfreemangordon: what to check even23:12
freemangordonI think we must LD_PRELOAD, otherwise we have issue because of the hacked libc version23:13
uvosthats not great23:14
freemangordonI wonder it works at all even with LD_PRELOAD if you use stock (and not 'hacked') blobs23:14
freemangordonsure it is not23:14
uvosno23:14
uvosld will just fail immidatly23:14
uvosbecause it checks the symbols23:14
freemangordonso, those are hacked blobs?23:14
uvosyeah23:14
freemangordonok, you have to LD_PRELOAD couple of them23:15
uvosjust usr/lib/dri/pvr_dri.so works fine23:15
freemangordonwon't work otherwise23:15
uvosanyhow maybe we should try and solve the issues that chromeos mesa has23:15
uvosas it needs nothing of the sort23:15
freemangordondoesn;t it need the same blob?23:15
uvosit uses some of the blobs23:16
uvosbut not that one23:16
uvoshttps://github.com/IMbackK/pvr-omap423:16
uvossee that23:17
freemangordonso, same shit, they are linked to the newer libc623:17
uvosright23:17
uvosbut no LD_PRELOAD needed23:17
uvosit just works after weaking the symbol versions23:17
freemangordonhmm, okm23:17
uvossince usr/lib/dri/pvr_dri.so is foss23:17
uvosand compiled against whatever23:17
uvosyou want23:17
freemangordonis it? ah. ok23:17
uvosyeah23:17
uvos(but it just loads the _MESA blobs23:18
uvos)23:18
uvosits just a wrapper23:18
uvosbut this is enough23:18
freemangordonright23:18
freemangordonbut, where is the source code?23:18
uvoshttps://github.com/IMbackK/mesa/tree/mesa-pvr-meamo23:18
freemangordonoh, this is the whole mesa23:19
uvosyeah the prv driver is in tree there23:19
uvosit just loads libglslcompiler.so and libGLESvX_PVR_MESA23:20
uvosno other blobs needed23:20
uvos(well you need extra stuff for pvrsrvinit23:20
uvos)23:21
freemangordonright23:21
uvosanyhow it has added issues23:21
uvosi use this for wayland23:21
uvosand for some reason glamor bails early because it dosent find a texture format it likes23:22
uvosand glmark also dosent work23:22
freemangordonhmm, weird23:22
uvosweston/ sway and kmscube work fine23:22
uvosso thats why i was trying to swtich23:22
uvosbut figureing iout why it dosent work might be better anyways23:23
uvossince you can compile egl with x11 support using this23:23
freemangordongbm should work23:23
freemangordonI am not sure you can, because x11 support was not compiled in the blobs23:24
uvoswell you can to the point that egl advertises it23:24
uvosbut no idea if it works23:24
uvossince xorg dosent start at all23:24
freemangordonit will not23:24
freemangordonbut anyway, it is too late here already and I am not sure I am of any use now :)23:25
uvosok23:25
freemangordonso going to have some sleep23:25
uvosbut where dose the x11 platform go into the blobs23:25
freemangordonin dri support23:25
uvossince the compiled out part you investigated long ago was in egl23:25
uvosok23:25
uvosso pvr_dri.so contains/needs support?23:26
freemangordondont; remember the details now, but every windowing system (WL,gbm, x11) has support in the blobs. x11 was compiled out23:26
uvosyeah but that was (at least what you showed in re) in libEGL23:27
freemangordonnot pvr_dri23:27
uvosthats no longer a blob23:27
freemangordonok, lemme check23:27
freemangordonuvos: IIRC, in libgbm23:28
uvosalso not a blob23:28
freemangordonuvos: libEGL23:33
uvosright23:34
uvosso theres a chance it would work23:34
freemangordonno, lemme show you the code23:35
freemangordon(decompiled)23:35
freemangordonuvos: see PM23:38
freemangordonbut basically:23:39
freemangordonif (WL) then plat = 223:39
freemangordonelse if (GBM) then plat = 123:39
freemangordonelse plat = 123:39
freemangordonso, only GBM and wayland23:39
freemangordonthe same is stated in release notes23:39
uvos(note for others): the function in question is implemented in foss mesa code in the chomos patches route23:44
uvoswe dont know however if this means anything23:44
uvosanyhow geting glamor to work for 2d accel on pure pvr blobs first makes sense23:45
uvossince then we dont have to contend with the additional buggyness of chomeos mesa23:45
uvoswe can switch later23:45
freemangordonuvos: h-d is build, night!23:52

Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!