Zugriff auf Inhalt eines USB-Sticks mittels USB Device ID

Wenn du dir nicht sicher bist, in welchem der anderen Foren du die Frage stellen sollst, dann bist du hier im Forum für allgemeine Fragen sicher richtig.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

Kann man unter Linux über DBus auch den Einhängepunkt eines USB-Speichers ermitteln?

Gruß
Atalanttore
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Ja. So wie ich schon in meiner ersten Antwort schrieb. Denn ueber DBUS werden die UDEV-Ereignisse verteilt, und denen kann man zuhoeren, und dann kann man rausfinden, was so alles passiert in deinem System, USB-Geraete-einstecken inklusive. Und Code der genau das tut, den habe ich um den Zeitpunkt zu dem du die Frage gestellt hast hier gepostet. Hast du den gesucht? Was ist dir daran unklar?
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Und die Frage wie man aus einem Blockdevice den Einhaengepuntk erhaelt habe ich *auch* schon beantwortet, zB mit lsblk. Wobei auch das ueber udev gehen muss, weil auch das einhaengen eines Geraetes ein Ereignis darstellt. Aber ggf. ist es ein bisschen einfacher.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

@__deets__: Ich habe deinen Code nun genommen und für meinen Anwendungszweck angepasst.

Code: Alles auswählen

from functools import partial
import os


class LinuxDeviceMonitor:

    def __init__(self):
        import pyudev
        self._context = pyudev.Context()

        self._monitor = pyudev.Monitor.from_netlink(self._context)
        self._monitor.start()

        self._devices = set()
        self._listeners = []
        self.rescan()

    def rescan(self):
        self._process_devices(
            self._context.list_devices(),
            action="add"
        )

    def add_listener(self, listener):
        self._listeners.append(listener)

    def remove_listener(self, listener):
        self._listeners.remove(listener)

    def fileno(self):
        return self._monitor.fileno()

    @property
    def devices(self):
        import pyudev
        return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]

    def process_incoming(self, *_args):
        from pyudev._util import eintr_retry_call
        read_device = partial(eintr_retry_call, self._monitor.poll, timeout=0)
        self._process_devices(iter(read_device, None))

    def _process_devices(self, devices, action=None):
        old = set(self._devices)
        for device in devices:
            action = device.action if action is None else action
            if action in ("add", "change") and self._is_usb_mass_storage_device(device):
                self._devices.add(device.sys_path)
            elif "remove" == action and device.sys_path in self._devices:
                self._devices.remove(device.sys_path)
        if self._devices != old:
            devices = self.devices
            for listener in self._listeners:
                listener(devices)

    def _is_usb_mass_storage_device(self, device):
        def dev_path(device, name):
            return "{}/{}".format(device.sys_path, name)

        def removable(device):
            removable_path = dev_path(device, "removable")
            try:
                return os.path.exists(removable_path) and int(open(removable_path).read())
            except ValueError:
                return False

        def has_size(device):
            size_path = dev_path(device, "size")
            try:
                return os.path.exists(size_path) and int(open(size_path).read())
            except ValueError:
                return False

        return device.subsystem == "block" and removable(device) and has_size(device)


if __name__ == "__main__":
    linux_device_monitor = LinuxDeviceMonitor()
    print(linux_device_monitor.devices)
An dem Code habe ich zwar einiges (noch) nicht verstanden, aber immerhin werden damit nun Pfade zu USB-Speichern ausgegeben.

Code: Alles auswählen

['/dev/sdb', '/dev/sr0']
Der eingesteckte USB-Stick hat den Pfad '/dev/sdb'. '/dev/sr0' ist anscheinend ein CD-ROM-Laufwerk. Mein Notebook hat zwar ein DVD-Laufwerk, aber auch einen Kartenleser.

Kann man mit `pyudev` die Ausgabe der Pfade auf USB-Sticks beschränken?

Gruß
Atalanttore
Benutzeravatar
sparrow
User
Beiträge: 4535
Registriert: Freitag 17. April 2009, 10:28

@Atalanttore Möchtest du denn bemerken, dass der USB-Stick angeschlossen wurde, oder reicht dir, wenn dein Programm danach sucht, wenn der Benutzer das auslöst?
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Dazu musst du das Praedikat _is_usb_mass_storage_device entsprechend erweitern. Wie genau - musst du schauen. Ich habe kein CD-Rom-Laufwerk oder was auch immer das genau ist. Du musst eben gucken, ob es da Informationen gibt, die eine Unterscheidung ermoeglichen.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

@sparrow: Das Programm sollte automatisch bemerken, wenn ein USB-Stick angeschlossen wurde.

@__deets__: `_is_usb_mass_storage_device()` habe ich schon versucht entsprechend zu erweitern. Das Ergebnis war bisher immer, dass danach gar nichts mehr erkannt wurde.

Mein zusammengebastelter Code mit Werten für `DEVICE_CLASS_MASS_STORAGE` und `INTERFACE_CLASS_MASS_STORAGE` hat nur bei USB-Sticks angeschlagen. Leider habe ich bisher keine Möglichkeit gefunden solche Abfragen auch mit `pyudev` vorzunehmen.

Mit `udevadm info -q all /dev/[sdb / sr0]` konnte ich mehrere Unterschiede ausmachen. Das Attribut `ID_CDROM` könnte ein passendes Unterscheidungsmerkmal sein, aber es scheitert nun bei mir am Zugriff auf dieses Attribut. Wie kann man darauf zugreifen?

Code: Alles auswählen

udevadm info -q all /dev/sr0
P: /devices/pci0000:00/0000:00:17.0/ata4/host3/target3:0:0/3:0:0:0/block/sr0
N: sr0
L: -100
S: cdrom
S: cdrw
S: disk/by-id/ata-HL-DT-ST_DVDRAM_GT80N_M44E61M3110
S: disk/by-id/wwn-0x5001480000000000
S: disk/by-path/pci-0000:00:17.0-ata-4
S: dvd
S: dvdrw
E: DEVLINKS=/dev/disk/by-id/wwn-0x5001480000000000 /dev/disk/by-path/pci-0000:00:17.0-ata-4 /dev/disk/by-id/ata-HL-DT-ST_DVDRAM_GT80N_M44E61M3110 /dev/cdrom /dev/cdrw /dev/dvdrw /dev/dvd
E: DEVNAME=/dev/sr0
E: DEVPATH=/devices/pci0000:00/0000:00:17.0/ata4/host3/target3:0:0/3:0:0:0/block/sr0
E: DEVTYPE=disk
E: ID_ATA=1
E: ID_ATA_FEATURE_SET_PM=1
E: ID_ATA_FEATURE_SET_PM_ENABLED=1
E: ID_ATA_SATA=1
E: ID_ATA_SATA_SIGNAL_RATE_GEN1=1
E: ID_BUS=ata
E: ID_CDROM=1
E: ID_CDROM_CD=1
E: ID_CDROM_CD_R=1
E: ID_CDROM_CD_RW=1
E: ID_CDROM_DVD=1
E: ID_CDROM_DVD_PLUS_R=1
E: ID_CDROM_DVD_PLUS_RW=1
E: ID_CDROM_DVD_PLUS_R_DL=1
E: ID_CDROM_DVD_R=1
E: ID_CDROM_DVD_RAM=1
E: ID_CDROM_DVD_RW=1
E: ID_CDROM_MRW=1
E: ID_CDROM_MRW_W=1
E: ID_FOR_SEAT=block-pci-0000_00_17_0-ata-4
E: ID_MODEL=HL-DT-ST_DVDRAM_GT80N
E: ID_MODEL_ENC=HL-DT-ST\x20DVDRAM\x20GT80N\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20
E: ID_PATH=pci-0000:00:17.0-ata-4
E: ID_PATH_TAG=pci-0000_00_17_0-ata-4
E: ID_REVISION=LN85
E: ID_SERIAL=HL-DT-ST_DVDRAM_GT80N_M44E61M3110
E: ID_SERIAL_SHORT=M44E61M3110
E: ID_TYPE=cd
E: ID_WWN=0x5001480000000000
E: ID_WWN_WITH_EXTENSION=0x5001480000000000
E: MAJOR=11
E: MINOR=0
E: SUBSYSTEM=block
E: SYSTEMD_MOUNT_DEVICE_BOUND=1
E: TAGS=:uaccess:seat:systemd:
E: USEC_INITIALIZED=3050832


udevadm info -q all /dev/sdb
P: /devices/pci0000:00/0000:00:14.0/usb1/1-6/1-6:1.0/host4/target4:0:0/4:0:0:0/block/sdb
N: sdb
S: disk/by-id/usb-090c_1000_10061017000299-0:0
S: disk/by-path/pci-0000:00:14.0-usb-0:6:1.0-scsi-0:0:0:0
E: DEVLINKS=/dev/disk/by-id/usb-090c_1000_10061017000299-0:0 /dev/disk/by-path/pci-0000:00:14.0-usb-0:6:1.0-scsi-0:0:0:0
E: DEVNAME=/dev/sdb
E: DEVPATH=/devices/pci0000:00/0000:00:14.0/usb1/1-6/1-6:1.0/host4/target4:0:0/4:0:0:0/block/sdb
E: DEVTYPE=disk
E: ID_BUS=usb
E: ID_INSTANCE=0:0
E: ID_MODEL=1000
E: ID_MODEL_ENC=1000
E: ID_MODEL_ID=1000
E: ID_PART_TABLE_TYPE=dos
E: ID_PART_TABLE_UUID=01a8fd35
E: ID_PATH=pci-0000:00:14.0-usb-0:6:1.0-scsi-0:0:0:0
E: ID_PATH_TAG=pci-0000_00_14_0-usb-0_6_1_0-scsi-0_0_0_0
E: ID_REVISION=1000
E: ID_SERIAL=090c_1000_10061017000299-0:0
E: ID_SERIAL_SHORT=10061017000299
E: ID_TYPE=disk
E: ID_USB_DRIVER=usb-storage
E: ID_USB_INTERFACES=:080650:
E: ID_USB_INTERFACE_NUM=00
E: ID_VENDOR=090c
E: ID_VENDOR_ENC=090c
E: ID_VENDOR_ID=090c
E: MAJOR=8
E: MINOR=16
E: SUBSYSTEM=block
E: TAGS=:systemd:
E: USEC_INITIALIZED=7887489066

Gruß
Atalanttore
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Ich würde ja eher ID_BUS=usb nehmen. Mein Ansatz funktioniert auf meinem Rechner augenscheinlich dank NVME SSD. Darum hat das für mich konkret gereicht.

Und schau dir halt mal an was das device Objekt so alles hat. Da sollten eigentlich alle Information stecken. Debugger anwerfen und reinschauen.
Benutzeravatar
sparrow
User
Beiträge: 4535
Registriert: Freitag 17. April 2009, 10:28

Unter Windows ist es etwas eklig über neue USB-Geräte informiert zu werden. Soviel ich weiß, funktioniert das nur, indem man einen eigenen Service startet.
Ich weiß nicht, wie dein genauer Anwendungsfall aussieht, aber ist es eine Option alle x-Sekunden zu pollen, ob der Stick nun da ist?
Benutzeravatar
sparrow
User
Beiträge: 4535
Registriert: Freitag 17. April 2009, 10:28

Sind die Ausgaben in 'dmesg' des USB-Subsystems unter Linux über alle Distributionen hinweg gleich?
Falls dem so ist, würde ich die Ausgabe von 'dmesg' und 'mount' parsen um an den Mountpoint zu kommen.
Wenn das Programm den Stick erwartet, würde ich alle x Sekunden pollen.

Code: Alles auswählen

import subprocess
import sys
if sys.platform == "win32":
    import winreg


def linux_find_mountpoint(device):
    out, err = subprocess.Popen(("mount",), stdout=subprocess.PIPE).communicate()
    searchstring = "/dev/{0}".format(device)
    for line in out.decode("UTF-8").splitlines():
        if line.startswith(searchstring):
            return line.split()[2]


def linux_read_device_from_dmesg(vendor_id, product_id):
    vendor_search = "IDVENDOR={0}".format(vendor_id)
    product_search = "IDPRODUCT={0}".format(product_id)
    out, err = subprocess.Popen(("dmesg",), stdout=subprocess.PIPE).communicate()
    usb_index = None
    scsi_host = None
    device = None
    for line in out.decode("UTF-8").splitlines():
        line = line.upper()
        if not usb_index \
            and "USB" in line and vendor_search in line and product_search in line:
            usb_index = line[line.find("USB")+4:line.find(":")]
        if usb_index and not scsi_host \
            and "SCSI HOST" in line and usb_index in line:
            scsi_host = line[line.find("SCSI HOST")+9:line.find(":")]
        if usb_index and scsi_host and not device \
            and "SD {0}:0:0:0:".format(scsi_host) in line \
            and "ATTACHED SCSI REMOVABLE DISK" in line:
            device = line[line.rfind("[")+1:line.rfind("]")].lower()
        if usb_index and scsi_host and device \
            and "USB {0}".format(usb_index) in line \
            and "USB DISCONNECT" in line:
            usb_index = None
            scsi_host = None
            device = None
    return device
            

def win_find_device_id(vendor_id, product_id):
    subkey = r"Software\Microsoft\Windows\CurrentVersion\Explorer\MountPoints2\CPC\Volume"
    volumeroot = winreg.OpenKey(winreg.HKEY_CURRENT_USER, subkey)
    volumenames = []
    try:
        for i in range(sys.maxsize):
            volumenames.append(winreg.EnumKey(volumeroot, i))
    except OSError:
        pass
    searchstring = bytearray("VID_{0}&PID_{1}".format(vendor_id, product_id),
                             "UTF-8")
    for volumename in volumenames:
        volumekey = winreg.OpenKey(volumeroot, volumename)
        data_value = winreg.QueryValueEx(volumekey, "Data")[0]
        cleared_value = data_value.replace(b"\0", b"")
        if searchstring in cleared_value:
            device_id = cleared_value[cleared_value.find(b"{"):
                                      cleared_value.find(b"}")]
            return device_id


def win_find_mountpoint_for_device(device_id):
    subkey = r"SYSTEM\MountedDevices"
    deviceroot = winreg.OpenKey(winreg.HKEY_LOCAL_MACHINE, subkey)
    try:
        for i in range(sys.maxsize):
            name, value, type_ = winreg.EnumValue(deviceroot, i)
            cleared_value = value.replace(b"\0", b"")
            if device_id in cleared_value and "DosDevices" in name:
                return name.split("\\")[-1]
    except OSError:
        pass


def get_path_for_usbid(vendor_id, product_id):
    vendor_id, product_id = vendor_id.upper(), product_id.upper()
    if sys.platform == "win32":
        device_id = win_find_device_id(vendor_id, product_id)
        if device_id:
            mountpoint = win_find_mountpoint_for_device(device_id)
            return mountpoint
    elif sys.platform == "linux":
        device = linux_read_device_from_dmesg(vendor_id, product_id)
        if device:
            return linux_find_mountpoint(device)
    else:
        error_text = "This does not work on {0}".format(sys.platform)
        raise NotImplementedError(error_text)


if __name__ == "__main__":
    print(get_path_for_usbid("abcd", "1234"))
Ich habe keien Ahnung was passiert, wenn es mehr als eine Partition auf dem Stick gibt.
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

@sparrow: diese Ausgaben basieren auf den Ereignissen von udev, die man direkt abgreifen kann. Mit dem Code der oben steht.
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Und das Kommando lsblk liefert einem für blockdevices ihre mountpoints und anders, im JSON Format.
Benutzeravatar
sparrow
User
Beiträge: 4535
Registriert: Freitag 17. April 2009, 10:28

Ich sehe noch ein Problem darin auf das Anstecken eins USB-Sticks zu reagieren.
Zumindest unter Kubuntu, da probiere ich das gerade aus, bedeutet das Einstecken eins USB-Sticks nicht, dass der auch automatisch gemounted wird. Der taucht dann zwar in der Geräteüberwachung auf, wird aber erst gemounted, wenn darüber darauf zugegriffen wird.
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Das hat erstmal nichts mit den Ereignissen zu tun. Das anstecken läuft garantiert in udev auf, was auch immer Ubuntu da sonst tun mag, folgt daraus.

Worauf wird denn da wie zugegriffen? Prinzipiell ist es ja zb mit pmount möglich auch ohne super User Geräte zu mounten. Was auch immer das dann auslöst.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

@sparrow: Danke für den vielen Code. Momentan bin ich allerdings mit dem Code von __deets__ für Linux noch vollkommen ausgelastet.


Bei der Erkennung von USB-Speichern konnte ich mein DVD-Laufwerk mittlerweile erfolgreich ausschließen. Ich habe noch etwas weiter an den Ausschlusskriterien geschraubt, damit auch per USB angeschlossene, externe DVD-Laufwerke nicht berücksichtigt werden.

Mittels `QSocketNotifier` konnte ich dann tatsächlich ein Signal auslösen, wenn ein USB-Stick an- oder abgesteckt wird. Allerdings ist da noch einiges im Argen. Der `QSocketNotifier` ist wohl nicht ganz korrekt eingebunden. :roll:
  • USB-Stick bei Programmstart nicht angesteckt:
    1. Wenn man ohne angestecktem USB-Stick das Programm startet, wird auf der Qt-Oberfläche ganz korrekt kein Pfad angezeigt.
    2. Steckt man den USB-Stick ein, während das Programm läuft, wird `[]` angezeigt. Anscheinend ist die Erkennung des USB-Sticks nicht schnell genug und es wird eine leere Liste zurückgegeben.
    3. Wenn man den USB-Stick wieder entfernt hat, wird weiterhin `[]` angezeigt.
  • USB-Stick bei Programmstart angesteckt:
    1. Mit angestecktem USB-Start beim Programmstart wird kein Pfad auf der Qt-Oberfläche angezeigt.
    2. Erst sobald man den USB-Stick entfernt, erscheint der Pfad und die Fehlermeldung unten läuft in Endlosschleife in der Konsole.
    3. Steckt man den USB-Stick wieder ein, erscheinen in der Konsole keine Fehlermeldungen mehr.

Fehlermeldung:

Code: Alles auswählen

Traceback (most recent call last):
  File "/home/ata/source/usb-detect/main.py", line 104, in <lambda>
    notifier.activated.connect(lambda: main_window.update_path(str(linux_device_monitor.devices)))
  File "/home/ata/source/usb-detect/main.py", line 50, in devices
    return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]
  File "/home/ata/source/usb-detect/main.py", line 50, in <listcomp>
    return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]
  File "/usr/lib/python3/dist-packages/pyudev/device/_device.py", line 83, in from_path
    return cls.from_sys_path(context, path)
  File "/usr/lib/python3/dist-packages/pyudev/device/_device.py", line 108, in from_sys_path
    raise DeviceNotFoundAtPathError(sys_path)
pyudev.device._errors.DeviceNotFoundAtPathError: No device at '/sys/devices/pci0000:00/0000:00:14.0/usb1/1-6/1-6:1.0/host4/target4:0:0/4:0:0:0/block/sdb'

main.py:

Code: Alles auswählen

from functools import partial
import os
import sys
from PyQt5.QtWidgets import QApplication, QMainWindow
from PyQt5.uic import loadUi
from PyQt5.QtCore import QSocketNotifier


class MainWindow(QMainWindow):

    def __init__(self, parent=None):
        super().__init__(parent)
        loadUi("mainwindow.ui", self)

    def update_path(self, path):
        self.label_usb_mass_storage_url.setText(path)


class LinuxDeviceMonitor:

    def __init__(self):
        import pyudev
        self._context = pyudev.Context()

        self._monitor = pyudev.Monitor.from_netlink(self._context)
        self._monitor.start()

        self._devices = set()
        self._listeners = []
        self.rescan()

    def rescan(self):
        self._process_devices(
            self._context.list_devices(),
            action="add"
        )

    def add_listener(self, listener):
        self._listeners.append(listener)

    def remove_listener(self, listener):
        self._listeners.remove(listener)

    def fileno(self):
        return self._monitor.fileno()

    @property
    def devices(self):
        import pyudev
        return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]

    def process_incoming(self, *_args):
        from pyudev._util import eintr_retry_call
        read_device = partial(eintr_retry_call, self._monitor.poll, timeout=0)
        self._process_devices(iter(read_device, None))

    def _process_devices(self, devices, action=None):
        old = set(self._devices)
        for device in devices:
            action = device.action if action is None else action
            if action in ("add", "change") and self._is_usb_mass_storage_device(device):
                self._devices.add(device.sys_path)
            elif "remove" == action and device.sys_path in self._devices:
                self._devices.remove(device.sys_path)
        if self._devices != old:
            devices = self.devices
            for listener in self._listeners:
                listener(devices)

    def _is_usb_mass_storage_device(self, device):
        def dev_path(device, name):
            return "{}/{}".format(device.sys_path, name)

        def removable(device):
            removable_path = dev_path(device, "removable")
            try:
                return os.path.exists(removable_path) and int(open(removable_path).read())
            except ValueError:
                return False

        def has_size(device):
            size_path = dev_path(device, "size")
            try:
                return os.path.exists(size_path) and int(open(size_path).read())
            except ValueError:
                return False

        def has_usb(device):
            return device.get("ID_BUS") == "usb"

        def has_no_disc(device):
            return device.get("ID_CDROM") is None

        return removable(device) and has_size(device) and has_usb(device) and has_no_disc(device)


def main():
    app = QApplication(sys.argv)
    main_window = MainWindow()
    main_window.show()
    linux_device_monitor = LinuxDeviceMonitor()

    notifier = QSocketNotifier(linux_device_monitor.fileno(), QSocketNotifier.Read, linux_device_monitor.process_incoming())
    notifier.activated.connect(lambda: main_window.update_path(str(linux_device_monitor.devices)))

    sys.exit(app.exec_())

if __name__ == '__main__':
    main()

mainwindow.ui:

Code: Alles auswählen

<?xml version="1.0" encoding="UTF-8"?>
<ui version="4.0">
 <class>MainWindow</class>
 <widget class="QMainWindow" name="MainWindow">
  <property name="geometry">
   <rect>
    <x>0</x>
    <y>0</y>
    <width>500</width>
    <height>250</height>
   </rect>
  </property>
  <property name="windowTitle">
   <string>MainWindow</string>
  </property>
  <widget class="QWidget" name="centralwidget">
   <layout class="QGridLayout" name="gridLayout">
    <item row="0" column="0">
     <widget class="QLabel" name="label_path_to_mass_storage">
      <property name="text">
       <string>USB-Massenspeicher:</string>
      </property>
     </widget>
    </item>
    <item row="0" column="1">
     <widget class="QLabel" name="label_usb_mass_storage_url">
      <property name="text">
       <string/>
      </property>
     </widget>
    </item>
   </layout>
  </widget>
  <widget class="QMenuBar" name="menubar">
   <property name="geometry">
    <rect>
     <x>0</x>
     <y>0</y>
     <width>500</width>
     <height>28</height>
    </rect>
   </property>
  </widget>
  <widget class="QStatusBar" name="statusbar"/>
 </widget>
 <resources/>
 <connections/>
</ui>
Gruß
Atalanttore
Benutzeravatar
sparrow
User
Beiträge: 4535
Registriert: Freitag 17. April 2009, 10:28

Wann genau willst du eigentlich die USB Device ID auswerten, die du im Threadtitel geschriebebn hast?
Und was meinst du mit der Device ID?
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Du rufst process_incoming AUF, statt es nur als callback zu übergeben.
__deets__
User
Beiträge: 14545
Registriert: Mittwoch 14. Oktober 2015, 14:29

Ok, nochmal genauer hingeschaut. Das ist ja noch etwas kruder... die process_incoming-Methode muss aufgerufen werden, wenn etwas auf dem Socket passiert. Was du stattdessen machst ist, deren Ruckgabewert (der zufällig None ist) als Parent an den notifier zu übergeben. WTF?

Stattdessen musst du den im activate Signal die process_incoming aufrufen lassen. Und die kann dann ggf zu Änderungen der GUI auffordern.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

sparrow hat geschrieben: Sonntag 10. März 2019, 17:00 Wann genau willst du eigentlich die USB Device ID auswerten, die du im Threadtitel geschriebebn hast?
Und was meinst du mit der Device ID?
Mit Device ID meinte ich die Vendor ID und Product ID von USB-Geräten. Eigentlich wollte ich damit den Einhängepunkt eines USB-Sticks herausfinden, aber das funktioniert offensichtlich nicht so einfach und auch nicht plattformunabhängig.
Atalanttore
User
Beiträge: 407
Registriert: Freitag 6. August 2010, 17:03

__deets__ hat geschrieben: Sonntag 10. März 2019, 17:47 Du rufst process_incoming AUF, statt es nur als callback zu übergeben.
__deets__ hat geschrieben: Sonntag 10. März 2019, 18:32 Ok, nochmal genauer hingeschaut. Das ist ja noch etwas kruder... die process_incoming-Methode muss aufgerufen werden, wenn etwas auf dem Socket passiert. Was du stattdessen machst ist, deren Ruckgabewert (der zufällig None ist) als Parent an den notifier zu übergeben. WTF?

Stattdessen musst du den im activate Signal die process_incoming aufrufen lassen. Und die kann dann ggf zu Änderungen der GUI auffordern.
Ohne Beispielcode musste ich mir den Code mithilfe der Doku selbst erstellen. Dabei kommt es bei mir regelmäßig zu Kollateralschäden, aber mit deinen Verbesserungsvorschlägen funktioniert es jetzt. :D

Nach dem Abziehen des USB-Sticks (ohne ordnungsgemäßem, vorherigen Aushängen) erscheint allerdings immer folgende Fehlermeldung in der Konsole:

Code: Alles auswählen

Traceback (most recent call last):
  File "/home/ata/source//usb-detect/main.py", line 58, in process_incoming
    self.main_window.update_path(str(self.devices))
  File "/home/ata/source//usb-detect/main.py", line 52, in devices
    return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]
  File "/home/ata/source//usb-detect/main.py", line 52, in <listcomp>
    return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]
  File "/usr/lib/python3/dist-packages/pyudev/device/_device.py", line 83, in from_path
    return cls.from_sys_path(context, path)
  File "/usr/lib/python3/dist-packages/pyudev/device/_device.py", line 108, in from_sys_path
    raise DeviceNotFoundAtPathError(sys_path)
pyudev.device._errors.DeviceNotFoundAtPathError: No device at '/sys/devices/pci0000:00/0000:00:14.0/usb1/1-6/1-6:1.0/host4/target4:0:0/4:0:0:0/block/sdb'
Wenn man den USB-Stick ordnungsgemäß aushängt, kommt es beim Abziehen zu keiner Fehlermeldung.

Lässt sich diese Fehlermeldung vermeiden oder ist das normal, wenn man ein USB-Gerät einfach abzieht?

main.py:

Code: Alles auswählen

from functools import partial
import os
import sys
from PyQt5.QtWidgets import QApplication, QMainWindow
from PyQt5.uic import loadUi
from PyQt5.QtCore import QSocketNotifier


class MainWindow(QMainWindow):

    def __init__(self, parent=None):
        super().__init__(parent)
        loadUi("mainwindow.ui", self)

    def update_path(self, path):
        self.label_usb_mass_storage_url.setText(path)


class LinuxDeviceMonitor:

    def __init__(self, main_window):
        import pyudev
        self._context = pyudev.Context()

        self._monitor = pyudev.Monitor.from_netlink(self._context)
        self._monitor.start()

        self._devices = set()
        self._listeners = []
        self.rescan()

        self.main_window = main_window

    def rescan(self):
        self._process_devices(
            self._context.list_devices(),
            action="add"
        )

    def add_listener(self, listener):
        self._listeners.append(listener)

    def remove_listener(self, listener):
        self._listeners.remove(listener)

    def fileno(self):
        return self._monitor.fileno()

    @property
    def devices(self):
        import pyudev
        return [pyudev.Devices.from_path(self._context, device).get("DEVNAME") for device in self._devices]

    def process_incoming(self, *_args):
        from pyudev._util import eintr_retry_call
        read_device = partial(eintr_retry_call, self._monitor.poll, timeout=0)
        self._process_devices(iter(read_device, None))
        self.main_window.update_path(str(self.devices))

    def _process_devices(self, devices, action=None):
        old = set(self._devices)
        for device in devices:
            action = device.action if action is None else action
            if action in ("add", "change") and self._is_usb_mass_storage_device(device):
                self._devices.add(device.sys_path)
            elif "remove" == action and device.sys_path in self._devices:
                self._devices.remove(device.sys_path)
        if self._devices != old:
            devices = self.devices
            for listener in self._listeners:
                listener(devices)

    def _is_usb_mass_storage_device(self, device):
        def dev_path(device, name):
            return "{}/{}".format(device.sys_path, name)

        def removable(device):
            removable_path = dev_path(device, "removable")
            try:
                return os.path.exists(removable_path) and int(open(removable_path).read())
            except ValueError:
                return False

        def has_size(device):
            size_path = dev_path(device, "size")
            try:
                return os.path.exists(size_path) and int(open(size_path).read())
            except ValueError:
                return False

        def has_usb(device):
            return device.get("ID_BUS") == "usb"

        def has_no_disc(device):
            return device.get("ID_CDROM") is None

        return removable(device) and has_size(device) and has_usb(device) and has_no_disc(device)


def main():
    app = QApplication(sys.argv)
    main_window = MainWindow()
    main_window.show()
    linux_device_monitor = LinuxDeviceMonitor(main_window)

    notifier = QSocketNotifier(linux_device_monitor.fileno(), QSocketNotifier.Read)
    notifier.activated.connect(linux_device_monitor.process_incoming)

    sys.exit(app.exec_())

if __name__ == '__main__':
    main()
Gruß
Atalanttore
Antworten