libera/#maemo-leste/ Tuesday, 2024-04-02

sicelohttps://talk.maemo.org/showthread.php?p=1576024#post157602407:16
sicelobtw, i've forgotten what the OP should do if he doesn't get automatic provisioning of the gprs connection. anyone can help?07:17
freemangordonuvos: https://github.com/maemo-leste/dsme/commit/c6aa4ef1cb4d9ea8334ed23b6bebb975a523c0bf08:19
freemangordonwe are already there08:19
uvos__freemangordon: thats nice but without wider integration with the init system (makeing dsme redundant in the first place in the case of systemd) none of the actual benefits are realized by this11:32
freemangordonuvos__: we can't do that over the night12:37
freemangordonnot to say we have higher-prio tasks first12:37
freemangordonalso, elogind!=systemd, no?12:38
freemangordonBTW, what is the benefit of system over dsme?12:42
freemangordon*systemd12:42
Wizzupcan we not like go over this once every few weeks :D12:43
freemangordonwell, we are old people, memories are aging... :)12:44
Wizzupwhat I mean is we're not going to use systemd in the next half year at least, so there's really no point for people to keep bringing it up12:44
freemangordonI think that even SFOS guys who use systemd are still keeping dsme12:45
uvos__i was just mentioning that we have a problem that there are various paths to shutdown, wich skip various parts12:47
uvos__i just mentiond systemd would solve this, not trying to spark a debate12:48
uvos__all the discussions about this thus far have also been about different problems that would be solved by systemd so its not repetative12:48
uvos__freemangordon: a comment on the upower situation would be more usefull12:50
dsc_Wizzup: regarding https://github.com/maemo-leste-extras/hamsterfiler/issues/212:51
dsc_scrolling works in QEMU12:52
dsc_"it behaves like desktop scrolling without a scrollbar" <== what does this mean? :p12:52
freemangordonuvos__: I can't understand what the issue is12:53
freemangordonseems I have missed something12:53
uvosfreemangordon: upower the using code is non-trivial and beocomeing increasingly so (with the need to add support for devices with mulitple batteries, like the pinephone with keyboard or the mapphones with lapdock etc)12:54
uvosfreemangordon: but its duplicated in mce and in the power status applet12:54
uvosfreemangordon: the mce module dosent really do anything12:55
uvosfreemangordon: its just enableing the led patterns (which i propose the status applet could do) and dose the early low voltage shutdown for mapphones (which dosent really belong in mce - but needs to be discussed)12:55
uvos(the module i added)12:56
uvosso the proposal was to remove the mce code to eliminate the duplication12:56
freemangordonthe code that shuts-down on low battery?12:57
uvosyes so this code is a duplication of the code in upower (that also shuts down on low battery) but with the twist that we have a problem that the upower code is not sufficant in all cases12:58
freemangordonstatus applet cannot do, as it runs on behalf of user12:58
uvosthe status applet is the wrong place anyhow12:58
freemangordonmhm12:58
freemangordonmce is the right place as it has all the info12:58
uvosbut its not mces job either12:58
freemangordonI would say it is12:58
uvosupower allready dose this12:59
uvosbut manly it shutsdown is to late12:59
freemangordondoes it have inhibitors?12:59
uvosno which is where the disscussion came to systemd12:59
uvossystemd has system wide inhibitors that blocks all sources of shutdown12:59
freemangordonah, so upower asks systemd to shutdown?12:59
uvoswich is mutch better12:59
uvosit asks logind12:59
uvoswich asks systemd13:00
freemangordonand dsme asks elogind/systemd as well13:00
uvoselgoind is a stub that dosent do the same things, yes13:00
freemangordonso I don't see where the issue with dsme in that scenario is13:00
siceloso i'm the one who started this ... my question (not particularly related to mce or status) was - our code (in mce and status) picks the first battery and first charger, randomly13:00
uvosdsme isent a problem13:00
uvosits just redundant in a systemd system13:00
freemangordonthen why you brought it to the table - that was my question :)13:01
uvossince systemd can do watchdogs, can do inhibit and can supervise deamons and can reboot when restarting deamons fails13:01
uvosafaik all features dsme brings13:01
siceloso i was wondering if we should rather switch to upower's composite device, DisplayDevice. that way upower has already aggregated all the batteries13:01
freemangordonsicelo: I don;t see why not13:01
Wizzupthat was super buggy before, but maybe with all the module blacklists it is not13:02
freemangordonuvos: does systemd support HW WDs?13:02
siceloalternative, is ... if we care about there being more than one battery (some users do), then let's not pick a battery randomly, and rather find a way to present all of them13:02
sicelofreemangordon: so i opened https://github.com/maemo-leste/bugtracker/issues/72613:02
uvosfreemangordon: yes13:03
freemangordonwhat about boot mode?13:03
uvosnot sure what that is, it supports runlevels ofc13:03
uvosone of wich is active at boot13:03
uvosh13:03
freemangordonno, this is what is the boot reason13:03
freemangordonpower button, alarm, reset, etc13:04
uvosi dont know, but currenly we have no way cross device way to query this so we need something else for this anyhow13:04
siceloWizzup: ok. maybe i can experiment with DisplayDevice again and test in VM, D4, and N900. shouldn't be an issue though, i think. but i can think of pinephone people who may want to display their additional battery ... but maybe that could go to a widget or something13:04
freemangordonuvos: yeah, anyway, nothing will happen RN, so lets not waste time discussing it13:05
uvossure but you understand how the disscusion whent there13:05
freemangordon:nod:13:05
freemangordonsicelo: did anyone test upstream upower?13:05
sicelonot with Leste, no. but even our version has DisplayDevice, so we don't necessarily need to upgrade13:06
Wizzupsicelo: it's not an issue *now* that you have blacklisted the modules13:06
Wizzupbut it definitely used to be13:06
inkyfolks i have this question. is leste working on some device which has shared memory between gsm and ... and the rest.13:06
siceloah, i completely understand :-)13:06
inkydo we have some kernel patches to prevent allocator to use some memory?13:07
inkyto not corrupt gsm memory?13:07
Wizzupinky: is the memory is shared, then no, there is no iommu in the phones13:07
Wizzupif13:07
Wizzupbut the pinephone should have this isolated completely13:07
inkyor on our devices gsm is just connected via usb?13:07
WizzupI don't know if the droid modem has access to the ram13:07
Wizzupit can be connected over usb and also have dma13:07
Wizzup(through another way)13:08
inkyso we never had such a problem. i wonder if android kernels have such problem or postmarketos.13:08
sicelofreemangordon: we do need to keep the upower fork as long as we don't have another way to deal with 'broken' fuel gauges as found in N900 and Droid 4. ideally (and upower people insist) those should be handled kernel-side, but again, kernel isn't really a place for 'hacks.' between a rock and hard place13:08
siceloinky: what problem?13:09
inkymy friends were having similar problems with their embedded system, but they have fpga, not gsm, that shares the memory. and at some point the kernel allocator uses the memory which is shared with fpga.13:09
inkyso i immediately thought of the phones, and if this is solved somehow in the phones.13:09
siceloWizzup: so i think i'll work on switching mce and status item to DisplayDevice ... and we see how it goes. later on, if some device shows up where something doesn't work correctly, we'll see how to fix that device so it plays nice with DisplayDevice13:11
Wizzupok, if you want to, it does take some flexibility away from our side, if there are some kernel issues we will have to resolve them before being able ot use a device13:13
Wizzuplike mce might just decide to power off something based on displaydevice13:13
Wizzupare we sure this is the purpose of the displaydevice?13:13
Wizzupit sounds like this might change on the use case13:13
Wizzupi.e. if you connect some external battery some way, it might show that battery instead13:13
siceloDisplayDevice is an aggregate of all supplies reported in /sysfs. So if two, or more batteries are available to power the system, DisplayDevice will take all of them into account13:16
uvosyeah but how is the question13:17
siceloaverage, and some extra decisions in the code. e.g. if one battery is charging, DisplayDevice is charging, etc.13:18
uvosiirc on the pinephone with keyboard13:18
uvosits fine for isntance if either of the batteries is empty as long as the other has charge13:18
uvoson the mapphoen with lapdock its fine if the phone battery is empty but its not fine if the latpdock battery is empty regardless of the charge of the phone battery13:19
uvoscan we cover this13:19
uvosthat sort of thing13:19
Wizzupmy inclination is a more conservative if it aint broken don't fix it, but I'm happy if you want to explore the displaydevice avenue, I am just not sure if it's meant for what we want it to be13:20
WizzupIt sounds like it's mostly meant for *displaying*13:20
sicelouvos: the question in the case of mapphones is ... does it appear as another battery? if not, then upower/mce/etc. can't do anything about it because they simply don't know anything about the lapdock's battery. for pinephone, the keyboard's battery is added as a real battery in /sys, hence kernel/upower knows it's there13:21
Wizzupyes, we'll have to assume for the sake of the argument that it is added as battery in sysfs13:22
uvossicelo: on android android can report the lapdock power state13:22
uvosso its exposed somehow13:22
uvosno driver on mainline atm but thats different story ofc13:22
siceloi guess then atm there's nothing that can be done for it. if it ever has a driver, then i'd suppose it would show up in /sys. DisplayDevice wouldn't be able to ensure who goes empty first13:24
Wizzupright, but can we/mce rely on upower's judgement on when to power down due to low bat?13:27
Wizzupon upowers displaydevice judgement13:27
siceloWizzup: i understand. it's just i thought relying on an aggregated battery (even if not perfect) is somewhat more reliable than picking the first one that shows up, especially for pinephone. but maybe let's see it when the pinephone is more popular/active in Leste, since we don't really have other devices with multiple batteries yet (besides N900 bogus ones)13:28
uvosi gues in the lapdock case we dont need to power down when the lapdock battery is empty, android just warns you and kicks you out of lapdock mode13:28
uvosthe deivce remains alive ofc13:28
uvosits just that the lapdock display dies13:28
Wizzupsicelo: well if you blacklist the other ones you get the most reliable :P13:28
sicelohaha, but in pinephone, which one is that? :-D13:29
Wizzupprobably the phone battery13:29
Wizzupsince it will charge itself using the keyboard one13:29
uvosyeah but then you dont know how mutch charge you have13:29
Wizzupso it will be set to 'charging' I assume13:29
uvoswhy not leave mce alone for now (untill i come around to removeing it entirely)13:30
sicelook ... but then, the problem here is also that .. how does upower decide which is battery #1?13:30
uvosand display the composit device in the applet13:30
siceloe.g. if you boot the PP with the keyboard contraption attached...13:30
uvossicelo: for mce pinephone we would just blacklist everything except the internal battery13:30
uvosthe applet shows display device13:30
uvosand we need to move the activation of the led patterns into the applet since otherwise it will have the full pattern activated at bogus times13:31
uvos(any time the keyboard battery is not empty and thus keeping the internal one at 100%)13:32
siceloyeah. anyway, maybe let's leave the status item alone as well for the time being. no pp people have complained yet, i think :-)13:32
siceloi've closed the issue. hope my summary is correct? https://github.com/maemo-leste/bugtracker/issues/726#issuecomment-203206906015:37
sicelodoes anyone know why the debug LEDs on N900 are set to default-on in the DTS? https://elixir.bootlin.com/linux/latest/source/arch/arm/boot/dts/ti/omap/omap3-n900.dts#L5618:40
siceloi suppose something wasn't working fine all along, so the LEDs didn't really turn on each time. however, in recent kernels (e.g. 6.8) they now are always turned on, which seems correct according to the DTS18:41
sicelommm, or it's a config option that i had disabled before, https://cateee.net/lkddb/web-lkddb/LEDS_TRIGGER_DEFAULT_ON.html19:01
inkyfolks my friend wrote it in oberon with my compiler.21:06
inkyhttps://jabber.def.am:5443/upload/2acff0d43a041ca6ae894ca9e8762050ac829d9c/VjyhlTuCW7WvdGD1Sps7bq4EDdA039arNSj9jU0g/IMG_6887.mp421:06
inkyi will package it for maemo.21:06
uvosneat, very wierd choice of language21:20
inkythat's my favourite language.21:25
inkythat's why i have founded vishap oberon compiler in 2012.21:25
diejuseinky Interesting. I didn't know that language.21:42
inkydiejuse, oh you're here.21:43
inkythe keyboard owner asked you to contact.21:43
inkyhe made some changes for you.21:43
inkyauthor.21:43
inkyasked me to tell you to contact him.21:43
diejuseYes. I've been on vacation. I have read what he wrote to me this afternoon.21:45
inkyvacation is good.22:49

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