| nightishaman | mal: what do you think | 00:09 |
|---|---|---|
| nightishaman | im gonna go to sleep but we probably have found the problem | 00:23 |
| mal | nightishaman: https://xairy.io/articles/pixel-kgdb#-apc-watchdog | 00:43 |
| T42 | <nightishaman> But I didn’t even attach a debugger, wtf | 00:45 |
| T42 | <nightishaman> Well, I’m probably just gonna disable the kernel modules in gs201_defconfig | 00:45 |
| T42 | <nightishaman> We don’t need the whole debug infrastructure in place | 00:45 |
| T42 | <nightishaman> Also im on telegram since I don’t have irc on my daily driver phone | 00:46 |
| mal | you could try that kernel command line option | 00:46 |
| mal | that is the quickest thing to try | 00:47 |
| mal | you can add to suitable place in device repos | 00:48 |
| mal | some place there defines the kernel command line | 00:49 |
| mal | in BoardConfig.mk or similar file | 00:49 |
| T42 | <nightishaman> I know, it’s in BoardConfig-common.mk for gs201 | 00:49 |
| mal | anyway, time for some sleep | 00:49 |
| T42 | <nightishaman> Although do we really need the debug infrastructure? I don’t see the point in keeping those modules | 00:50 |
| mal | let's not do too many changes at once | 00:51 |
| T42 | <nightishaman> okay, but if it is crashing again tomorrow I’m gonna remove them | 00:54 |
| *** xmn_ is now known as xmn | 13:25 | |
| nightishaman | mal: it reboots, so apparently the kernel command line args work | 20:04 |
| mal | nightishaman: how soon does it reboot? | 20:09 |
| nightishaman | after about 3s | 20:09 |
| mal | you have some reboot command in the init-script now? | 20:09 |
| nightishaman | yeah, after 25 modules it reboots | 20:10 |
| nightishaman | i increase now to 50 | 20:10 |
| nightishaman | with 50 it did reboot now one time with BL1 requested, the next time it hangs. | 20:14 |
| nightishaman | ah now after a minute it rebooted, also with bl1 requested | 20:15 |
| nightishaman | according to bootloader it was the apc watchdog again | 20:16 |
| mal | which partitions did you flash? | 20:17 |
| mal | check using that unpack_bootimg.py which one contains the parameter, use --format mkbootimg if other you can't see the command line | 20:18 |
| nightishaman | i flashed vendor_boot and init_boot | 20:20 |
| nightishaman | the command line is set in vendor_boot | 20:21 |
| nightishaman | with 25 it did work, while yesterday it didnt, so i guess its another one of the debug kernel modules | 20:24 |
| nightishaman | i think the best way forward would be to remove these | 20:24 |
| T42 | <b100dian> rinigus: sorry I missed your silent ping again. Android shows something like "Add plan" in settings for devices with eSIM. From there there's a app by google that is launched.. | 20:56 |
| T42 | <b100dian> mal: still debugging video playback, but found an interesting thing: waydroid on sake did not play back until I didn't cherry-pick this https://github.com/waydroid/waydroid/issues/1813 | 20:57 |
| T42 | <b100dian> Some forum people say playback with appsupport is impacted on nagara, maybe appsupport needs those nodes too. | 20:58 |
| nightishaman | mal: what do you think? | 20:59 |
| nightishaman | unfortunately, some other modules depend on some of the debug modules, so i cant just remove all of them directly | 21:14 |
| mal | @b100dian interesting, need to check | 21:31 |
| mal | nightishaman: removing those would require doing those from defconfig and then rebuilding boot images | 21:31 |
| mal | but still very odd why some modules would breaking it like that | 21:32 |
| mal | nightishaman: did the commandline change work or not? if I understood what you wrote it didn't, not sure what that bl1 you said means | 21:33 |
| nightishaman | mal: the change did work, it loaded the first 25 modules but increasing to 50 made it hang again, so something else hangs | 21:40 |
| nightishaman | bl1 means bootloader requested by software | 21:40 |
| nightishaman | claude suggested just using a skip list in the init script first, since exynos coresight isnt a dependency of another module | 22:18 |
| T42 | <b100dian> rinigus: forgot to respond - I don't have a lpac build, but pretty sure this would work on OBS too https://github.com/juanro49/harbour-lpac. | 22:34 |
| T42 | <b100dian> SimPro I didn't find the sources. But `LPAC_APDU_GBINDER_UIM_SLOT=2 LPAC_APDU=gbinder lpac profile download` should work just the same | 22:35 |
| mal | @b100dian about that droidmedia PR, does that std::max(4, android::BufferQueue::NUM_BUFFER_SLOTS / 2); make sense? isn't that always android::BufferQueue::NUM_BUFFER_SLOTS / 2 | 23:18 |
| T42 | <b100dian> Probably, I dont have all your devices, maybe there are devces with less slots | 23:19 |
| T42 | <b100dian> We could just use the MAX value in fact | 23:20 |
| mal | but that is constant in android header | 23:20 |
| mal | and what I could find NUM_BUFFER_SLOTS is 64 | 23:22 |
| T42 | <b100dian> MAX is NUM_BUFFER_SLOTS - 2 | 23:23 |
| nightishaman | so with skipping exynos-coresight-etm I got to 30, at 50 i got 2 hangs in 9 reboots, so 7 times it rebooted immediately fine | 23:24 |
| T42 | <b100dian> mal: heading to bed, was gonna mention the PR tomorrow. I would be interested if fp5 with media buffers works too | 23:25 |
| mal | @b100dian yes but your code would always try to first set 32 and then 4 if the first one failed | 23:25 |
| mal | sure, I can test | 23:25 |
| T42 | <b100dian> I was being conservative | 23:26 |
| mal | in the change I did earlier I originally planned 16 but then after some testing with a device with different issue 4 was enough there, didn't think it would affect fp5 also | 23:27 |
| T42 | <b100dian> Lets see | 23:29 |
| nightishaman | this is very annoying to debug | 23:39 |
| nightishaman | i purchased a usb cereal but its only coming by end of month | 23:39 |
| mal | nightishaman: have you found which module hangs it this time? | 23:44 |
| nightishaman | no idea, i found out i can enable some kind of ramdump with fastboot oem ramdump enable | 23:45 |
| nightishaman | but no idea where to read it out | 23:45 |
| nightishaman | ok i just had a black screen with red title saying "Preparing for ramdump" | 23:45 |
| nightishaman | i can pull kernel log from ramdump | 23:51 |
| T42 | <Mister_Magister> ye just run strings on it | 23:59 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!