<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">> ​If
 removed from here (which I agree we should) we also need to clear the 
context when the modem goes away, e.g. in finalize(), or we would be 
leaking the context and the port references. Could you also add that in 
the patch?</div><div class="gmail_extra"><br><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">​I've missed that, sure I'll do it​</div><br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On 29 August 2017 at 07:34, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On 28/08/17 14:18, Carlo Lobrano wrote:<br>
> Currently when the modem is disabled, it also releases SIM hot swap ports context,<br>
> and it is not able to receive any notification about the SIM status anymore.<br>
><br>
> This patch keeps these ports opened when Modem is disabled and released them<br>
> only when SIM swap is detected.<br>
><br>
> ---<br>
><br>
> Hi Aleksander,<br>
><br>
> as you might remember we talked about this issue while discussing "Handling QSS unsolicited in power state transitions" (it's the email's subject).<br>
> I'm back working on this topic and saw that this part hasn't been submitted yet.<br>
><br>
<br>
</span><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">​​</div>If removed from here (which I agree we should) we also need to clear the context when the modem goes away, e.g. in finalize(), or we would be leaking the context and the port references. Could you also add that in the patch?<br>
<span class="gmail-"><br>
> ---<br>
><br>
>  src/mm-broadband-modem.c | 4 ----<br>
>  1 file changed, 4 deletions(-)<br>
><br>
> diff --git a/src/mm-broadband-modem.c b/src/mm-broadband-modem.c<br>
> index 552bdc5..b482705 100644<br>
> --- a/src/mm-broadband-modem.c<br>
> +++ b/src/mm-broadband-modem.c<br>
> @@ -8640,10 +8640,6 @@ disabling_stopped (MMBroadbandModem *self,<br>
>          self->priv->enabled_ports_ctx = NULL;<br>
>      }<br>
><br>
> -    if (self->priv->sim_hot_swap_<wbr>ports_ctx) {<br>
> -        ports_context_unref (self->priv->sim_hot_swap_<wbr>ports_ctx);<br>
> -        self->priv->sim_hot_swap_<wbr>ports_ctx = NULL;<br>
> -    }<br>
>      return TRUE;<br>
>  }<br>
><br>
><br>
<br>
<br>
--<br>
</span>Aleksander<br>
<a href="https://aleksander.es" rel="noreferrer" target="_blank">https://aleksander.es</a><br>
</blockquote></div><br></div></div>