Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-16 Thread John Crispin
Hi, still pending not had time to look into this. it'll take me a moment as i am not familiar with the code so i need to find a free moment to properly review this, sorry for the delay John On 16/12/2016 08:51, Nikolay Ledovskikh wrote: > So. What about patch? 'sync' call allows us not t

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-15 Thread Nikolay Ledovskikh
So. What about patch? 'sync' call allows us not to save cids in interface state. And every time interface is up, it will be prepared correctly. 2016-12-12 12:28 GMT+03:00 Bjørn Mork : > Matti Laakso writes: >> On 12.12.2016 08:52, Petr Štetiar wrote: >>> Matti Laakso [2016-12-09 11:35:35]: >>> >

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-12 Thread Bjørn Mork
Matti Laakso writes: > On 12.12.2016 08:52, Petr Štetiar wrote: >> Matti Laakso [2016-12-09 11:35:35]: >> >>> On 09.12.2016 11:27, Petr Štetiar wrote: I knew, that there's only one profile defined, operator has deleted all other profiles, there's currently only one APN defined. >>

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-12 Thread Matti Laakso
On 12.12.2016 08:52, Petr Štetiar wrote: Matti Laakso [2016-12-09 11:35:35]: On 09.12.2016 11:27, Petr Štetiar wrote: I knew, that there's only one profile defined, operator has deleted all other profiles, there's currently only one APN defined. AT+CGDCONT? +CGDCONT: 1,"IPV4V6","","0.0

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-11 Thread Petr Štetiar
Matti Laakso [2016-12-09 11:35:35]: > On 09.12.2016 11:27, Petr Štetiar wrote: >> I knew, that there's only one profile defined, operator has deleted all other >> profiles, there's currently only one APN defined. >> >> AT+CGDCONT? >> +CGDCONT: 1,"IPV4V6","","0.0.0.0",0,0 > > Well, there's no

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Nikolay Ledovskikh
OK. It seems I missed something when doing cleanup. Your patch is working, but only after startup. When you disconnect modem from usb we have old cids saved. Then we connect modems back, modems won't bring up automatically (and I suppose they souldn't). And when we try to bring interface up we are

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Nikolay Ledovskikh
> Are all modems the same? Do they have same firmware versions? Are you sure > that the mapping of different /dev/cdc-wdmX nodes always stay the same? It > could be that on some boot the modem with the Megafon SIM gets /dev/cdc-wdm0 > and on the second boot it gets /dev/cdc-wdm1 or some other numbe

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Matti Laakso
Hi Nikolay, On 09.12.2016 11:57, Nikolay Ledovskikh wrote: I'm using multiple modems config interface 'wan1' option proto 'qmi' option device '/dev/cdc-wdm0' option apn 'internet.megafon.ru' option ipv6 '0' option metric '1' config interface 'wan2'

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Nikolay Ledovskikh
We are using sierra MC7710, MC7304. 2016-12-09 14:41 GMT+03:00 Nikolay Ledovskikh : >> I agree fully. That's why I added the autoconnect option in my last patch >> and defaulted it to disabled. >> >> Matti > > So, what we get: uqmi exit with 'Call failed'. > > -- > Best regards, Nikolay Ledovskikh

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Nikolay Ledovskikh
> I agree fully. That's why I added the autoconnect option in my last patch > and defaulted it to disabled. > > Matti So, what we get: uqmi exit with 'Call failed'. -- Best regards, Nikolay Ledovskikh. ___ Lede-dev mailing list Lede-dev@lists.infradea

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Nikolay Ledovskikh
I'm using multiple modems config interface 'wan1' option proto 'qmi' option device '/dev/cdc-wdm0' option apn 'internet.megafon.ru' option ipv6 '0' option metric '1' config interface 'wan2' option proto 'qmi' option device '/dev/cdc-wdm1'

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Matti Laakso
On 09.12.2016 11:27, Petr Štetiar wrote: Matti Laakso [2016-12-09 10:23:24]: I don't think that the autoconnect setting is "transferred" to the network, rather it could be tied to a specific call profile index, which has the wrong APN. If your Slovakian SIM card has several predefined profiles

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Petr Štetiar
Matti Laakso [2016-12-09 10:23:24]: > I don't think that the autoconnect setting is "transferred" to the > network, rather it could be tied to a specific call profile index, which > has the wrong APN. If your Slovakian SIM card has several predefined > profiles, this could be the case. If y

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-09 Thread Matti Laakso
On 08.12.2016 17:11, Bjørn Mork wrote: Petr Štetiar writes: Matti Laakso [2016-12-08 15:39:57]: Hi, I don't like the autoconnect at all, it being so unpredictable across various models. This autoconnect feature should burn in hell :-) The funny thing about this feature is, that the behav

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Petr Štetiar
Bjørn Mork [2016-12-08 16:11:18]: Hi, > I don't think we can trust the modem firmware with complex features like > autoconnect. There are too many things that can go wrong. And we don't have > any way to debug or fix any of those issues when they.happen inside the > modem firmware "black box".

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Bjørn Mork
Petr Štetiar writes: > Matti Laakso [2016-12-08 15:39:57]: > > Hi, > >> I don't like the autoconnect at all, it being so unpredictable across >> various models. > > This autoconnect feature should burn in hell :-) The funny thing about this > feature is, that the behaviour is network specific[1]

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Nikolay Ledovskikh
> This autoconnect feature should burn in hell :-) The funny thing about this > feature is, that the behaviour is network specific[1] also. Maybe. I'm not too familiar with qmi proto. I just catched this behaviour in our test environment and found pdf that says: "Gobi3000 devices, though, have the

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Petr Štetiar
Matti Laakso [2016-12-08 15:39:57]: Hi, > I don't like the autoconnect at all, it being so unpredictable across > various models. This autoconnect feature should burn in hell :-) The funny thing about this feature is, that the behaviour is network specific[1] also. I still can't find some time

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Nikolay Ledovskikh
No. All this is not working if cid is not released for some reason. For example: we have multiple modems with external power source, if usb DATA cable connection is broken for some reason, the cid will not be released and new network connection will not be established until you release the cid manu

Re: [LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-08 Thread Matti Laakso
> > Add uqmi 'sync' command call to release stalled cid when preparing to > setup new connection. As a result it prevents 'POLICY MISMATCH' errors. > > Signed-off-by: Nickolay Ledovskikh

[LEDE-DEV] [LEDE-DEV, 3/3, v3] uqmi: Prevent 'POLICY MISMATCH' error.

2016-12-07 Thread Nickolay Ledovskikh
Add uqmi 'sync' command call to release stalled cid when preparing to setup new connection. As a result it prevents 'POLICY MISMATCH' errors. Signed-off-by: Nickolay Ledovskikh --- package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/packag