Bug 87635 - Nokia N9 via Bluetooth DUN fails in AT+CSCS=?
Summary: Nokia N9 via Bluetooth DUN fails in AT+CSCS=?
Status: RESOLVED FIXED
Alias: None
Product: ModemManager
Classification: Unclassified
Component: plugins (show other bugs)
Version: git master
Hardware: Other All
: medium normal
Assignee: ModemManager bug user
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-23 11:21 UTC by Aleksander Morgado
Modified: 2015-01-16 16:12 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Patch (2.72 KB, patch)
2014-12-25 11:53 UTC, Aleksander Morgado
Details | Splinter Review

Description Aleksander Morgado 2014-12-23 11:21:47 UTC
AT+CSCS=? sent to the N9 via Bluetooth DUN times out and completely breaks the connection attempt; the default 3s timeout isn't enough. A test with a 20s timeout worked properly, though, so we should try to update that in the Nokia plugin.

See NM bug for more info:
https://bugzilla.gnome.org/show_bug.cgi?id=741813
Comment 1 Aleksander Morgado 2014-12-25 11:53:30 UTC
Created attachment 111322 [details] [review]
Patch
Comment 2 Weng Xuetian 2014-12-26 06:54:24 UTC
Hi, I just did a more careful check.

Seems it's not this patch fixed my problem, but some other commit betwen master and 1.4.0.

So I wonder if this patch is not necessary anymore.

commit b8cd5ae838b1a17ce1bb6f01e6491c7b887e8474
Author: Lubomir Rintel <lkundrak@v3.sk>
Date:   Wed Oct 1 12:02:36 2014 +0200

    device: Keep track of devices as they are move across sysfs
    
    For certain devices the name changes with their status. Notably, RFCOMM
    devices move from /devices/virtual/ to underneath the HCI that is used
    for the connection as the session is estabilished, and return back when
    it's torn down.
Comment 3 Weng Xuetian 2014-12-26 06:55:04 UTC
The commit above is found by git bisect.
Comment 4 Aleksander Morgado 2014-12-26 08:59:44 UTC
(In reply to Weng Xuetian from comment #3)
> The commit above is found by git bisect.

Ah, good to know. I'll cherry pick that patch for mm-1-4; thanks.
Comment 5 Aleksander Morgado 2014-12-26 09:05:07 UTC
(In reply to Weng Xuetian from comment #2)
> Hi, I just did a more careful check.
> 
> Seems it's not this patch fixed my problem, but some other commit betwen
> master and 1.4.0.
> 
> So I wonder if this patch is not necessary anymore.

The AT+CSCS issue happened at least once. I checked the last logs you sent me (when you tried the change to 20s) and the AT+CSCS issue didn't happen there. So don't know, if this patch works I'd merge it anyway, as it should fix a real issue, even if it wasn't the one breaking your connection attempts all the time. Can you send me anyway logs of the connection attempt with this patch on, to see if it works?
Comment 6 Aleksander Morgado 2015-01-16 16:12:01 UTC
Patch was merged to git master.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.