<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 14, 2014 at 4:37 PM, Ben Chan <span dir="ltr"><<a href="mailto:benchan@chromium.org" target="_blank">benchan@chromium.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On Wed, May 14, 2014 at 12:08 PM, Dan Williams <<a href="mailto:dcbw@redhat.com">dcbw@redhat.com</a>> wrote:<br>


><br>
> Looks like the right approach.  The only thing I'd worry about is<br>
> whether we reliably get notifications for packet service at the right<br>
> times?  Should the patch be more aggressive about clearing<br>
> self->priv->HADC to ensure it isn't stale?  For example, perhaps the<br>
> device doesn't send a Packet Service "off" notification after being<br>
> explicitly disconnected because the firmware is stupid and assumes that<br>
> since the host asked for disconnection, it knows to clean up the HADC.<br>
><br>
<br>
</div>This seems tricky. The modem can still attach to a packet service<br>
without activating an IP session. ModemManager currently doesn't<br>
explicitly detach the packet service upon a client / network initiated<br>
disconnect. So any detach action should come from the modem<br>
internally, which means we would expect the modem to "Do The Right<br>
Thing (tm)" and notify us via a MBIM_CID_PACKET_SERVICE notification.<br>
</blockquote></div><br></div><div class="gmail_extra">Dan / Aleksander, do you have any other concerns on this patch at its current form?</div></div>