linux-kernel - Re: [PATCH V7 03/10] PCI: Create PCI library functions in support of DOE mailboxes.

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: <20220406111110.00000e72@Huawei.com>
Date:   Wed, 6 Apr 2022 11:11:10 +0100
From:   Jonathan Cameron <Jonathan.Cameron@...wei.com>
To:     Ira Weiny <ira.weiny@...el.com>
CC:     Lukas Wunner <lukas@...ner.de>,
        Christoph Hellwig <hch@...radead.org>,
        "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>,
        Ben Widawsky <ben.widawsky@...el.com>,
        <linux-kernel@...r.kernel.org>, <linux-cxl@...r.kernel.org>,
        <linux-pci@...r.kernel.org>
Subject: Re: [PATCH V7 03/10] PCI: Create PCI library functions in support
 of DOE mailboxes.

On Tue, 5 Apr 2022 16:22:11 -0700
Ira Weiny <ira.weiny@...el.com> wrote:

> On Sat, Apr 02, 2022 at 04:48:45PM +0200, Lukas Wunner wrote:
> > On Thu, Mar 31, 2022 at 08:19:56AM -0700, Ira Weiny wrote:  
> > > On Wed, Mar 30, 2022 at 10:50:31PM -0700, Christoph Hellwig wrote:  
> > > > On Wed, Mar 30, 2022 at 04:59:13PM -0700, ira.weiny@...el.com wrote:  
> > > > > Introduced in a PCI v6.0[1], DOE provides a config space based mailbox
> > > > > with standard protocol discovery.  Each mailbox is accessed through a
> > > > > DOE Extended Capability.  
> > > > 
> > > > I really don't think this should be built unconditionally and bloat
> > > > every single kernel built with PCI support.  
> > > 
> > > I can add a Kconfig.  
> > 
> > Ideally, that config option should live in the pcie/ subdirectory,
> > i.e. in drivers/pci/pcie/Kconfig, alongside drivers/pci/pcie/doe.c,
> > as we try to consolidate PCIe-specific features there and reserve
> > core code in drivers/pci/*.c for functionality that also applies
> > to Conventional PCI.  
> 
> Thanks for letting me know about this direction.  I was unaware of this.

We had this in the pcie directory, but Bjorn asked us to move it to the pci
directory as there isn't anything specific to PCIe about DOE. You could
implement it on pci-x (maybe 2.0?) I think even though it's in the PCIe specification.

https://lore.kernel.org/linux-pci/20210413194927.GA2241331@bjorn-Precision-5520/

Thanks,

Jonathan


> 
> I'll move the file no problem, thanks,
> Ira
> 
> > 
> > Thanks,
> > 
> > Lukas