On 12/20/2016 4:13 AM, Selvin Xavier wrote: > This series introduces the RoCE driver for the Broadcom > NetXtreme-E 10/25/40/50G RoCE HCAs. > This driver is dependent on the bnxt_en NIC driver and is > based on the bnxt_re branch in linux-rdma repository. > bnxt_en changes required for this patch series are already > available afore mentioned branch. > > These changes are available for your reference in > the bnxt_re_v3 branch of following repository. > https://github.com/Broadcom/linux-rdma-nxt/ > > Doug, > Please review and consider applying this to linux-rdma repository > for 4.11 merge cycle.
I certainly won't get it done before the holiday break coming up (Red
Hat, and by extension, myself, have a 1 week shutdown over the holiday
season, so I'll be offline starting this Friday).
However, even though you changed the api filename, there is still a lot
of inconsistency in the naming of your files. The module itself is
bnxt_re, not bnxtre or any other variant. Please make all uses of
bnxtre or other variants match bnxt_re. This includes the api file that
you just moved. I also want the directory in drivers/infiniband/hw to
be bnxt_re not bnxtre. Also, putting bnxt_re as part of the file name
for files already in the bnxt_re directory is redundant (all except for
bnxt_re.h, where it is appropriate). Just name the files things like
main.c and ib_verbs.c. And for the qplib files, drop the bnxt_ prefix
and just use qplib_*.
I'll make other comments as I sort through the files, but those are
things I would like to see changed so I wanted to get that feedback to
you sooner rather than later.
> Thanks,
> Selvin Xavier
>
> v2->v3:
> * Fix 0day build breakage
> * Fix cocci, kbuild robot, sparse, smatch and checkpatch warnings
> * Changed the filename bnxt_re_uverbs_abi.h to bnxtre-abi.h
> * Removed the __packed qualifier from the uverbs structure and adjusted
> the structure alignment to 64bits.
> * Added retry count to bail out in case of delayed or no response
> to FW commands
> * Removed the debugfs support from this patch series
> * Changed some of the defines as inline functions based on Jason's comment
> * Split two functions to get rid of switch within switch construct
> * Removed bnxt_re_copy_to_udata as it is just a wrapper for ib_copy_to_udata
> * Added maintainers information to MAINTAINERS file
>
> v1-> v2:
> * The license text in each file updated to reflect Dual license.
> * Makefile and Kconfig changes are pushed to the last patch
> * Moved bnxt_re_uverbs_abi.h to include/uapi/rdma folder
> * Remove duplicate structure definitions from bnxt_re_hsi.h as
> it is available in the corresponding bnxt_en header file (bnxt_hsi.h)
> * Removed some unused code reported during code review.
> * Fixed few sparse warnings
>
>
> Selvin Xavier (21):
> bnxt_re: Add bnxt_re RoCE driver files
> bnxt_re: Introducing autogenerated Host Software Interface(hsi) file
> bnxt_re: register with the NIC driver
> bnxt_re: Enabling RoCE control path
> bnxt_re: Adding Notification Queue support
> bnxt_re: Support for PD, ucontext and mmap verbs
> bnxt_re: Support for query and modify device verbs
> bnxt_re: Adding support for port related verbs
> bnxt_re: Support for GID related verbs
> bnxt_re: Support for CQ verbs
> bnxt_re: Support for AH verbs
> bnxt_re: Support memory registration verbs
> bnxt_re: Support QP verbs
> bnxt_re: Support post_send verb
> bnxt_re: Support post_recv
> bnxt_re: Support poll_cq verb
> bnxt_re: Handling dispatching of events to IB stack
> bnxt_re: Support for DCB
> bnxt_re: Set uverbs command mask
> bnxt_re: Add QP event handling
> bnxt_re: Add bnxt_re driver build support
>
> MAINTAINERS | 11 +
> drivers/infiniband/Kconfig | 2 +
> drivers/infiniband/hw/Makefile | 1 +
> drivers/infiniband/hw/bnxtre/Kconfig | 9 +
> drivers/infiniband/hw/bnxtre/Makefile | 6 +
> drivers/infiniband/hw/bnxtre/bnxt_qplib_fp.c | 2167 +++++++++++++++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_fp.h | 441 ++++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_rcfw.c | 692 +++++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_rcfw.h | 231 ++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_res.c | 825 ++++++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_res.h | 223 ++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_sp.c | 838 ++++++
> drivers/infiniband/hw/bnxtre/bnxt_qplib_sp.h | 160 ++
> drivers/infiniband/hw/bnxtre/bnxt_re.h | 150 ++
> drivers/infiniband/hw/bnxtre/bnxt_re_hsi.h | 2821 ++++++++++++++++++++
> drivers/infiniband/hw/bnxtre/bnxt_re_ib_verbs.c | 3206
> +++++++++++++++++++++++
> drivers/infiniband/hw/bnxtre/bnxt_re_ib_verbs.h | 196 ++
> drivers/infiniband/hw/bnxtre/bnxt_re_main.c | 1340 ++++++++++
> include/uapi/rdma/bnxtre-abi.h | 89 +
> 19 files changed, 13408 insertions(+)
> create mode 100644 drivers/infiniband/hw/bnxtre/Kconfig
> create mode 100644 drivers/infiniband/hw/bnxtre/Makefile
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_fp.c
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_fp.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_rcfw.c
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_rcfw.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_res.c
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_res.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_sp.c
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_qplib_sp.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_re.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_re_hsi.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_re_ib_verbs.c
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_re_ib_verbs.h
> create mode 100644 drivers/infiniband/hw/bnxtre/bnxt_re_main.c
> create mode 100644 include/uapi/rdma/bnxtre-abi.h
>
--
Doug Ledford <dledf...@redhat.com>
GPG Key ID: B826A3330E572FDD
Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD
signature.asc
Description: OpenPGP digital signature
