linux-kernel - Re: [PATCH v7 0/3] Modularization of DFL private feature drivers

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]

Date:   Wed, 19 Aug 2020 21:16:03 -0700
From:   Moritz Fischer <mdf@...nel.org>
To:     Xu Yilun <yilun.xu@...el.com>
Cc:     mdf@...nel.org, linux-fpga@...r.kernel.org,
        linux-kernel@...r.kernel.org, trix@...hat.com, lgoncalv@...hat.com
Subject: Re: [PATCH v7 0/3] Modularization of DFL private feature drivers

Hi Xu,

On Wed, Aug 19, 2020 at 03:45:18PM +0800, Xu Yilun wrote:
> This patchset makes it possible to develop independent driver modules
> for DFL private features. It also helps to leverage existing kernel
> drivers to enable some IP blocks in DFL.
> 
> Patch #1: Release the dfl mmio regions after enumeration, so that private
>           feature drivers could request mmio region in their own drivers.
> Patch #2: Introduce the dfl bus, then dfl devices could be supported by
>           independent dfl drivers.
> Patch #3: An example of the dfl driver for N3000 nios private feature.
> 
> 
> Main changes from v1:
> - Add the new Patch #1, to improve the feature id definition.
> - Change the dfl bus uevent format.
> - Change the dfl device's sysfs name format.
> - refactor dfl_dev_add()
> - Add the Patch #4 as an example of the dfl driver.
> - A lot of minor fixes for comments from Hao and Tom.
> 
> Main changes from v2:
> - Add the doc for dfl-n3000-nios driver.
> - Minor fixes for comments from Tom.
> 
> Main changes from v3:
> - improve the dfl devices' uevent format, 4 bits for type & 12 bits for id
> - change dfl_device->type to u8
> - A dedicate field in struct dfl_feature for dfl device instance.
> - error out if dfl_device already exist on dfl_devs_init().
> - Move the err log in regmap implementation, and delete
>    n3000_nios_writel/readl(), they have nothing to wrapper now.
> - Minor fixes and comments improvement.
> 
> Main changes from v4:
> - The patch "fpga: dfl: change data type of feature id to u16" is already
>    applied to for-next
> - Unify the naming of some functions in dfl.c
> - Fix the output of fec_mode sysfs inf to "no" on 10G configuration, cause
>    no FEC mode could be configured for 10G.
> - Change the N3000 Nios driver name from "dfl-n3000-nios" to "n3000-nios",
>    and also rename some structures and functions from dfl_n3000_nios_* to
>    n3000_nios_*
> - Minor fixes and comments improvement.
> 
> Main changes from v5:
> - Fix the output of fec_mode sysfs inf to "not supported" if in 10G,
>    or the firmware version major < 3.
> - The input param of dfl_devs_add() changes to
>    struct dfl_feature_platform_data.
> - Minor fixes and improves comments.
> 
> Main changes from v6:
> - Rebased to 5.9-rc1.
> - Improves comments.
> 
> Xu Yilun (3):
>   fpga: dfl: map feature mmio resources in their own feature drivers
>   fpga: dfl: create a dfl bus type to support DFL devices
>   fpga: dfl: add support for N3000 Nios private feature
> 
>  Documentation/ABI/testing/sysfs-bus-dfl            |  15 +
>  .../ABI/testing/sysfs-bus-dfl-devices-n3000-nios   |  21 +
>  Documentation/fpga/dfl-n3000-nios.rst              |  80 +++
>  Documentation/fpga/index.rst                       |   1 +
>  drivers/fpga/Kconfig                               |  11 +
>  drivers/fpga/Makefile                              |   2 +
>  drivers/fpga/dfl-n3000-nios.c                      | 542 +++++++++++++++++++++
>  drivers/fpga/dfl-pci.c                             |  24 +-
>  drivers/fpga/dfl.c                                 | 449 ++++++++++++++---
>  drivers/fpga/dfl.h                                 |  93 +++-
>  10 files changed, 1153 insertions(+), 85 deletions(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-bus-dfl
>  create mode 100644 Documentation/ABI/testing/sysfs-bus-dfl-devices-n3000-nios
>  create mode 100644 Documentation/fpga/dfl-n3000-nios.rst
>  create mode 100644 drivers/fpga/dfl-n3000-nios.c
> 
> -- 
> 2.7.4
> 

I'll get to your series this week, sorry for the delay.

Thanks,
Moritz