RE: [Patch v6 00/12] Introduce Microsoft Azure Network Adapter (MANA) RDMA driver

From: Long Li
Date: Wed Sep 21 2022 - 14:53:52 EST


> Subject: Re: [Patch v6 00/12] Introduce Microsoft Azure Network Adapter
> (MANA) RDMA driver
>
> On Tue, Sep 20, 2022 at 06:22:20PM -0700, longli@xxxxxxxxxxxxxxxxx wrote:
> > From: Long Li <longli@xxxxxxxxxxxxx>
> >
> > This patchset implements a RDMA driver for Microsoft Azure Network
> > Adapter (MANA). In MANA, the RDMA device is modeled as an auxiliary
> > device to the Ethernet device.
> >
> > The first 11 patches modify the MANA Ethernet driver to support RDMA
> driver.
> > The last patch implementes the RDMA driver.
> >
> > The user-mode of the driver is being reviewed at:
> >
> https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> > ub.com%2Flinux-rdma%2Frdma-
> core%2Fpull%2F1177&amp;data=05%7C01%7Clongl
> >
> i%40microsoft.com%7C1d30ce9d49bc411b0fb808da9bede4bd%7C72f988bf86
> f141a
> >
> f91ab2d7cd011db47%7C1%7C0%7C637993743320198387%7CUnknown%7CT
> WFpbGZsb3d
> >
> 8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%
> 3D%7C
> >
> 3000%7C%7C%7C&amp;sdata=4cT9PFnKqb2yHRLkKNrbVVMAGwO6Ig91eoQ
> zomlm6lY%3D
> > &amp;reserved=0
> >
> >
> > Ajay Sharma (3):
> > net: mana: Set the DMA device max segment size
> > net: mana: Define and process GDMA response code
> > GDMA_STATUS_MORE_ENTRIES
> > net: mana: Define data structures for protection domain and memory
> > registration
> >
> > Long Li (9):
> > net: mana: Add support for auxiliary device
> > net: mana: Record the physical address for doorbell page region
> > net: mana: Handle vport sharing between devices
> > net: mana: Add functions for allocating doorbell page from GDMA
> > net: mana: Export Work Queue functions for use by RDMA driver
> > net: mana: Record port number in netdev
> > net: mana: Move header files to a common location
> > net: mana: Define max values for SGL entries
> > RDMA/mana_ib: Add a driver for Microsoft Azure Network Adapter
>
> Still some basic checkpatchy stuff:
>
> /tmp/tmpm2fsg47h/0012-RDMA-mana_ib-Add-a-driver-for-Microsoft-
> Azure-Network-Adapter.patch:412: WARNING: quoted string split across
> lines
> #412: FILE: drivers/infiniband/hw/mana/main.c:70:
> + "vport handle %llx pdid %x doorbell_id %x "
> + "tx_shortform_allowed %d tx_vp_offset %u\n",
>
> /tmp/tmpm2fsg47h/0012-RDMA-mana_ib-Add-a-driver-for-Microsoft-
> Azure-Network-Adapter.patch:540: WARNING: quoted string split across
> lines
> #540: FILE: drivers/infiniband/hw/mana/main.c:198:
> + "size_dma_region %lu num_pages_total %lu, "
> + "page_sz 0x%llx offset_in_page %u\n",
>
> And it thinks you should write more for the kconfig symbol, eg why would
> someone want to turn it on (hint, to use dpkd on some Azure
> instances)

Will fix those.

>
> /tmp/tmpm2fsg47h/0012-RDMA-mana_ib-Add-a-driver-for-Microsoft-
> Azure-Network-Adapter.patch:100: WARNING: please write a help
> paragraph that fully describes the config symbol
> #100: FILE: drivers/infiniband/hw/mana/Kconfig:2:
> +config MANA_INFINIBAND
> + tristate "Microsoft Azure Network Adapter support"
> + depends on NETDEVICES && ETHERNET && PCI &&
> MICROSOFT_MANA
> + help
> + This driver provides low-level RDMA support for
> + Microsoft Azure Network Adapter (MANA).
>