[PATCH] bearer-qmi: listen for Packet Service Status indication and handle disconnection

[Aleksander Morgado]

On Fri, Apr 15, 2016 at 5:28 PM, Diego López <dlopezm at teltronic.es> wrote:
> It look's good, I applied the patch over currently HEAD (commit:
> 650f4a2e911e5d626e1c9d365437c7e1b2191fcc). I've tested it with IPv4:
>
> ModemManager[2115]: [/dev/cdc-wdm1] Received message (translated)...
>>>>>>> QMUX:
>>>>>>>   length  = 33
>>>>>>>   flags   = 0x80
>>>>>>>   service = "wds"
>>>>>>>   client  = 8
>>>>>>> QMI:
>>>>>>>   flags       = "indication"
>>>>>>>   transaction = 0
>>>>>>>   tlv_length  = 21
>>>>>>>   message     = "Packet Service Status" (0x0022)
>>>>>>> TLV:
>>>>>>>   type       = "Connection Status" (0x01)
>>>>>>>   length     = 2
>>>>>>>   value      = 01:00
>>>>>>>   translated = [ status = 'disconnected' reconfiguration_required =
>>>>>>> 'no' ]
>>>>>>> TLV:
>>>>>>>   type       = "Call End Reason" (0x10)
>>>>>>>   length     = 2
>>>>>>>   value      = 01:00
>>>>>>>   translated = generic-unspecified
>>>>>>> TLV:
>>>>>>>   type       = "Verbose Call End Reason" (0x11)
>>>>>>>   length     = 4
>>>>>>>   value      = 03:00:C4:09
>>>>>>>   translated = [ type = 'cm' reason = '2500' ]
>>>>>>> TLV:
>>>>>>>   type       = "IP Family" (0x12)
>>>>>>>   length     = 1
>>>>>>>   value      = 04
>>>>>>>   translated = ipv4
> ModemManager[2115]: <info>  [1460731761.537849] [mm-bearer-qmi.c:801]
> packet_service_status_indication_cb(): bearer call end reason (1):
> 'generic-unspecified'
> ModemManager[2115]: <info>  [1460731761.537872] [mm-bearer-qmi.c:812]
> packet_service_status_indication_cb(): bearer verbose call end reason
> (3,2500): [cm] (null)
> ModemManager[2115]: <debug> [1460731761.537892] [mm-port.c:95]
> mm_port_set_connected(): (rename4): port now disconnected
> ModemManager[2115]: <info>  [1460731761.537919] [mm-iface-modem.c:1431]
> __iface_modem_update_state_internal(): Modem
> /org/freedesktop/ModemManager1/Modem/0: state changed (connected ->
> registered)

Superb, thanks for testing.


-- 
Aleksander
https://aleksander.es