On Mon, 14 Sep 2020 21:16:40 +0200 Denis 'GNUtoo' Carikli <[email protected]> wrote:
> This is probably to handle modem crashes. If the modem crash, we > probably need to load its firmware again. Else it could be annoying as > you would not know that the modem crashed and assume that your phone > is working fine and miss SMS, calls and other notifications. > > On the kernel side, you seem to have a watchdog timer and its > associated interrupt (irq_cp_wdt) for the modem. So if it crash, I > would guess that this should trigger somehow. I need to look on how we > do it for the other devices we support. > > In any case we just need to make sure that the boot sequence is tried > again, so we might not need to keep the device nodes open. I couldn't find the relevant code, so I asked Paul Kocialkowski and I got the following response: > <@paulk-leonov> GNUtoo: no I'm pretty sure it's detected > <@paulk-leonov> IIRC we got SIGHUP on the fd or something like that > <@paulk-leonov> > https://git.replicant.us/replicant/hardware_replicant_libsamsung-ril/tree/client.c#n146 > <@paulk-leonov> well it's basically as soon as something goes wrong So libsamsung-ril (not libsamsung-ipc) handles that by restarting the modem if the .poll() fails. Following the code leads to /dev/umts_ipc0 which is handled in the kernel. In the case of the other supported devices we probably know it works because it already happened. Nevertheless I'll continue tracking that later on. Denis.
pgpNDdEiuy0vp.pgp
Description: OpenPGP digital signature
_______________________________________________ Replicant mailing list [email protected] https://lists.osuosl.org/mailman/listinfo/replicant
