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:   Tue, 21 Apr 2020 00:23:45 +0000
From:   "Saleem, Shiraz" <shiraz.saleem@...el.com>
To:     Leon Romanovsky <leon@...nel.org>,
        "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>
CC:     "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "jgg@...pe.ca" <jgg@...pe.ca>,
        "Ismail, Mustafa" <mustafa.ismail@...el.com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
        "nhorman@...hat.com" <nhorman@...hat.com>,
        "sassmann@...hat.com" <sassmann@...hat.com>
Subject: RE: [RFC PATCH v5 01/16] RDMA/irdma: Add driver framework
 definitions

> Subject: Re: [RFC PATCH v5 01/16] RDMA/irdma: Add driver framework
> definitions
> 
> On Fri, Apr 17, 2020 at 10:12:36AM -0700, Jeff Kirsher wrote:
> > From: Mustafa Ismail <mustafa.ismail@...el.com>
> >
> > Register irdma as a virtbus driver capable of supporting virtbus
> > devices from multi-generation RDMA capable Intel HW. Establish the
> > interface with all supported netdev peer drivers and initialize HW.
> >
> > Signed-off-by: Mustafa Ismail <mustafa.ismail@...el.com>
> > Signed-off-by: Shiraz Saleem <shiraz.saleem@...el.com>
> > ---
> >  drivers/infiniband/hw/irdma/i40iw_if.c | 228 ++++++++++
> > drivers/infiniband/hw/irdma/irdma_if.c | 449 ++++++++++++++++++
> >  drivers/infiniband/hw/irdma/main.c     | 573 +++++++++++++++++++++++
> >  drivers/infiniband/hw/irdma/main.h     | 599 +++++++++++++++++++++++++
> >  4 files changed, 1849 insertions(+)
> >  create mode 100644 drivers/infiniband/hw/irdma/i40iw_if.c
> >  create mode 100644 drivers/infiniband/hw/irdma/irdma_if.c
> >  create mode 100644 drivers/infiniband/hw/irdma/main.c
> >  create mode 100644 drivers/infiniband/hw/irdma/main.h
> >
> 
> I didn't look in too much details, but three things caught my attention immediately:
> 1. Existence of ARP cache management logic in RDMA driver.

Our HW has an independent ARP table for the rdma block. 
driver needs to add an ARP table entry via an rdma admin
queue command before QP transitions to RTS.

> 2. Extensive use of dev_*() prints while we have ibdev_*() prints
The ib device object is not available till the end of the device init
similarly its unavailable early on in device deinit flows. So dev_*
is all we can use in those places.

3.Extra includes
> (moduleparam.h ???).
> 
This should be cleaned up. Thanks!

Powered by blists - more mailing lists