ModemManager uses a wrong plugin for Dell DW5818/5819

Bug #1735134 reported by Robert Liu
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OEM Priority Project
Fix Released
Critical
Unassigned
modemmanager (Ubuntu)
Fix Released
Critical
Unassigned

Bug Description

[Impact]

* Dell Wireless DW5818/5819 modems showed an incorrect signal strength and were using a ttyUSB* port for data connections instead of the MBIM device (which provides better performance).

Since linux-4.4.0-98, the kernel additionally loads gcserial driver for Dell Wireless DW5818/5819. The reason behind it is to support firmware switching and upgrading. However, the change makes ModemManager use Gobi plugin for this two modules. With Gobi plugin, the modules could establish data links, but it failed to retrieve the signal state. And it caused the mmcli and nm-applet giving wrong signal strength. The modules support the MBIM protocol, so ModemManager should use Dell plugin for these two modules.

I have worked out a patch to forbid these two modules in Gobi plugin, and it does work well.

[Test Case]

Current MM:
* Create connection with
$ nmcli c add type gsm ifname ttyUSB2 con-name gsmconn apn <operator APN>
* Without the patched package, mmcli shows, with an active connection (see comment #2):
    primary port: 'ttyUSB2'
    signal quality: '0' (recent)

Patched MM:
* Create connection with
$ nmcli c add type gsm ifname cdc-wdm0 con-name gsmconn apn <operator APN>
* With the patched package, mmcli shows, with an active connection (see comment #7):
    primary port: 'cdc-wdm0'
    signal quality: '38' (cached)

[Regression Potential]

The patch simply adds the Sierra modems VID/PIDs to the list of forbidden ids in the Gobi plugin, so the possibility of a regression is very small: only products with said VID/PID will be affected.

Tags: patch
Revision history for this message
Robert Liu (robertliu) wrote :

Current version, registered, but not connected
output of mmcli -m 0

Revision history for this message
Robert Liu (robertliu) wrote :

Current version, connected
output of mmcli -m 0
signal strength is 0

Revision history for this message
Robert Liu (robertliu) wrote :

Current version
log of ModemManager

Revision history for this message
Robert Liu (robertliu) wrote :

Patch to forbid the Dell Wireless DW5818/5819 modules

Revision history for this message
Robert Liu (robertliu) wrote :

debdiff to forbid the Dell Wireless DW5818/5819 modules

Revision history for this message
Robert Liu (robertliu) wrote :

Patched version, registered, but not connected
output of mmcli -m 0

Revision history for this message
Robert Liu (robertliu) wrote :

Patched version, connected
output of mmcli -m 0
Signal strength is not 0

Revision history for this message
Robert Liu (robertliu) wrote :

Patched version
log of ModemManager

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "gobi-set-dell-modules-as-forbidden-devices.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Robert Liu (robertliu) wrote :

there is a bug #1725190 which is going to backport ModemManager 1.6 series to Xenial. Yesterday, I tested the proposed packages. It resolves the issue as well. If the release is coming soon, then I think it's not necessary to apply the fix. If not, could you please update the 1.4 series first?

Chih-Hsyuan Ho (chih)
no longer affects: modemmanager
description: updated
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I would say this is much more fitting for an SRU than the big backport. I would propose trying to release this anyway as the backport can take longer to get prepared and then reviewed - big upstream releases as SRUs are never easy to handle.

Looking at the debdiff, it's mostly ok although we would need a DEP-3 header on the patch [1] to give more background on the change that's being made. Also, the 'Non-maintainer upload' changelog entry is not required in Ubuntu. Could you fix those and re-attach the patch? Thanks!

[1] http://dep.debian.net/deps/dep3/

Revision history for this message
Robert Liu (robertliu) wrote :

[hide the comment #12, update the DEP-3 headers inside the patch]

Hi @Łukasz
the comments were addressed. and re-upload the debdiff.
Thanks.

Changed in oem-priority:
importance: Undecided → Critical
status: New → Confirmed
Changed in oem-priority:
status: Confirmed → Fix Released
Revision history for this message
Mathew Hodson (mhodson) wrote :

This bug was fixed in the package modemmanager - 1.6.4-1ubuntu0.16.04.1

---------------
modemmanager (1.6.4-1ubuntu0.16.04.1) xenial; urgency=medium

  * Backport to xenial (LP: #1725190).
  * debian/control: bump down debhelper to xenial vesion.

 -- Łukasz 'sil2100' Zemczak <email address hidden> Tue, 28 Nov 2017 12:40:02 +0100

Changed in modemmanager (Ubuntu):
importance: Undecided → Medium
importance: Medium → Critical
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.