linux-kernel - Re: [PATCH v11 07/20] cxl/hdm: Use stored Component Register mappings to map HDM decoder capability

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 for Android: free password hash cracker in your pocket

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

Message-ID: <ZS1ENmg1OfNluhXm@rric.localdomain>
Date:   Mon, 16 Oct 2023 16:09:58 +0200
From:   Robert Richter <rrichter@....com>
To:     Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc:     Alison Schofield <alison.schofield@...el.com>,
        Vishal Verma <vishal.l.verma@...el.com>,
        Ira Weiny <ira.weiny@...el.com>,
        Ben Widawsky <bwidawsk@...nel.org>,
        Dan Williams <dan.j.williams@...el.com>,
        Davidlohr Bueso <dave@...olabs.net>,
        Dave Jiang <dave.jiang@...el.com>, linux-cxl@...r.kernel.org,
        linux-kernel@...r.kernel.org, Bjorn Helgaas <bhelgaas@...gle.com>,
        Terry Bowman <terry.bowman@....com>
Subject: Re: [PATCH v11 07/20] cxl/hdm: Use stored Component Register
 mappings to map HDM decoder capability

On 02.10.23 15:43:51, Jonathan Cameron wrote:
> On Wed, 27 Sep 2023 17:43:26 +0200
> Robert Richter <rrichter@....com> wrote:
> 
> > Now, that the Component Register mappings are stored, use them to
> > enable and map the HDM decoder capabilities. The Component Registers
> > do not need to be probed again for this, remove probing code.
> > 
> > The HDM capability applies to Endpoints, USPs and VH Host Bridges. The
> > Endpoint's component register mappings are located in the cxlds and
> > else in the port's structure. Duplicate the cxlds->reg_map in
> > port->reg_map for endpoint ports.
> > 
> > Signed-off-by: Terry Bowman <terry.bowman@....com>
> > Signed-off-by: Robert Richter <rrichter@....com>
> > Reviewed-by: Dave Jiang <dave.jiang@...el.com>
> > [rework to drop cxl_port_get_comp_map()]
> > Signed-off-by: Dan Williams <dan.j.williams@...el.com>
> A few comments inline.
> 
> Also, Dan's SoB doesn't make sense if you are the Author and he's not
> the one sending the email.  The fun of patches bounced back and forwards
> is sometimes you have to tweak this stuff on each posting... :(

Yeah, there was some back and forth with the code going on: Terry
changed a little, Dan was taking it and reposting, Terry was also
submitting the previous series, Dave reviewed it. So I decided here to
just swap Terry and my SOBs as I was posting the series now and left
the remaining to Dan the way as it will look like when he will apply
the patch, which means updating the SOB chain again. This keeps the
comment for the change he made to the patch.

-Robert

> 
> > ---
> >  drivers/cxl/core/hdm.c  | 48 ++++++++++++++++-------------------------
> >  drivers/cxl/core/port.c | 29 +++++++++++++++++++------
> >  drivers/cxl/mem.c       |  5 ++---
> >  3 files changed, 43 insertions(+), 39 deletions(-)