libera/#maemo-leste/ Saturday, 2021-11-20

uvosfreemangordon: ?00:55
uvosfreemangordon: no that dosent exist on fremantle either00:55
uvosmust be something you added localy.....00:55
uvosfreemangordon: yeah wierd @x11perf00:59
uvossame here on amdgpu + xephyr00:59
uvosfunny it goes away if you compose (tested with kwin_x11)01:00
uvoswith Xephyr -resizeable01:01
uvosyou can kinda see whats going on here01:02
uvosx11perf looks to be rendering in blocks01:02
uvosand fails to work correctly if a block is partially off screen01:02
uvosthat might be as intended01:02
uvosits copywinwin after all01:03
uvoscant copy a 100x100 block if the display isent that large01:03
uvoshappens on copypixwin too01:06
uvosso idk01:06
uvoswierd01:06
freemangordonuvos: for sure there is "end current task" on fremantle07:42
freemangordonctrl-backspace doesn't work either07:54
freemangordonwtf?07:54
freemangordonoh, it seems glmark2-es2 --fullscreen does not create window, somehow07:55
freemangordonhmm, it does not create even in non-fullscreent07:56
freemangordonhow is that possible?07:56
freemangordonyay, no more rendering artefacts in h-d with pvr EXA :)08:41
Wizzup:)09:01
freemangordonyeah, will clean-up the code a bit and will push09:02
Wizzupfreemangordon: fwiw I have end current task on leste09:02
freemangordonmodest scrolls with ~30fps vs 19 without pvr exa09:02
Wizzupfreemangordon: we will get it built today :)09:02
freemangordonWizzup: glmark2-es2?09:02
Wizzupfreemangordon: n900 or d4?09:03
freemangordond409:03
Wizzupfreemangordon: your code09:03
freemangordonhmm?09:03
freemangordondidn;t get that last one09:03
freemangordonafk, breakfast09:03
Wizzupfreemangordon: I said we'll build it once you clean it up09:04
tmlindfreemangordon: maybe the flicker is caused by racy vblank handling, maybe on n900 see if the flicker disappears if you temprorarily revert 860382142f5b ("drm/omap: Fix drm_handle_vblank() handling for command mode panels")09:07
tmlindreverting that should stop updating the command mode lcd on d4 though, but if it causes races it would explain the issue09:08
freemangordontmlind: ok09:12
freemangordontmlind: still, as I said, I think it is not kernel that shall force-update the display09:13
freemangordonuserspace knows when to do it, not kernel09:13
freemangordonfor example - I modified omap-video driver to flush the framebuffer on render operation being done09:14
freemangordonbut yeah, will see if it works fine on n900 with this reverted09:14
freemangordontmlind: do you know, if there is any 'drm' way to know if display is manually updated?09:16
freemangordonWizzup: keep in mind, what I am going to push will work corrctly on d4 only09:22
freemangordonthere are 2 things that have to be done to have n900 supported:09:22
freemangordonsomehow detect command-mode displays and force-flush only on those09:23
freemangordonand the second one is VRFB :)09:23
tmlindfreemangordon: no idea on how the manual update is supposed to happen, probably userspace09:31
tmlindwe already call omap_crtc_flush() that's a nop except for command mode displays, then 860382142f5b09:33
tmlindcommit attempts to simulate vblank09:33
freemangordontmlind: yeah, but I would still like to avoid unnecesarry ioctls if possible09:33
tmlindagreed yeah09:33
freemangordonlemme push what I have done so war and then will look at this09:34
tmlindthe weston timeout config options might give clues how it's supposed to work, sounds like weston has a timer to force refresh if it did not happen earlier09:34
freemangordontmlind: this sounds like a hack to me, but well, if there is no option09:35
tmlindyeah but maybe you're right and kernel does not even know except for the framebuffer stuff09:36
freemangordonI would avoid unnecessary wakeups (timer-based flushed) if possible09:36
tmlindright, event triggered refresh would be the best09:36
freemangordonI don;t see how kernel could possibly know when a render op is complete09:37
freemangordonimagine - it could be IVA that renders, not SGX09:37
freemangordon'renders' - like drawing to front buffer09:37
freemangordonit will be ugly, I know, but possible in practice09:38
tmlindyeah09:38
freemangordonhow is omapdrm supposed to know that IVA has finished?09:38
tmlindno idea, dma callback done seems like the only trigger09:38
freemangordonnot to say I don;t see any fences support in omapdrm, but it could be that it counts on drm code for that09:39
tmlindnot even sure iva uses dma, it might be bus mastering with it's internal dma09:39
freemangordonI forgot the details09:39
freemangordonbut it is not relevant09:39
freemangordonthe point is that it is userspace that has the information on when a flush, flip, swap, whatever shall happen09:39
tmlindwell reverting 860382142f5b and seeing if it fixed flicker on n900 might be a good test to narrow this down09:40
freemangordonyeah, will do, just lemme push what I have done so far :)09:40
tmlindyeah will be offline for most part, but will read the results at some point :)09:40
freemangordonok09:40
freemangordonWizzup: done09:49
freemangordonLMK when we have a kernel that produces SGX headers09:50
freemangordontmlind: with 860382142f5b reverted neither d4 nor n900 boot10:24
freemangordonoh, wait10:24
freemangordonI have one more change10:24
freemangordonlemme revert it and retry10:24
freemangordonok, n900 boots now10:32
Wizzupfreemangordon: ok, some time today10:49
freemangordonok10:49
uvosfreemangordon: yeah i do have end current task on leste11:03
uvosbut not on fremantle n900 maybe its something you added in cssu11:03
uvoswrt ctrl-backspace11:03
uvosi made this configurable per device, and unquley to the mapphones its configured to be the home button instead11:03
uvosofc you aperantly dont have ts-buttons in your kernel so its nothing now11:03
freemangordonwe should have consistent behaviour11:04
freemangordondevice-specific should be addition, not replacement11:04
freemangordonanyway, gtg now11:04
uvosright that was the plan11:04
freemangordonah, ok11:05
uvosbut we cant bind multiple keys to one action yet11:05
freemangordonok11:05
Wizzupfreemangordon: so this is ready for building? https://github.com/maemo-leste/droid4-linux/commits/droid4-pending-pvr-omapdrm-v5.1512:38
uvosWizzup: any luck with asking whats up with gitorious,org?16:56
uvos(just curious no pessure)16:56
freemangordonWizzup: yes17:55
freemangordontmlind: reverting 860382142f5b doesn't help18:00
Wizzupuvos: completely forgot...18:15
Wizzupnote to self http://web.archive.org/web/20130816141615/http://gitorious.org/droid18:16
bencohwell, I just ran apt-get upgrade on a droid4 (haven't upgraded for a few months I think), and it looks like xorg turned itself off (I guess)18:28
bencohI think it's trying to reboot18:28
bencohis that supposed to happen?18:28
uvosno18:33
bencohwell ...18:33
uvosdsme is broken with apt18:33
bencohhuhu18:33
uvosyou have to disable lifeguard18:33
uvosor its just broken18:33
bencohI guess I just broke my rootfs then :D18:33
uvosany daemon that dosent start after upgrade for any reason will break the system18:33
bencohusb doesn't work properly here, so ....18:33
uvosby dsme shuting down while apt is still upgrading18:34
bencohohwell18:34
uvosin recent times this was ke-recv18:34
WizzupI think dsme stop is a no-op, no?18:34
uvosWizzup: no thats not the problem18:35
uvosWizzup: any deamon that dosent start (in this case it was just ke-recv not starting becasue of a small bug that sometimes caused the init script to fail if restarted to fast) will break the entire system becaus lifeguard then reboots while apt is till upgrading18:36
Wizzupright, but why would init scripts fail18:36
uvosbencoh: just touch /etc/no_lg_reboots to disable lifeguard18:36
uvosWizzup: they should not, but dsme breaking the entire system becasue one deamon failed to restart during an upgrade is not ok18:37
uvosbencoh: you can usually avoid reinstalling by booting into recovery shell and finishing the upgrade18:37
uvosbencoh: depdens on what broke exactly ofc.18:38
Wizzupdsme does the right thing because if for example mce fails to start (10 times mind you) the device cannot be locked or unlocked, the ts is never enabled/disabled and a lot more is insanely broken18:39
freemangordon:nod:18:39
uvosWizzup: landing in a boot loop in this case is less usefull than a running system you can debug18:39
uvosbut this isent even about that18:39
bencohuvos: I think we should do that or something similar in every preinst/postinst18:39
uvosits abou dsme rebooting _DURING AN APT UPGRADE_18:40
uvosthis dsme should _NEVER_ do under any circumstances18:40
uvosbecause it breaks the system way more and way more permanently than some failing deamon18:40
bencohalright, let's forcehalt it then18:40
bencohI guess it at least finished writing buffers by now18:41
Wizzupuvos: yeah maybe we can do something with apt upgrades18:41
Wizzupnokia had special moved for that iirc18:41
Wizzupmodes*18:41
uvosi still think dmse rebooting is less usefull than it not doing anything18:41
uvosif the deamon is failing more than 10 times18:42
uvos a reboot wont help most likely18:42
uvosand it just forces an endless reboot loop18:42
uvosreally dsme should send a libnotfy notification18:42
uvoswith oh shit this deamon is broken seek help18:42
uvosthat would accualty be usefull18:42
Wizzupdoesn't help in your pocket18:43
Wizzupor for X18:43
Wizzupbut yes some way to assess system health would be nice18:44
uvosthe reboot dosent help either18:44
freemangordonno, what is actually failproof and useful is to *not* restart daemons on upgrade18:44
uvosno its not18:44
freemangordoncritical ones that is18:44
uvosdeamons have resources18:44
Wizzupyeah this is a sad debianism18:44
freemangordonno, this is set in debian/rules fille18:45
freemangordonwhat to do on upgrade (start/restart or not)18:45
uvosi def deamons should restart18:45
uvosits silly to have to reboot after upgrade18:45
freemangordonno, if you do system upgrade, you better restart the whole device18:45
freemangordonthis is not 24/7 server18:45
freemangordoneven you beloved android restarts on system upgrade ;)18:46
uvosnot haveing to reboot after upgrade is a usefull and well regarded feature on any device18:46
freemangordonanyway, have to cook18:46
Wizzupwhat does sphone do on upgrade? does it kill itself and start the new version?18:46
uvosi dont like android, why do you think im here18:46
freemangordon:p18:46
freemangordonttyl18:46
bencohapt-get said nothing to be done from emergency shell ... let's see18:46
parazydlmao why would you have to restart the whole system for a single daemon upgrade18:47
uvosWizzup: no it fails to work18:47
parazydYou should only be rebooting on kernel upgrades18:47
uvosWizzup: but thats not hwo its supposed to be ofc18:47
bencohwow, it booted to hildon18:47
bencohnot *that* broken after all18:47
uvosbencoh: lucky18:47
bencohyeah18:48
bencohwell, I should probably start debugging that usb thing, but ....18:48
bencoh(host says [1297603.744934] usb 2-1: USB disconnect, device number 33 after 3 seconds)18:49
Wizzupbencoh: this is exactly what someone else experieced a few days ago as well18:51
bencohI've been seeing this here since I moved to leste18:52
freemangordonparazyd: only when upgrading critical stuff like mce or Xorg, for example18:53
bencohPicking 'droid4-linux' as source package instead of 'linux-image-droid4'18:53
bencohE: Unable to find a source package for droid4-linux18:53
bencohwhen running apt-get source linux-image-droid418:53
bencohI dunno if that's expected18:53
uvosi made sure mce survives restart just fine18:53
uvosplease restart it on upgrade18:54
parazydbencoh: Did you add a deb-src in sources.list?18:54
bencohdeb-src http://maedevu.maemo.org/leste beowulf main contrib non-free droid418:55
bencohI might be missing a repository, but ...18:55
uvosdo we do source pacakges?18:55
bencoh$ apt-cache show droid4-linux18:55
bencohN: Unable to locate package droid4-linux18:55
bencohthis might explain the issue I guess18:55
parazydbencoh: ah it's not there indeed18:56
parazydMaybe because it's built a bit differently (using kernel scripts)18:56
uvosbencoh: its here btw https://github.com/maemo-leste/droid4-linux18:58
uvosif you need it18:58
Wizzupfreemangordon: yeah xorg also has abi18:58
Wizzupif we upgrade it and the input drivers change, then this can quickly become a problem18:59
Wizzupespecially since we disabled devices as well18:59
bencohuvos: yeah I'll just fetch from there, thx19:00
bencohI wanted to check what was used to build it actually, but ... let's see :)19:00
Wizzupthere's also 5.15 branch we're working on now19:04
bencohhow do you guys handle kernel upgrades / debugging on droid4?19:07
uvosbencoh: what do you want to acchive exactly?19:08
bencohdebug usb ... but I'd be happy with just a quick/easy way to downgrade in case I really mess the kernel19:08
uvosim not sure what the problem is19:09
uvosbuild your debugging kernel19:09
uvoscopy it to the device19:09
uvosand make a new boot entry19:09
uvosthen you can just switch when your new debugging/dev kernel fails for some reason19:09
bencohah, it's that easy with kexecboot? alright19:10
uvosyeah19:10
uvoslike grub really :P19:10
uvos(bootentrys are in /boot/boot/booot.cfg btw)19:11
bencoh(yeah, just found that, thx :) )19:11
bencohlooks nice19:11
bencohuh, I don't really understand why the maemo branch contain only the debian folders tbh19:18
parazyd19:10 <parazyd> ok so:19:24
parazyd19:10 <parazyd> $ git checkout maemo-5.1519:24
parazyd19:10 <parazyd> $ git checkout maemo/beowulf-experimental debian19:24
parazyd19:10 <parazyd> $ gbp buildpackage --no-sign --git-ignore-branch --git-ignore-new19:24
parazyd19:10 <parazyd> (I think that's more or less how Jenkins does it too)19:25
parazydbencoh: ^19:25
bencohthx19:25
bencohis that the debian recommended way to manage kernel packages (debian/ in a separate branch with an "upstream" tag)?19:28
bencoh(the 5.10.9 tag points to a 5.10.30 kernel, btw)19:28
uvosuh19:30
parazydSometimes, there's no definite standard really19:30
uvosthis is stupid19:30
uvoswe increment the version19:30
uvosindependent of mainline19:30
parazydThe tags are a mess but that's gonna change now with 5.1519:30
uvosgreat19:30
uvosi hated that19:30
uvosbencoh: also generaly check if your usb issue is in 5.15 even19:31
bencohparazyd: any reason why we don't keep a debian/ folder in the regular branch and just rebase/merge/whatever onto mainline everytime we need to?19:32
uvosbencoh: you can just build the droid4-pending-5.15 branch and boot the emergency shell with it19:32
uvos(it wont boot the gui ofc19:32
uvos)19:32
bencohis usbnet supposed to work without any config from userspace?19:33
bencoh(do we use configfs?)19:33
parazydbencoh: One (of a few) reasons is that the Linux kernel has its own build scripts and also has a .gitignore for debian/ by default.19:33
parazydbencoh: But you can easily checkout the directory when you're on the maemo-5.y branch19:34
bencohyeah that's what I did :)19:36
bencohah, we use configfs19:37
freemangordonugh, glmark2-es2 sets swap_interval to 0, which I guess means - 'do not wait for vsync', however, dri2 code understands this as 'blit to front buffer'20:02
freemangordonthat's why result on n900 is 28 vs 37 for -drm20:03
Wizzupah20:03
freemangordonbut, knowing this doesn;t help much with flickering20:03
freemangordonI suspect this is something in omapdrm, but have no idea how to chase it20:04
freemangordonhmm, if I disable page-flip path in DDX (so it blits) there is no flickering20:43
freemangordonand this is the reason why glmark does not flicker, but h-d does20:46
freemangordonh-d takes the fast route (page-flip_20:46
freemangordonunfortunately it is as slow as blit and flickers on top20:46
freemangordonso it is something on omapdrm20:46
freemangordonsomewhere in page-flip code20:46
Wizzupgetting there it sounds like :)21:34
freemangordonunfortunately no21:46

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