lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:   Sun, 9 Sep 2018 23:42:01 +0200
From:   Lukas Wunner <lukas@...ner.de>
To:     Andreas Noever <andreas.noever@...il.com>,
        Mika Westerberg <mika.westerberg@...ux.intel.com>,
        Michael Jamet <michael.jamet@...el.com>,
        Yehezkel Bernat <YehezkelShB@...il.com>
Cc:     Stephen Hemminger <stephen@...workplumber.org>,
        linux-kernel@...r.kernel.org
Subject: [PATCH 0/5] Thunderbolt material for v4.20

Thunderbolt material for v4.20, comprising:

* A fix to skip disabled ports on tunnel establishment.  I am not aware
  that this has caused breakage in the real world so far, so no need to
  apply to v4.19 or backport to stable.

* Obtain PCI slot number from DROM and use it to correlate PCI devices
  with Thunderbolt ports.

* I am nominating myself as co-maintainer, see commit message for the
  reasons.

Regarding the driver's verbosity which Stephen Hemminger and Mika
Westerberg have taken exception to:

* Patch [2/5] logs the PCI slot number as well as data whose purpose is
  unknown at KERN_INFO severity.  I am not opposed to log the former at
  KERN_DEBUG severity, but the latter must stay at KERN_INFO to allow us
  to collect the data of various Thunderbolt devices in the wild and
  reverse-engineer its meaning.  If Intel divulges the data's meaning then
  it need no longer be dumped.  This would be ideal for everyone involved.

* Patch [4/5] logs a message at KERN_INFO severity whenever a PCI device
  correlates or no longer correlates with a Thunderbolt port.  I am not
  opposed to toning this down to KERN_DEBUG, subject to the introduction
  of tb_port_dbg().

Thanks,

Lukas


Lukas Wunner (5):
  thunderbolt: Skip disabled ports on tunnel establishment
  thunderbolt: Obtain PCI slot number from DROM
  thunderbolt: Move upstream_port to struct tb
  thunderbolt: Correlate PCI devices with Thunderbolt ports
  MAINTAINERS: Add Lukas Wunner as co-maintainer of thunderbolt

 MAINTAINERS                       |   1 +
 drivers/thunderbolt/Makefile      |   2 +-
 drivers/thunderbolt/adapter_pci.c | 166 ++++++++++++++++++++++++++++++
 drivers/thunderbolt/adapter_pci.h |  19 ++++
 drivers/thunderbolt/domain.c      |  12 +++
 drivers/thunderbolt/eeprom.c      |  21 ++++
 drivers/thunderbolt/icm.c         |  65 +++---------
 drivers/thunderbolt/switch.c      |  14 ++-
 drivers/thunderbolt/tb.c          |  23 ++---
 drivers/thunderbolt/tb.h          |  45 ++++++++
 include/linux/thunderbolt.h       |   2 +
 11 files changed, 305 insertions(+), 65 deletions(-)
 create mode 100644 drivers/thunderbolt/adapter_pci.c
 create mode 100644 drivers/thunderbolt/adapter_pci.h

-- 
2.18.0

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ