libera/#maemo-leste/ Wednesday, 2023-11-29

Wizzupfreemangordon: I can't figure out how to make a channel using ensureChannel* or createChannel* and actually pass any properties on the channel00:34
Wizzuptelepathyprovider.cpp:136:135: error: use of deleted function ‘Tp::Client::ConnectionInterfaceAnonymityInterface::ConnectionInterfaceAnonymityInterface(Tp::Client::ConnectionInterfaceAnonymityInterface&&)’00:58
Wizzupreally00:58
WizzupI don't know how anyone ever just groks the TP APIs00:58
Wizzupi'm just trying to set the anonymity bits using tp qt and it seems freaking impossible00:58
Wizzupthey just offer this https://telepathy.freedesktop.org/doc/telepathy-qt/a09124.html00:59
Wizzupand a similar way for a channel00:59
Wizzuplooking at this: https://telepathy.freedesktop.org/spec/Connection_Interface_Anonymity.html#Property:SupportedAnonymityModes01:32
WizzupNote: Connections implementing this property SHOULD provide a corresponding parameter named org.freedesktop.Telepathy.Connection.Interface.Anonymity.AnonymityMandatory with the DBus_Property flag. Clients SHOULD update this property by calling UpdateParameters on the relevant Account rather than setting the property directly; change notification is via AccountPropertyChanged.01:32
WizzupI can't get this to work with mdbus201:33
Wizzupit works fine on the connection interface, but not on the account01:34
Wizzup$ mdbus2  org.freedesktop.Telepathy.AccountManager /org/freedesktop/Telepathy/Account/ring/tel/account0 org.freedesktop.Telepathy.Account.UpdateParameters '{'org.freedesktop.Telepathy.Connection.Interface.Anonymity.AnonymityMandatory': <false>}' '<'Ignore'>'01:34
Wizzup[ERR]: GDBus.Error:org.freedesktop.Telepathy.Error.InvalidArgument: Protocol 'tel' does not have parameter 'org.freedesktop.Telepathy.Connection.Interface.Anonymity.AnonymityMandatory'01:34
WizzupI think tp-ring just doesn't support it this way01:42
Wizzupinterestingly enough, even with AnonymityMandatory=true and AnonimityModes=1 (req anon), the vm calls still show my number01:45
Wizzupon my vm01:45
Wizzupmaybe my operator doesn't allow it?01:45
uvos__Wizzup: or its broken on that modem/ofono driver01:50
uvos__that would explain why no one compained about sphone hideing the number on pp01:51
uvos__since sphone used to ask ofono to hide the number because it was swaped on d401:51
Wizzupmaybe yeah01:52
Wizzupin any case I have a hacky patch for voicecall-manager01:52
Wizzuptested in my d4, to not hide caller id01:52
Wizzups/in my/on my/01:52
uvos__its a bit wierd that setting it to 0 gives you a hidden number on d401:53
Wizzup0 means nothing set, to do what modem or carrier says01:53
uvos__this is supposed to mean operator preferance, on my d4 this results in it shown01:53
Wizzup1 is request anon, 2 is do not request anon01:53
uvos__hidden operator preferance is wierd imo01:53
Wizzupbut I am talking about tp bits01:53
Wizzuphttps://telepathy.freedesktop.org/spec/Connection_Interface_Anonymity.html#Property:SupportedAnonymityModes01:53
Wizzupwell, at least this is solved I guess01:56
Wizzupuvos__: btw, I made some hacky changes to sphone to allow any text in the dialer text field so that I could dial sip numbers and xmpp accounts01:58
Wizzupfreemangordon: (re tracker) nothing in syslog01:59
Wizzuparno11: pls poke me tomorrow eve and we can make some more changes to the default images02:02
Wizzup 5.31 mW     53,1 ms/s      0,00        Process        [PID 6799] /usr/libexec/tracker-extract02:05
Wizzup 4.72 mW     47,2 ms/s      0,00        Timer          mix_interrupt_randomness02:05
Wizzup 2.15 mW     21,5 ms/s      0,00        Process        [PID 4369] /usr/libexec/tracker-miner-fs02:05
Wizzupgah02:05
Wizzupfreemangordon: the weird thing is that I didn't even add any music or really anything02:06
siceloWizzup: please have a look at https://github.com/maemo-leste-upstream-forks/ofono/pull/4 when you get time06:18
freemangordonWizzup: so, tracker lost its db again and wouldn't recover it?07:29
siceloWizzup: i found the fix for ofono.RadioSettings interface issue i mentioned yesterday. will send it upstream shortly08:27
arno11Wizzup: ok09:29
Wizzupfreemangordon: looks like it09:46
Wizzupsicelo: great, will do09:46
ac_laptophello people09:58
ac_laptopbooted leste today after a full charge, it didn't have time to display the battery icon but it had time to play the "battery low" sound before shutting down09:59
ac_laptopI stand by my point : leste is not aware at boot that the battery is full, it needs "something" to be aware of it. I just don't know what that "something" is exactly10:00
Wizzupare you still swapping batteries?10:00
ac_laptopNow I'd like to know what program/daemon is responsible for the "if battery low then shutdown behavior", and how to disable it.10:01
ac_laptopWizzup: I didn't swap batteries this time, I let the device plugged with the same battery10:01
ac_laptopNow I'd like to know what program/daemon is responsible for the "if battery low then shutdown behavior", and how to disable it.10:02
ac_laptop↑↑↑ ↑↑↑ ↑↑↑10:02
Wizzuplinux kernel10:02
Wizzupat least on various devices, since it reads it directly from the battery gauging chip10:02
Wizzupwhich is probably still a little confused10:03
Wizzupbut mce might act on low battery too and initiate shutdown10:03
ac_laptopIf the linux kernel was responsible for it the device would go down after the linux kernel is loade, it would not wait for Xorg to be up10:04
siceloWizzup: yes, i would be happy for a new release of ofono. that patch helps a lot for my operator10:09
ac_laptopok I've checked with an external charger, this time the battery is actually low10:10
ac_laptopthere is something wrong with the charging10:10
Wizzupsicelo: ok\10:11
ac_laptopcan someone tell me the characteristics of the original nokia n900 charger ? the output values10:15
siceloNokia AC-1010:16
siceloOutput: DC 5.0V / 1200 mA10:17
Wizzupac_laptop: be mindful it might also be your usb port slowly getting loose10:17
ac_laptopsicelo: thank you10:18
ac_laptopI've got DC5V, 1.3A10:18
sicelooriginal charger?10:18
Wizzupsicelo: do I write 'add operator parsing' ?10:19
sicelonote N900 will draw high current from charger *only* if the charger has its D- and D+ pins shorted (that was the USB standard back then)10:19
Wizzupsicelo: or shall I just take your commit msg for the changelog10:19
ac_laptopsicelo: I thought it was, but I'm not sure now10:19
Wizzupwill just do that10:20
siceloWizzup: you can, or maybe 'improve operator name parsing', since the commit adds a second way to get operator name10:20
Wizzupbuilding now10:20
ac_laptopsicelo: you mean on the USB cable micro-USB end ?10:22
sicelowherever :-)10:22
siceloas long as by the time it connects to N900, that short exists10:22
siceloanyway, charging on lower currents should still work10:23
sicelocan you describe properly/easily what you observe ...10:23
ac_laptopok well on the original cable (which I stopped using because I couldn't flash u-boot with it), I see the full 5 pins so I guess it's not an original one10:26
ac_laptopthe cable I've been using since then is a samsung cable from an other phone, it has 4 pins and D+ and D- are shorter indeed10:26
ac_laptopthe AC/USB adapter even though it has nokia written on it might not be an original nokia one10:27
sicelosorry if i was not clear. by 'short' ... i meant short-circuit10:28
ac_laptopsicelo: connected together ?10:30
siceloanyway, the thing about short-circuit only affects how fast the battery will charge. as already mentioned, at 500mA max, e.g. when connected to a laptop's USB port, charging should still work normally10:30
siceloconnected together, yes10:30
WizzupI have found n900 fremantle to be a bit more picky than n900 leste, there are a few chargers it just won't use10:30
Wizzupbut this is easily solved by us10:30
Wizzupusing another cable or charger10:30
Wizzupsicelo: should be built, ty for the pr10:30
sicelothanks!10:31
Wizzupwill be around later today again10:31
ac_laptopsicelo: ok well I guess I'll only charge on a laptop now, or use an external charger10:32
sicelocharge the battery from outside the N900?10:33
ac_laptopsicelo: yes, using something like https://www.ebay.com/itm/13466988272510:34
sicelook. the fuel gauge won't like it, but it's not a problem. you'll always see "Needs Calibration"10:36
ac_laptopso we're now left with the possibility that the usb cable gets loose as Wizzup mentioned10:43
ac_laptoptesting with an other battery while this one charges on the external charger, plugging the cable10:46
ac_laptopLED is green, wobbling the cable inside the microUSB10:47
ac_laptopno changes10:47
sicelothe thing is - none of us can know what's really the specific problem you have. we depend on the clarity of your explanation10:49
ac_laptopfremantle at least doesn't report any disconnection when I wobble the usb cable10:52
sicelo 10:59 < ac_laptop> booted leste today after a full charge, it didn't have time to display the battery icon but it had time to play the "battery low" sound before shutting down ....10:53
sicelo^^^ so, you're saying the battery was really low, right? so leste did the right thing to shutdown here?10:54
ac_laptopsicelo: well, yes, the battery was actually low, when I put it on the external charger to check it showed less than 25% (probably near 0%)10:55
ac_laptopso I'm trying to understand why the n900 didn't charge its battery during the night while plugged10:56
ac_laptopmaybe it's my AC adapter that is faulty, maybe I have a hardware issue on my n90010:58
siceloor things were simply not connected properly :-)10:58
ac_laptopbut this time the problem does not depend on the OS, since the phone was off when it was plugged10:59
siceloyes ... in fact we have mentioned a number of times that it's not Leste. even when the fuel gauge data is corrupt, voltage readout is correct, and that's what informs the shutdown decision in leste11:00
siceloFYI, my main Leste device has completely broken USB port, so I always charge from outside the device ... but Leste always behaves normally11:01
ac_laptopsicelo: well I definitely had a couple occurrences when both fremantle and the external charger reported a full battery when lested reported a low one so there is also an issue there11:02
ac_laptopbut I need to fix the hardware problem first11:03
ac_laptopI'll stop using the AC-USB adapter and see if I still have issues11:04
siceloi'll provide a kernel patch in the hopefully near-future to deal with some edge cases when fuel gauge contains corrupted or inaccurate data11:05
arno11sicelo: i checked again limits.conf stuff: rtprio option doesn't work without rt kernel apparently.12:02
arno11if you can have a look when you have time12:02
sicelooh that sucks12:02
arno11yep12:02
siceloi think in upcoming days things should be better on my end. done with the ofono work :-)12:03
siceloso yeah, i hope to look at it soon12:03
arno11ok and cool for ofono :)12:03
siceloactually there's one more ofono thing i'd like to do, but not right away - the way it toggles the modem gpios is deprecated in the kernel. so we need to migrate to the new way12:05
arno11ah12:06
siceloarno11: CONFIG_GPIO_SYSFS in kernel is deprecated, and currently ofono uses it :-p12:11
arno11that's sucks too12:53
arno11*that12:54
sicelokernel people actually said they'd remove that in 2020. we're lucky it's still there12:57
arno11indeed...12:59
uvos__"<sicelo> yes ... in fact we have mentioned a number of times that it's not Leste. even when the fuel gauge data is corrupt, voltage readout is correct, and that's what informs the shutdown decision in leste"13:28
uvos__thats may not be copleatly true13:28
uvos__mce shutsdown when the voltage is too low, or when upower reports that the battery is empty13:29
uvos__im not sure how upower makes its determination exactly but possibly it may report the battery to be empty if the fule guage is very confused13:30
siceloleste has a forked upower, which estimates capacity if fuel gauge is confused. the metric used for that is ... voltage13:31
uvos__what is ofono doing directly writeing to gpios, that sounds like a bad idea in the firstplace13:32
uvos__sicelo: ok13:32
sicelohttps://github.com/maemo-leste-upstream-forks/upower/commit/13edc4622d19056c934bab196661145de8cbce0113:33
uvos__sicelo: so the fule guge thinking it knows whats going on but is wrong is not possible?13:33
sicelowriting to gpios from ofono is absolutely normal. even pinephone uses something similar, but because they use MM, which can't handle gpios, they needed an additional daemon, eg25-manager or some such13:34
uvos__writeing gpios from usespace in liu of a kernel driver is generally fround uppon13:34
siceloyes, but there are cases where it's needed, hence libgpiod, which i'll implement for the n900's modem in ofono13:35
uvos__sure13:35
uvos__but whats it doing exactly13:35
uvos__why is it "needed"13:36
sicelothe modem is like a completely separate phone. you 'press' buttons to get it to a certain state, e.g. go online13:36
uvos__that still very mutch sounds like a kernel kernel module should be doing this13:37
uvos__like on d4 for instance13:37
uvos__so with the fule guage: if somehow the fule guage thinks its callibrated, but is wrong a early shutdown should result due to upower reporting empty13:38
sicelomaybe ... although i would ask why the large community around pinephone didn't do it that way13:38
uvos__sicelo: i mean thair kernel isent exatcly great13:39
uvos__look at all the patches13:39
siceloyes, but not all areas13:39
siceloactually EG-25 is a generic modem, even Wizzup has it on his thinkpad, i think13:39
uvos__he dose13:40
siceloregarding fuel gauge, thankfully, it never thinks it's calibrated when it's not.13:40
siceloplus, as i already mentioned, my N900 has such 'corrupted' fuel gauge because i always charge externally. the upower patches keep leste happy13:41
siceloin pmos, where forking upower was not accepted, i carry a kernel patch13:41
uvos__sicelo: maybe, im just wondering if there is some path to bug it out, via switching battiers or so13:41
uvos__i dont know how the guage knows its not callibrated so cant comment if its somehow possible13:42
uvos__was just mentioning that there is a second potential path to sutdown that dosent include voltage13:42
siceloi always switch batteries :-)13:42
siceloto rephrase ... yes upower will normally force shutdown when it thinks battery is empty. leste's upower fork invalidates that 2nd path though, by making its own estimate, based on voltage. so essentially, in leste there's just the voltage path13:45
uvos__if the kernel knows not to report that the percentage/ capacity when its wrong in all cases, and cant be triced by any means13:48
uvos__*tricked13:48
uvos__i presume this is true13:48
sicelorephrase?13:48
uvos__if the kernel reports capactiy = 1 when its wrong, by some means, mce will shut down. if there is some way to trick it to think this even though the battery is full then a premature shutdown will result13:50
uvos__this presumably is not possible, but im saying a potential bug in the hw or kernel could do this13:50
siceloyes that's true. however, mce in leste does not ask kernel for capacity, right?13:51
uvos__no but it shuts down on crictial level which iiuc is based on some theshold of capcaity13:52
sicelowho tells it the capacity? upower, right?13:53
uvos__yes from upower, but its estimation code wont be used if the kernel reports capactiy = 1 so it will essentally path it though in this case13:54
sicelook, yes you're right. in the case of N900, kernel is guaranteed to report 0% for the case of corrupted data13:55
uvos__yes but how dose the kernel know that the data is corrupted, if the stars aling poorly maybe just maybe it will get wrong valid data13:56
uvos__i dont think this is likely13:56
sicelothe hw knows :-)13:56
uvos__but im saying it may be possible13:56
siceloand it tells kernel13:56
sicelojust to be clear, kernel won't set the 0% itself. the HW will, so kernel simply reports what it's told14:01
siceloi had a long discussion with the upower people, where i was trying to get upower to factor in other parameters before deciding to power off (CAPACITY_LEVEL to be specific). unfortunately we didn't reach agreement, hence the kernel hack14:02
sicelohttps://gitlab.com/postmarketOS/pmaports/-/blob/master/device/community/linux-postmarketos-omap/0012-bq27xxx_battery-add-voltage-based-capacity-estimatio.patch14:03
uvos__sicelo: ok14:03
uvos__sicelo: so not barring hw failure14:03
siceloyeah, if hw has failed, then all hell can break loose :-)14:04
uvos__yes14:04
siceloactually what i want to do for N900 in leste is ... to stop using the minimal voltage set via mce.ini, and go back to using percentage as estimated by upower. for those with calibrated batteries, we'll magically detect that minimal voltage (3248mV) by using the appropriate HW flag (mapped to sysfs CAPACITY_LEVEL), and clamp reported capacity to something really low, e.g. 1%, which will cause14:20
siceloupower/mce to shutdown14:20
uvos__sicelo: the problem with that is that the charge estimation in upower is _terrible-14:21
uvos___14:21
uvos__so it will result in the device staying on too long14:22
uvos__when uncallibrated14:22
siceloit won't ... the hw-based CAPACITY_LEVEL will still be hit, and having clamped the capacity to 1%, things look 'normal' to mce/upower14:24
uvos__un uncallibrated mode capacity isent reported at all14:24
uvos__iiuc14:24
siceloit reports 0%14:25
uvos__right and that cause upower to estmate on voltage14:25
uvos__wich its terrible at14:25
uvos__so may be wrong up to 10%14:25
uvos__this forces mce to act on voltage14:25
siceloyup,14:26
uvos__so we need to fix estmation, but thats a known problem14:27
uvos__then we maybe can get rid of the voltage threshold14:27
uvos__pending on how reliable our estmation is then14:27
siceloregardless of how wrong upower's estimation will be, as long as the hw threshold hasn't been hit, you're sure you're ok. when it gets hit, kernel will know, independently of upower being correct or wrong :-)14:28
uvos__sure but mce dosent know that14:28
siceloat that point, we report 1%, and upower stops estimating14:28
uvos__and has no way of knowing via upower afaik14:28
siceloanyway, what i'm talking about is something i'm already using for a couple of months (under pmos) ... works fine with upstream upower. i just need to prepare one more kernel patch for an edge case, then i'm ready to submit it to leste14:30
uvos__great14:30
uvos__but we then have the problem14:31
uvos__that we cant remove the estmation14:31
uvos__because its needed on other devices14:31
siceloyeah14:31
uvos__so we need some way to enable it selectively14:31
uvos__gets messy14:31
siceloit's totally fine ... the estimation selects itself based on seeing a 0% condition. the rest of the time, all the devices aren't estimating, so we're good overall14:32
siceloi actually wonder if we shouldn't upstream some of spinal's patches to upower14:33
arno11sicelo: got voicecalls working at 100% i think. The key is to use high RR sched with pulseaudio. it works fine with chrt but can't find another way to do that.15:34
arno11no other priority tweaks are needed anymore excepting cmt_pulse nice/renice15:35
arno11no bug (but i'm using 16bpp and custom transitions)15:36
arno11i'll try with a fresh install and see...15:37
siceloeven if the limits.conf isn't working, i think we can still use chrt/renice in a startup script of some sort, while keeping cmt_pulse running as user15:42
arno11yes i think so15:42
siceloWizzup: or someone else, btw why do we have pulse running in system mode? not pushing for a change per se, but just to understand16:15
uvos__gets messy16:53
uvos__sicelo: we used to do this because we had no session wich was required16:55
uvos__sicelo: it should be possible to run pa in session mode now16:55
uvos__sicelo: this would improve performance because it would use shm then16:55
uvos__sicelo: the estmation code needs to go long term16:55
uvos__sicelo: its not generic at all, a nimh or li-fe battery would for instance be estimated totaly wrong16:56
uvos__there is no way any of that can be upstreamed16:56
uvos__so it gets messy since good estimation will be device dependant16:57
uvos__never mind that it dosent work right even for the li-nmc cells we have now16:58
freemangordonoh, yes, pa should go to login session17:06
freemangordonwe totally forgot that17:06
sicelouvos__: yes the estimation stuff is not upstreamable. but there's, for example, a patch to prefer hw based time to full/empty17:15
sicelocool @ pa17:16
Wizzupsorry, I will be back later17:16
arno11cool for PA session mode, could help for N900 cpu usage17:55
arno11freemangordon: i noticed that composition seems not disabled when an app is running in fullscreen (example: smplayer is laggy with no overclock and 360p video in fullscreen. Ctl+Shift+N solves the issue...same with pcsx)18:41
arno11it's particulary visible in 16bpp18:43
uvos__arno11: yes this is a know issue20:49
uvos__arno11: hildon-desktop is silly, it needs a special atom to tell it to go into non-composed mode20:50
uvos__arno11: all other wms just unredirect when fullscreen20:50
arno11ah ok, really good to know20:52
uvos__https://github.com/maemo-leste/hildon-desktop/blob/6324ad83ad5c28f3114595756a52835a76cb6310/src/mb/hd-comp-mgr.c#L172620:57
uvos__we should just remove this and base it on it being fullscreen only imo20:57
uvos__the only downside is a tiny delay on switch or when a popup notifcation comes up20:57
uvos__but it benefits everything essentally20:58
uvos__fullscreen applications are most likely to be heavy and or of a nature that the user dosent constanly switch away20:58
uvos__this is why all desktop wms disable in this state20:58
arno11indeed makes sense21:01
* Wizzup catching up23:08
Wizzupsounds fine to me @ pa in user23:17

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