libera/#maemo-leste/ Saturday, 2022-10-01

buZzfreemangordon: yeah not sure if make ALL locales is every really needed for anyone00:03
freemangordonthose are not all, just what we support00:04
freemangordonand usually this is done in the builder, libc upgrades are rare00:04
buZzwelp, its a lot :)00:05
freemangordonomg, it booted after the upgrade :)00:14
freemangordonsicelo: which kernel you were seeing usb issues with?00:23
freemangordonon n90000:23
sicelo5.19 onwards07:33
freemangordonah07:51
freemangordonbut, don;t we have bigger issue there, like, pvr driver cannot be compiled07:52
siceloi didn't test pvr :-)08:38
sicelommm, but i think migg08:40
siceloi think mighty17[m] has pvr working on 5.1908:41
freemangordonchromium is next to unusable on n900 :(08:48
freemangordonalso, I was thinking we are hitting off mode there, but I see average power usage ~450 mW in idle, do I miss something?08:48
sicelowe don't hit off mode at all on n90009:00
siceloneeds investigation.09:00
freemangordonmhm09:06
freemangordonlemme build omapconf there09:06
sicelowon't work :-(09:07
siceloomapconf is for omap409:07
freemangordonomapconf?09:07
freemangordonah09:07
freemangordonwhat is am335x09:07
freemangordonisn't that omap3?09:08
siceloyou can check with Wizzup ,but in general it seems using serial is better/easier way to track down off mode issues09:08
freemangordonI dont; have one :)09:08
siceloyes am335x is omap3 iirc09:08
freemangordonso, omap3 should be support09:09
freemangordonmaybe a matter of make flags09:09
* freemangordon checks09:09
siceloOMAPCONF CURRENTLY SUPPORTS TI OMAP44XX AND OMAP54XX DEVICES. LEGACY TI OMAP PLATFORMS (OMAP[1-2-3]) ARE NOT SUPPORTED.09:10
sicelofrom the readme09:10
freemangordonok, but code says different thing09:12
sicelooh ... let's hope it runs then :-)09:13
freemangordonhmm, debugfs is not mounted on n90009:15
freemangordonsicelo: actually sgx is the only one hitting OFF09:16
freemangordon:)09:16
siceloyou already got omapconf on it?09:17
freemangordonno09:17
freemangordon/sys/kernel/debug/pm_debug/count09:18
siceloright09:18
freemangordonhmm, usbhost_ick is on09:20
freemangordonsicelo: could you pastebin again the usb issue you have? or, it just does not work?09:28
siceloI don't have device/laptop handy right now, but let me check if there's somewhere i can still find at least part of it09:36
siceloi don't have it. sorry09:38
freemangordonyep, omapconf refuses to work09:41
uvosthere is a rebased pvr driver on kernel 6.0 in the sgx repo10:21
uvosso should compile10:21
uvosam335x is a omap4 with cortex a8, but otherwise more like an omap4 irrc10:23
freemangordonbut I am not sure it works, at least by looking at the ML10:23
freemangordonyeah, omapconf does not work10:23
uvossure if it works is a different question ;)10:24
freemangordonI have looked at the patches sent over the ML and I am not convinced those are the right ones10:25
freemangordonbut will look at it when it comes to it10:25
uvosyeah hitting off mode on n900 is only possible with nothing runnning and lots of kernel modules removed10:27
uvosWizzup has details, he messed with this alot10:27
freemangordonyep, will wait for him to comment10:29
freemangordonuvos: hmm, sphone is using 32104 MB RES memory, on n90010:43
freemangordonthis is the second largest user after hildon-desktop10:43
uvosres is not terribly useful mesurement, since it fails to acount for shm10:49
freemangordonhmm, h-d uses 34MB, on fremantle it use less than 10MB10:49
uvostry with https://github.com/pixelb/ps_mem10:49
uvosanyhow i would expect sphone to be a very hungy user, relatively speaking10:49
freemangordonwhy?10:49
uvosit keeps manny of its gtk windows in ram at all times10:50
freemangordonoh10:50
uvosim not sure its really nessecary, but the original author did this for responsiveniss on a new call etc10:50
uvosi gues the htc whatever it was was fairly slow spawning new windows10:51
uvosi have seperated all the ui code out from the backend code with the intention of slowly replaceing the windows one by one with qt alternatives, so theres that10:52
freemangordon7.4 MiB +   2.9 MiB =  10.4 MiBsphone10:52
uvosi mean its not nothing10:52
uvosbut its okayish i think10:52
freemangordon61.9 MiB +  18.6 MiB =  80.5 MiBmaemo-launcher (9)10:52
freemangordon:D10:52
uvosyes i mesured this expieramentaly before10:53
freemangordonI think you must make sphone .launcher in that case10:53
freemangordonit will free at least 9 MB10:53
uvosnot really10:53
uvosi mesured launcher reduceing ram by a couple of 100 kb10:54
uvosover all processies10:54
uvos(it moves ram usage around)10:54
freemangordonyes, really, see https://pastebin.com/CHG8PJm410:54
uvosim not sure what your trying to show me with that10:55
uvosi know it changes accounting10:55
freemangordonthe sum of all .launcher processes RAM usage is 80 MB10:56
freemangordonsphone alone uses 1010:57
uvosbecause the other processies dont have lots of pixmaps to thair name10:57
freemangordonif you move it as a launcher, I bet it will free at least 5MB10:57
freemangordonpixmaps are shared10:57
freemangordonin sapwood10:57
freemangordonalso, if you run as .launcher, you share pixmaps with other .launcher processes10:58
uvosstill distorying windows in sphone reduces ram by a lot10:58
uvosthe other processies besides h-d dont have any at that point10:58
freemangordonand this is valid for all theme pixmaps10:58
uvosi messured this with xterm before10:58
uvostotal ram usage is almost unchanged with .launcher xterm vs normal xterm10:58
freemangordonok, lemme open xterm to see how it affects the usage10:58
freemangordonwith 5 xterm windows, usage increased from 213.5 MiB to 219.4 MiB11:00
freemangordonlemme run it through maemo-summoner11:01
uvosmaemo-summoner is worse11:02
freemangordon221.2 MiB11:02
uvosthan just execing the binary11:02
freemangordonhow's that?11:02
uvosidk11:02
uvosbut that was the case11:02
uvos(back wen the binary could be execed)11:02
freemangordonyou cannot just execute the binary11:02
uvospre glibc changes11:02
freemangordonah11:02
freemangordonwell, now this is the result ^^^11:03
freemangordonI don;t see why maemo-summoner will make a difference11:03
uvosas i say some kb per process11:03
freemangordonno11:03
freemangordonlemme repeat for one xterm11:03
uvosit also dosent change btw11:03
uvosso a small application benefits most11:04
uvospercentage wise11:04
freemangordonno xterm -> 215.2 MiB11:04
freemangordon1 xterm -> 219.0 MiB11:04
uvosyes si11:05
uvosthere is allways only one xterm11:05
freemangordonsummoner - 219.6 MiB11:06
uvosso the first one using the most is unsuprising11:06
freemangordonsure11:06
freemangordonabout 600 KB diff for xterm11:06
freemangordonlemme check for addressbook11:06
uvosits allways around that yeah11:07
freemangordonok, but for 10 applications this is 6MB11:07
uvossure11:08
uvosusefullt on n90011:08
freemangordonmhm11:08
uvoson more moden devices not so mutch11:08
freemangordonright11:08
freemangordonalso, we don;t .launch qt symbols11:08
freemangordonif we do, qt apps will benefit a lot11:08
uvosi doubt11:08
uvossame story with launch speed to btw11:09
uvosalmost no difference on d411:09
freemangordonwhy, remember, this is c++ where symbol resolution takes ages11:09
uvosexcept launcher means that libs are allready loaded11:09
uvos(ie warm starts are the same)11:09
freemangordonesp for a huge lib like qt this differs a lot11:09
uvosim not conviced a launcher module for qt can work11:10
uvoswithout being a mess since qt changes alot with eatch release11:10
freemangordonwill see, when and if it comes to it11:10
uvosanyhow if you want to reduce sphones ram use11:10
uvosits more usefull to make it release its resources11:11
freemangordonsure, but the point it that .launch savings come for free11:11
freemangordon*is11:11
uvosnot really11:11
freemangordonhow so?11:11
uvossince it needs to sill work without launcher its just buildsystem work vs code work11:12
uvosand the code work benefits both11:12
freemangordonbuildsystem work is just adding dh_launcher to reules file11:12
sicelofreemangordon you wanted to perhaps fix the usb problem for n900?11:12
freemangordonsicelo: well, at least to try to identigy it11:12
freemangordonuvos: and maybe exporting a symbol through .defs file11:13
freemangordonwhile reworking resource usage is lot more work I assume11:13
siceloI'll see if can build kernel and share11:13
freemangordonok11:13
freemangordonno hurry11:13
sicelo:-)11:14
freemangordonuvos: re https://github.com/maemo-leste/libhildon/pull/911:14
freemangordonwhat is that workflow thingie?11:15
siceloI'm in a hurry, lol, before qt takes away your cpu cycles 😁11:15
freemangordon"1 workflow awaiting approval"?11:15
freemangordonsicelo: I don;t plan qt soon11:15
freemangordonnext is compositing accell in omap pvr xorg driver, most-probably11:15
uvosfreemangordon: i try and make sphones module load order not matter11:16
freemangordonno, I understand why the pr11:16
uvosfreemangordon: some modules require libhildon11:16
freemangordonwait11:16
uvosto not violate the ordering guarntee11:16
uvosi just init libhildon in eatch module that requires it11:16
uvosand then it throws warnings11:16
freemangordongithub wants me to approve some "workflow"11:16
uvosi dont think the warnings are critical11:16
freemangordonagree11:16
freemangordonand I am to merge it11:17
uvosidk about the workflow thing11:17
uvosnever seen it before11:17
freemangordonwhat I wont to know is if you requested that "workflow"11:17
freemangordonok11:17
freemangordonoh, it seems we have automatic tests11:18
uvosthats news to me11:20
freemangordononly libhildon seems to have it11:20
freemangordonI remember we try to keep it compatible with upstream gtk11:20
freemangordonuvos: so, in theory you should be able to run sphone with libhildon outside maemo11:21
freemangordonas long as you don;t use maemo-specific stuff11:21
uvoswhy would i need to?11:21
freemangordonneed?11:21
uvossphone just ifdefs libhildon stuff11:21
freemangordonno need, only if you want11:21
uvosthat works fine for me11:21
uvosok11:21
uvosalso as i say11:22
uvosi want to (and have prepared all the backend logic) to move sphone to qt one window at a time11:22
uvosso then libhildon becomes less relevant anyhow11:23
uvoswith time11:23
freemangordontmlind: I tested with that patch reverted on n900, I see no issues11:23
freemangordonsicelo: and once we have that, most-probably I will try to implement TE support in omapdss and then VRFB11:25
siceloI'll be really happy (vrfb)11:27
siceloi know/accept that n900 has really reached end of usefulness, but any little improvements/maintenance makes a welcome difference:-)11:29
freemangordonwell, rotation will not add much to the usefullness11:32
freemangordonthat's why it is with so low prio for me11:33
freemangordonoh, wait, I was about to fix connui-cellular :(11:34
uvoson larger devices it dose add a lot of usefullness imo11:35
uvosusing d4 one handed in landscape is akward11:35
uvosbut on n900 yeah11:35
uvosits small enough11:35
freemangordonwell, it is useful in lots of cases11:35
freemangordoncall-ui, book reading, etc11:36
uvossure, but i think its more amplified the bigger the device gets11:36
freemangordon:nod:11:36
freemangordonoh, I really want pvr stuff first, that's what I will do :)11:37
* mighty17[m] > <@sicelo:libera.chat> i think mighty17 has pvr working on 5.1911:51
* mighty17[m] doesnt remember working on 5.1911:51
mighty17[m]I'll give it a go tho11:51
freemangordonuvos: were you able to gather some ofono issues log?11:58
Wizzupfreemangordon: we hit off mode on n900 with certain drivers removed11:58
Wizzupfreemangordon: we also jhave some tools to debug this11:59
Wizzuphttps://github.com/maemo-leste/bugtracker/issues/54511:59
freemangordonI tried your script, but it does not work11:59
freemangordonmaybe a kernel patch is missing11:59
Wizzuphttps://github.com/maemo-leste/n900-pm/blob/master/scripts/openrc/n900-pm#L15212:00
Wizzupthis script?12:00
freemangordonyes12:00
freemangordonFile "<stdin>", line 14, in <module>12:00
freemangordonIndexError: list index out of range12:00
freemangordond=2022-10-01|t=11:16:03|i=OFF:0,RET:0|p=0|c=NA|b=none12:00
Wizzupfreemangordon: with our kernel?12:00
freemangordonyes12:00
Wizzuphmmmm I wonder if uvos forgot the patch them12:00
WizzupI don't test this specific feature every release of course12:01
Wizzupdebugfs is mounted?12:01
freemangordonthere is no idlest1 in /sys/kernel/debug/pm_debug/count12:02
freemangordonyes it is12:02
freemangordonthough I wonder why it is not mounted by default12:02
freemangordonso I have to mount it by hand12:02
Wizzupwell, debugfs is a security risk in some arguable cases12:02
freemangordonwell, I think we are far from security concerns12:02
freemangordonesp on n90012:02
freemangordonit is automounted on d412:03
uvosnopasswd sudo :P12:03
uvosfreemangordon: ni12:03
Wizzupso I am not sure what the exact patch name was12:03
uvosfreemangordon: no but will this weekend/monday12:03
freemangordonok12:03
uvosWizzup: what did i forget?12:03
Wizzupthe n900 patch that exports ret/off blocker info12:03
freemangordonWizzup: maybe this will help https://lists.goldelico.com/pipermail/letux-kernel/2019-December/004509.html12:04
uvospossible12:04
freemangordonWizzup: if you have older tree, just git log arch/arm/mach-omap2/pm-debug.c12:04
Wizzupfreemangordon: we had the patch in12:05
Wizzupfreemangordon: yes it's that patch but I had forward ported it12:05
Wizzupit's a bit silly we keep losing stuff12:05
uvosi think a problem is12:06
uvosthat several different people commit stuff to the kernel12:06
uvosas in patches12:06
uvosso it gets very dissorganized all th etime12:06
Wizzupas long as we don't merge in braches but rebase it should be ok12:07
uvoswith what is sill required what made it upstream and so on12:07
Wizzupbecause then all our commits are on top12:07
uvosdosent really help12:07
uvosbecuase you have to restart with linux-next really12:07
freemangordonWizzup: ok, what's the patch sha id?12:07
Wizzupfreemangordon: I am trying to find it for 10 minutes12:07
freemangordon"Wizzup: if you have older tree, just git log arch/arm/mach-omap2/pm-debug.c"12:08
Wizzupsec12:08
freemangordonand you'll see it12:08
Wizzupyeah waking up still12:08
Wizzupuvos: why @ restart?12:08
Wizzupfreemangordon: you'll want 8917211408204f5de88cf0fec3e42f3c42c8157012:08
Wizzupand also my revert of fb2c599f056640d289b2147fbe6d9eaee689f1b2 if you don't have it already12:08
uvosWizzup: because you have to examine every patch to see if its sill needed12:08
uvoshttps://github.com/IMbackK/droid4-linux/commits/n900-51612:08
uvosit should be here12:09
uvosi gues its not12:09
uvosso yes i forgot it12:09
uvosi makes the most sense to have feature branches like that12:09
WizzupI found this commit in wip/n900/maemo-5.15-cleaned-up12:09
freemangordonwell, I think it is because we merged to omap kernel12:09
uvosso that the patches are organized in a way that makes sense when it comes to figureing out which ones are still needed12:09
freemangordonand IMO it is normal some stuff was forgotten given the number of patches we carry12:10
freemangordonWizzup: lemme check what fb2c599f056640d289b2147fbe6d9eaee689f1b2 is12:10
Wizzupit's the commit that automatically disabled off mode12:10
Wizzupthis will cause all kinds of problems, during boot and at runtime (since off is not stable)12:10
uvosimo if we add stuff to feature branches first, it would make less stuff get forgontne12:10
freemangordonWizzup: have we ever tested on 5.18?12:11
freemangordonto enable off mode during boot that is12:11
freemangordonWizzup: we have it, 0396bd47a666fd16d14bb30dc3906c35f7d1138312:13
freemangordonok, so I'll pick 8917211408204f5de88cf0fec3e42f3c42c81570 and will rebuild the kernel12:14
Wizzupfreemangordon: not sure @ 5.18, but in general off mode seemed to have quite a few problems, so when I'd hit it,power usage would be great, but eventually it would crash/hang12:17
freemangordonugh, this adds stuff in debian12:17
Wizzupfreemangordon: huh?12:17
Wizzupoh :/12:17
freemangordonI'll fix it12:17
Wizzupty12:17
siceloregarding the pm patch, what would prevent upstreaming it btw? i don't remember if there was anything specific12:20
freemangordonno idea12:21
Wizzupit's just debug info, without any stable interface12:21
WizzupI suppose it could be, probably up to tmlind12:21
freemangordonI wonder if this is going to work correctly on omap412:22
freemangordonWizzup: a thing that is bugging me since long time ago - on d4, first wifi connection attempt always fails (when done by icd), any idea?12:35
WizzupI think the connections sometimes fail because the wpa supplicant dbus ids for a specific network in a scan expire and wpa supplicant has a new id for the same network12:36
Wizzupbut this is just my guess12:36
freemangordonthis happens only the first time on boot12:36
Wizzupfreemangordon: we already set the off mode bit on omap4 manually, so the auto patch doesn't do much there, but the kernel does not have code to enter off mode there, it enables us to get into RET12:36
Wizzupfreemangordon: I also have it in other cases12:36
freemangordonno, I am talking about tmlind's patch12:37
WizzupI don't think it works12:37
freemangordonthat dump cm_idlest12:37
freemangordonyes, but could it result in oops12:37
Wizzupour d4 pm script already has a waty read12:37
Wizzupah12:37
freemangordond=2022-10-01|t=13:37:52|i=OFF:0,RET:0|p=508|c=NA|b=ST_SDRC,ST_SDMA,ST_OMAPCTRL,ST_UART2,ST_I2C1,ST_MCSPI4,ST_MMC112:38
Wizzupyeah so if you boot to n900 without h-d and most drivers loaded, we easily hit ret all the time, and even off12:39
Wizzupif you enable off mode we -sometimes- hit ret, iirc, but it's unstable already12:39
Wizzupwe only hit it since I fixed up the ts driver to idle ok12:39
Wizzup(hit ret)12:39
Wizzupbut typically there's many modules that block idle, even ret12:39
Wizzuphow to best get at this list is what the long github issue is about, but I don't think I have a definitive list12:40
Wizzupsome automated way of loaded modules one at a a time (with deps) and reporting on idle could be interesting12:40
freemangordonlets see what chron will report12:41
freemangordon*cron12:41
freemangordonanyway, I am not going to debug that now12:42
freemangordonI wonder how hard would be to write off mode code for d412:42
freemangordonit should be in the android kernel already12:42
freemangordonseems there is also a patch that was send pack then https://lore.kernel.org/all/1315144466-9395-15-git-send-email-santosh.shilimkar@ti.com/12:46
freemangordonwait, this is in mainline12:49
freemangordonwhay do we think OFF mode is not supported?12:49
Wizzupwell, for the cpu yes, but not for the whole12:51
Wizzupwhich is what is necessarily afaiu12:51
freemangordonWizzup: https://www.spinics.net/lists/linux-omap/msg71661.html13:02
freemangordonI wonder why it didn;t make it13:02
freemangordonWizzup: for reference https://pastebin.com/HgiFwaV013:10
Wizzupfreemangordon: hmm I see13:18
freemangordonok, I still wonder why off mode series didn't make it upstream13:20
freemangordonthere is only one note https://www.spinics.net/lists/arm-kernel/msg179301.html , which tero replied to and that's all13:20
Wizzupmhm, don't know13:21
freemangordonI think it might make sense to try to port13:21
Wizzuplet's check with tony first13:23
freemangordonyeah, sure13:23
Wizzupbut nice find :)13:28
tmlindi think it's mostly context save and restore that's missing, that should be under drivers/soc now17:22
tmlindi can try to do an experimental rebase to play with at some point when i have some spare time17:23
sicelofreemangordon: i found some pics i took of the dmesg errors related to the usb issue. i know images may aren't the easiest/best way to share error messages. will still send proper log later, but here goes:21:14
sicelohttps://postimg.cc/Tytk5FzZ  , https://postimg.cc/vcwvTG14  ,  https://postimg.cc/tsXtdDNy  ,  https://postimg.cc/WqL0d71B  ,  https://postimg.cc/tYDVbbFQ21:15
Wizzuptmlind: would be sweet21:44

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