<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sun, Dec 22, 2024 at 6:37 PM Aleksander Morgado <<a href="mailto:aleksander.morgado@gmail.com">aleksander.morgado@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hey,<br></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> > After updating to the latest main (at eda130f3bf1c), in our team<br>
> > we've noticed, that in our CI, that QMI port sometimes fails to<br>
> > probe,<br>
> > with Telit LE910C4-WWXD. The result is the modem having only AT<br>
> > ports<br>
> > available, thus falling back to using PPP for the data connection.<br>
> <br>
> Well that's not awesome; I'll dig into it. If you figure out anything<br>
> too, let me know.<br>
> <br>
<br>
When this happens, does MM attempt to probe the port as AT before<br>
trying QMI?<br>
<br></blockquote><div><br></div><div>With the changes in that referenced commit, it looks like serial_probe_step() may be running serial_open_at() also for the QMI and MBIM ports, which is not right, as that would create a new MMSerialPort in ctx->serial.</div><div><br></div><div>The serial_probe_step() (ideally renamed as probe_step() to make it clearer that it is not serial-specific) should not run any serial-specific operation (e.g. CUSTOM_INIT, AT_OPEN_PORT, ....) for ports that do not require any AT or QCDM probing.<br></div><div><br></div><div>WDYT?<br></div></div></div></blockquote><div><br></div><div>The issue can be reproduced with modems for which the corresponding plugin has "custom_init" configured, e.g. Telit or Cinterion:</div><div><br></div><div>ModemManager[4112]: <dbg> [1734889629.161510] [cdc-wdm0/probe] no AT/QCDM/QMI probing in possible MBIM port<br>ModemManager[4112]: <dbg> [1734889629.162057] [cdc-wdm0/probe] port is not AT-capable<br>ModemManager[4112]: <dbg> [1734889629.162867] [cdc-wdm0/probe] port is not QCDM-capable<br>ModemManager[4112]: <dbg> [1734889629.163621] [cdc-wdm0/probe] port is not QMI-capable<br>ModemManager[4112]: <dbg> [1734889629.164981] [cdc-wdm0/probe] launching port probing: 'mbim'<br>ModemManager[4112]: <msg> [1734889629.166516] [cdc-wdm0/probe] probe step: start<br>ModemManager[4112]: <msg> [1734889629.166988] [cdc-wdm0/probe] probe step: custom init open port <br></div></div></div>