linux-kernel - Re: [PATCH v3 0/5] Add general DVSEC/VSEC support

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 PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 23 Sep 2021 08:44:17 -0700
From:   "David E. Box" <david.e.box@...ux.intel.com>
To:     Hans de Goede <hdegoede@...hat.com>, lee.jones@...aro.org,
        bhelgaas@...gle.com, andy.shevchenko@...il.com
Cc:     mgross@...ux.intel.com, srinivas.pandruvada@...el.com,
        linux-kernel@...r.kernel.org, platform-driver-x86@...r.kernel.org,
        linux-pci@...r.kernel.org
Subject: Re: [PATCH v3 0/5] Add general DVSEC/VSEC support

On Thu, 2021-09-23 at 11:04 +0200, Hans de Goede wrote:
> Hi,
> 
> On 9/22/21 11:30 PM, David E. Box wrote:
> > This patch enables general support for Intel defined PCIe VSEC and DVSEC
> > capabilities in the Intel Platform Monitoring Technology (PMT) driver.
> > Though the driver was written exclusively for PMT capabilities, newer DVSEC
> > and VSEC IDs for other capabilities can exist on the same device requiring
> > that the driver handle them.
> > 
> > V3 is mostly a resend of V2. It drops a platform/x86 patch that was picked
> > up separately by Hans in the last cycle. It also adds a new patch to
> > support an upcoming capability.
> > 
> > David E. Box (5):
> >   PCI: Add #defines for accessing PCIE DVSEC fields
> >   MFD: intel_pmt: Support non-PMT capabilities
> >   MFD: intel_pmt: Add support for PCIe VSEC structures
> >   MFD: intel_pmt: Add DG2 support
> >   MFD: intel_extended_cap: Add support for Intel SDSi
> 
> Since this mostly touches drivers/mfd/intel_pmt.c, I assume this is
> going to get merged through the MFD trees.

Yes. Thanks.

> 
> For the few small drivers/platform/x86 changes:
> 
> Acked-by: Hans de Goede <hdegoede@...hat.com>
> 
> Regards,
> 
> Hans
> 
> 
> 
> 
> 
> > 
> >  drivers/mfd/intel_pmt.c                    | 258 +++++++++++++++------
> >  drivers/platform/x86/intel/pmt/class.c     |   2 +
> >  drivers/platform/x86/intel/pmt/crashlog.c  |   2 +-
> >  drivers/platform/x86/intel/pmt/telemetry.c |   2 +-
> >  include/uapi/linux/pci_regs.h              |   4 +
> >  5 files changed, 191 insertions(+), 77 deletions(-)
> > 
>