linux-kernel - Re: [PATCH V9 6/9] cxl/port: Read CDAT table

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Ypk+Cc0fWelIcamX@iweiny-desk3>
Date:   Thu, 2 Jun 2022 15:47:37 -0700
From:   Ira Weiny <ira.weiny@...el.com>
To:     Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc:     Dan Williams <dan.j.williams@...el.com>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Alison Schofield <alison.schofield@...el.com>,
        Vishal Verma <vishal.l.verma@...el.com>,
        Dave Jiang <dave.jiang@...el.com>,
        Ben Widawsky <ben@...dawsk.net>, linux-kernel@...r.kernel.org,
        linux-cxl@...r.kernel.org, linux-pci@...r.kernel.org
Subject: Re: [PATCH V9 6/9] cxl/port: Read CDAT table

On Thu, Jun 02, 2022 at 11:31:56AM -0700, Ira wrote:
> On Wed, Jun 01, 2022 at 05:31:13PM +0100, Jonathan Cameron wrote:
> > On Wed, 1 Jun 2022 16:35:40 +0100
> > Jonathan Cameron <Jonathan.Cameron@...wei.com> wrote:
> > 
> > > On Tue, 31 May 2022 08:26:29 -0700
> > > ira.weiny@...el.com wrote:
> > > 
> > > > From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
> > > > 
> > > > The OS will need CDAT data from CXL devices to properly set up
> > > > interleave sets.  Currently this is supported through a DOE mailbox
> > > > which supports CDAT.
> > > > 
> > > > Cache the CDAT data for later parsing.  Provide a sysfs binary attribute
> > > > to allow dumping of the CDAT.
> > > > 
> > > > Binary dumping is modeled on /sys/firmware/ACPI/tables/
> > > > 
> > > > The ability to dump this table will be very useful for emulation of real
> > > > devices once they become available as QEMU CXL type 3 device emulation will
> > > > be able to load this file in.
> > > > 
> > > > This does not support table updates at runtime. It will always provide
> > > > whatever was there when first cached. Handling of table updates can be
> > > > implemented later.
> > > > 
> > > > Finally create a complete list of DOE defines within cdat.h for code
> > > > wishing to decode the CDAT table.
> > > > 
> > > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@...wei.com>
> > > > Co-developed-by: Ira Weiny <ira.weiny@...el.com>
> > > > Signed-off-by: Ira Weiny <ira.weiny@...el.com>
> > > >   
> > > 
> > > Fun question of ownership inline...
> > 
> > And a follow up due to triggering a bug that predated this series...
> > 
> > I'd send a fix, but I'm off on a long weekend shortly :)
> 
> NP I discussed with Dan and the use of dev_groups should allow me to move this
> to port probe where it belongs.  I put it here for the sysfs stuff.

Not to make a habit of replying to my own mails but this works.

So I'm going to go forward with spinning this again.

Ira