linux-kernel - Re: [PATCH v11 0/4] misc: Add Synopsys DesignWare xData IP driver

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

Date:   Tue, 6 Apr 2021 19:34:02 +0200
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Gustavo Pimentel <Gustavo.Pimentel@...opsys.com>
Cc:     linux-doc@...r.kernel.org, linux-pci@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Derek Kiernan <derek.kiernan@...inx.com>,
        Dragan Cvetic <dragan.cvetic@...inx.com>,
        Arnd Bergmann <arnd@...db.de>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Jonathan Corbet <corbet@....net>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Krzysztof Wilczyński <kw@...ux.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>
Subject: Re: [PATCH v11 0/4] misc: Add Synopsys DesignWare xData IP driver

On Tue, Apr 06, 2021 at 07:26:45PM +0200, Gustavo Pimentel wrote:
> This patch series adds a new driver called xData-pcie for the Synopsys
> DesignWare PCIe prototype.
> 
> The driver configures and enables the Synopsys DesignWare PCIe traffic
> generator IP inside of prototype Endpoint which will generate upstream
> and downstream PCIe traffic. This allows to quickly test the PCIe link
> throughput speed and check is the prototype solution has some limitation
> or not.
> 
> Changes:
>  V2: Rework driver according to Greg Kroah-Hartman' feedback
>      - Replace module parameter by sysfs use.
>      - Replace bit fields structure with macros and masks use.
>      - Removed SET() and GET() macros by the writel() and readl().
>      - Removed some noisy info messages.
>  V3: Fixed issues detected while running on 64 bits platforms
>      Rebased patches on top of v5.11-rc1 version
>  V4: Rework driver according to Greg Kroah-Hartman' feedback
>      - Add the ABI doc related to the sysfs implemented on this driver
>  V5: Rework driver accordingly to Leon Romanovsky' feedback
>      - Removed "default n" on Kconfig
>      Rework driver accordingly to Krzysztof Wilczyński' feedback
>      - Added some explanatory comments for some steps
>      - Added some bit defines instead of magic numbers
>  V6: Rework driver according to Greg Kroah-Hartman' feedback
>      - Squashed patches #2 and #3
>      - Removed units (MB/s) on the sys file
>      - Reduced mutex scope on the functions called by sysfs
>      Rework driver accordingly to Krzysztof Wilczyński' feedback
>      - Fix typo "DesignWare"
>  V7: Rework driver according to Greg Kroah-Hartman' feedback
>      - Created a sub device (misc device) that will be associated with the PCI driver
>      - sysfs group is now associated with the misc drivers instead of the PCI driver
>  V8: Rework driver according to Greg Kroah-Hartman' feedback
>      - Added more detail to the version changes on the cover letter
>      - Squashed patches #1 and #2
>      - Removed struct device from the dw_xdata_pcie structure
>      - Replaced the pci_*() use by dev_*()
>      - Added free call for the misc driver name allocation
>      - Added reference counting
>      - Removed snps_edda_data structure and their usage
>      Rebased patches on top of v5.12-rc4 version
>  V9: Squashed temporary development patch #5 into the driver patch #1
>  V10: Reworked the write_store() and read_store() to validate the input using kstrtobool()
>      Removed stop_store()
>      Update ABI documentation accordingly
>  V11: Fixed the documentation based on the warnings detected by Stephen Rothwell

As mentioned on the other thread, I can't take these as I took your v10
into my public tree.  Please send fixes on top of that.

thanks,

greg k-h