Unsupported USB devices are using power even if not used

Bug #1905603 reported by Marco Trevisan (Treviño)
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libfprint (Ubuntu)
Fix Released
Medium
Marco Trevisan (Treviño)
Focal
Fix Released
Medium
Marco Trevisan (Treviño)
Groovy
Fix Released
Medium
Marco Trevisan (Treviño)

Bug Description

[ Impact ]

Fingerprint devices that we know are not supported, aren't properly put in power-state mode

[ Test case ]

 - Ensure you've a non-supported device from this list:
   https://gitlab.freedesktop.org/libfprint/wiki/-/wikis/Unsupported%20Devices#usb-devices
 - Ensure the device stays in power-save mode:
   - You can use powertop, or follow this [1]

[ Regression potential ]

A device that is supported or used by another tool may be put in power-save mode

[1] https://web.archive.org/web/20120420112755/http://www.lesswatts.org/projects/devices-power-management/usb.php

Changed in libfprint (Ubuntu Groovy):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
importance: Undecided → Medium
status: New → In Progress
Changed in libfprint (Ubuntu Focal):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libfprint - 1:1.90.3+tod1-0ubuntu2

---------------
libfprint (1:1.90.3+tod1-0ubuntu2) hirsute; urgency=medium

  * debian/gbp.conf: Include debian's GNOME team preferred settings
  * debian/patches (upstream cherry-picks):
    - Properly handle aes3k devices, resubmitting commands.
      This also needs proper commands cancellation (LP: #1897613)
    - Add support for new Synaptics devices IDs (LP: #1905593)
    - Fix vfs301 verification using device pointer on callbacks (LP: #1905597)
    - Ensure we do not waste power for unsupported devices (LP: #1905603)
    - Add identification support to synaptics driver.
      It allows to unlock / log-in with any finger. (LP: #1905600)

 -- Marco Trevisan (Treviño) <email address hidden> Wed, 25 Nov 2020 19:18:20 +0100

Changed in libfprint (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Marco, or anyone else affected,

Accepted libfprint into groovy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libfprint/1:1.90.3+tod1-0ubuntu2~20.10.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-groovy to verification-done-groovy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-groovy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in libfprint (Ubuntu Groovy):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-groovy
Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Marco, or anyone else affected,

Accepted libfprint into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libfprint/1:1.90.2+tod1-0ubuntu1~20.04.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in libfprint (Ubuntu Focal):
status: In Progress → Fix Committed
tags: added: verification-needed-focal
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

tricky ~
❯ apt-cache policy 'libfprint-2-2'
libfprint-2-2:
  Installato: 1:1.90.2+tod1-0ubuntu1~20.04.3
  Candidato: 1:1.90.2+tod1-0ubuntu1~20.04.3
  Tabella versione:
 *** 1:1.90.2+tod1-0ubuntu1~20.04.3 400
        400 http://archive.ubuntu.com/ubuntu focal-proposed/main amd64 Packages
        100 /var/lib/dpkg/status

I've a blacklisted device here:
 Bus 001 Device 002: ID 138a:0090 Validity Sensors, Inc. VFS7500 Touch Fingerprint Sensor

And that's properly marked:

❯ udevadm info --query=all -n /dev/bus/usb/001/002
P: /devices/pci0000:00/0000:00:14.0/usb1/1-6
N: bus/usb/001/002
L: 0
E: DEVPATH=/devices/pci0000:00/0000:00:14.0/usb1/1-6
E: DEVNAME=/dev/bus/usb/001/002
E: DEVTYPE=usb_device
E: DRIVER=usb
E: PRODUCT=138a/90/164
E: TYPE=255/16/255
E: BUSNUM=001
E: DEVNUM=002
E: MAJOR=189
E: MINOR=1
E: SUBSYSTEM=usb
E: USEC_INITIALIZED=2054283
E: ID_VENDOR=138a
E: ID_VENDOR_ENC=138a
E: ID_VENDOR_ID=138a
E: ID_MODEL=0090
E: ID_MODEL_ENC=0090
E: ID_MODEL_ID=0090
E: ID_REVISION=0164
E: ID_SERIAL=138a_0090_16fc88b91447
E: ID_SERIAL_SHORT=16fc88b91447
E: ID_BUS=usb
E: ID_USB_INTERFACES=:ff0000:
E: ID_VENDOR_FROM_DATABASE=Validity Sensors, Inc.
E: ID_MODEL_FROM_DATABASE=VFS7500 Touch Fingerprint Sensor
E: ID_PATH=pci-0000:00:14.0-usb-0:6
E: ID_PATH_TAG=pci-0000_00_14_0-usb-0_6
E: LIBFPRINT_DRIVER=Hardcoded whitelist

tricky ~ SIGINT(2) 17:27:22
❯ cat /sys/devices/pci0000:00/0000:00:14.0/usb1/1-6/power/control
auto

tags: added: verification-done-focal
removed: verification-needed-focal
Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Marco, or anyone else affected,

Accepted libfprint into groovy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libfprint/1:1.90.3+tod1-0ubuntu2~20.10.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-groovy to verification-done-groovy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-groovy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Marco, or anyone else affected,

Accepted libfprint into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libfprint/1:1.90.2+tod1-0ubuntu1~20.04.4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

tags: added: verification-needed-focal
removed: verification-done-focal
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

As per https://bugs.launchpad.net/ubuntu/+source/libfprint/+bug/1905603/comments/4 this was already verified on focal, nothing changed in the latest upload, so re-marking it as verified

tags: added: verification-done-focal
removed: verification-needed-focal
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Also verified on groovy now:

apt-cache policy libfprint-2-2
libfprint-2-2:
  Installed: 1:1.90.3+tod1-0ubuntu2~20.10.2
  Candidate: 1:1.90.3+tod1-0ubuntu2~20.10.2
  Version table:
 *** 1:1.90.3+tod1-0ubuntu2~20.10.2 500
        500 http://archive.ubuntu.com/ubuntu groovy-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     1:1.90.3+tod1-0ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu groovy/main amd64 Packages

udevadm info --query=all -n /dev/bus/usb/001/004
P: /devices/pci0000:00/0000:00:14.0/usb1/1-9
N: bus/usb/001/004
L: 0
E: DEVPATH=/devices/pci0000:00/0000:00:14.0/usb1/1-9
E: DEVNAME=/dev/bus/usb/001/004
E: DEVTYPE=usb_device
E: DRIVER=usb
E: PRODUCT=6cb/bd/0
E: TYPE=255/16/255
E: BUSNUM=001
E: DEVNUM=004
E: MAJOR=189
E: MINOR=3
E: SUBSYSTEM=usb
E: USEC_INITIALIZED=3475228
E: ID_VENDOR=06cb
E: ID_VENDOR_ENC=06cb
E: ID_VENDOR_ID=06cb
E: ID_MODEL=00bd
E: ID_MODEL_ENC=00bd
E: ID_MODEL_ID=00bd
E: ID_REVISION=0000
E: ID_SERIAL=06cb_00bd_ea61678c21e3
E: ID_SERIAL_SHORT=ea61678c21e3
E: ID_BUS=usb
E: ID_USB_INTERFACES=:ff0000:
E: ID_VENDOR_FROM_DATABASE=Synaptics, Inc.
E: ID_MODEL_FROM_DATABASE=Prometheus MIS Touch Fingerprint Reader
E: ID_PATH=pci-0000:00:14.0-usb-0:9
E: ID_PATH_TAG=pci-0000_00_14_0-usb-0_9
E: LIBFPRINT_DRIVER=Synaptics Sensors

u@u-ThinkPad-X1-Yoga-4th:~$ cat /sys/devices/pci0000\:00/0000\:00\:14.0/usb1/1-9/power/control
auto

tags: added: verification-done verification-done-groovy
removed: verification-needed verification-needed-groovy
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libfprint - 1:1.90.3+tod1-0ubuntu2~20.10.2

---------------
libfprint (1:1.90.3+tod1-0ubuntu2~20.10.2) groovy; urgency=medium

  * debian/patches:
    - image-device: Delay completion until deactivation (LP: #1897613)
    - Support more synaptics devices PIDs (LP: #1905593)
    - Ensure we properly handle references of prints galleries (LP: #1908107)

libfprint (1:1.90.3+tod1-0ubuntu2~20.10.1) groovy; urgency=medium

  * debian/gbp.conf:
    - Include debian's GNOME team preferred settings
    - Prepare for groovy branching
  * debian/patches (upstream cherry-picks):
    - Properly handle aes3k devices, resubmitting commands.
      This also needs proper commands cancellation (LP: #1897613)
    - Add support for new Synaptics devices IDs (LP: #1905593)
    - Fix vfs301 verification using device pointer on callbacks (LP: #1905597)
    - Ensure we do not waste power for unsupported devices (LP: #1905603)
    - Add identification support to synaptics driver.
      It allows to unlock / log-in with any finger. (LP: #1905600)

 -- Marco Trevisan (Treviño) <email address hidden> Mon, 14 Dec 2020 19:19:45 +0100

Changed in libfprint (Ubuntu Groovy):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for libfprint has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libfprint - 1:1.90.2+tod1-0ubuntu1~20.04.4

---------------
libfprint (1:1.90.2+tod1-0ubuntu1~20.04.4) focal; urgency=medium

  * debian/patches:
    - image-device: Delay completion until deactivation (LP: #1897613)
    - Support more synaptics devices PIDs (LP: #1905593)
    - Ensure we properly handle references of prints galleries (LP: #1908107)

libfprint (1:1.90.2+tod1-0ubuntu1~20.04.3) focal; urgency=medium

  * debian/patches (upstream cherry-picks):
    - Properly handle aes3k devices, resubmitting commands.
      This also needs proper commands cancellation (LP: #1897613)
    - Add support for new Synaptics devices IDs (LP: #1905593)
    - Fix vfs301 verification using device pointer on callbacks (LP: #1905597)
    - Ensure we do not waste power for unsupported devices (LP: #1905603)
    - Add identification support to synaptics driver.
      It allows to unlock / log-in with any finger. (LP: #1905600)

 -- Marco Trevisan (Treviño) <email address hidden> Mon, 14 Dec 2020 19:28:09 +0100

Changed in libfprint (Ubuntu Focal):
status: Fix Committed → 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.