mbox series

[V3,rdma-core,0/5] Broadcom's user library update

Message ID 20210528093946.900940-1-devesh.sharma@broadcom.com (mailing list archive)
Headers show
Series Broadcom's user library update | expand

Message

Devesh Sharma May 28, 2021, 9:39 a.m. UTC
The focus of this patch series is to move SQ and RQ
wqe posting indices from 128B fixed stride to 16B
aligned stride. This allows more flexibility in choosing
wqe size.

v2 -> V3
 - Split the ABI change into separate patch
 - committed ABI patch using standard rdma-core script.

Devesh Sharma (5):
  Update kernel headers
  bnxt_re/lib: Read wqe mode from the driver
  bnxt_re/lib: add a function to initialize software queue
  bnxt_re/lib: Use separate indices for shadow queue
  bnxt_re/lib: Move hardware queue to 16B aligned indices

 kernel-headers/rdma/bnxt_re-abi.h |   5 +-
 providers/bnxt_re/bnxt_re-abi.h   |   5 +
 providers/bnxt_re/db.c            |  10 +-
 providers/bnxt_re/main.c          |   4 +
 providers/bnxt_re/main.h          |  26 ++
 providers/bnxt_re/memory.h        |  37 ++-
 providers/bnxt_re/verbs.c         | 522 ++++++++++++++++++++----------
 7 files changed, 431 insertions(+), 178 deletions(-)

Comments

Devesh Sharma May 31, 2021, 4:39 p.m. UTC | #1
On Fri, May 28, 2021 at 3:09 PM Devesh Sharma
<devesh.sharma@broadcom.com> wrote:
>
> The focus of this patch series is to move SQ and RQ
> wqe posting indices from 128B fixed stride to 16B
> aligned stride. This allows more flexibility in choosing
> wqe size.
>
> v2 -> V3
>  - Split the ABI change into separate patch
>  - committed ABI patch using standard rdma-core script.
>
> Devesh Sharma (5):
>   Update kernel headers
>   bnxt_re/lib: Read wqe mode from the driver
>   bnxt_re/lib: add a function to initialize software queue
>   bnxt_re/lib: Use separate indices for shadow queue
>   bnxt_re/lib: Move hardware queue to 16B aligned indices
>
>  kernel-headers/rdma/bnxt_re-abi.h |   5 +-
>  providers/bnxt_re/bnxt_re-abi.h   |   5 +
>  providers/bnxt_re/db.c            |  10 +-
>  providers/bnxt_re/main.c          |   4 +
>  providers/bnxt_re/main.h          |  26 ++
>  providers/bnxt_re/memory.h        |  37 ++-
>  providers/bnxt_re/verbs.c         | 522 ++++++++++++++++++++----------
>  7 files changed, 431 insertions(+), 178 deletions(-)
>
> --
> 2.25.1
>
Jason/Leon,

Both user and kernel components related to these changes have been
posted quite a while ago. Could you please take these in if there are
further comments.
Jason Gunthorpe May 31, 2021, 6:53 p.m. UTC | #2
On Mon, May 31, 2021 at 10:09:06PM +0530, Devesh Sharma wrote:
> On Fri, May 28, 2021 at 3:09 PM Devesh Sharma
> <devesh.sharma@broadcom.com> wrote:
> >
> > The focus of this patch series is to move SQ and RQ
> > wqe posting indices from 128B fixed stride to 16B
> > aligned stride. This allows more flexibility in choosing
> > wqe size.
> >
> > v2 -> V3
> >  - Split the ABI change into separate patch
> >  - committed ABI patch using standard rdma-core script.
> >
> > Devesh Sharma (5):
> >   Update kernel headers
> >   bnxt_re/lib: Read wqe mode from the driver
> >   bnxt_re/lib: add a function to initialize software queue
> >   bnxt_re/lib: Use separate indices for shadow queue
> >   bnxt_re/lib: Move hardware queue to 16B aligned indices
> >
> >  kernel-headers/rdma/bnxt_re-abi.h |   5 +-
> >  providers/bnxt_re/bnxt_re-abi.h   |   5 +
> >  providers/bnxt_re/db.c            |  10 +-
> >  providers/bnxt_re/main.c          |   4 +
> >  providers/bnxt_re/main.h          |  26 ++
> >  providers/bnxt_re/memory.h        |  37 ++-
> >  providers/bnxt_re/verbs.c         | 522 ++++++++++++++++++++----------
> >  7 files changed, 431 insertions(+), 178 deletions(-)
> >
> >
> Jason/Leon,
> 
> Both user and kernel components related to these changes have been
> posted quite a while ago. Could you please take these in if there are
> further comments.

Where is the kernel part here:

https://patchwork.kernel.org/project/linux-rdma/list/

?

Jason
Devesh Sharma June 1, 2021, 5:05 a.m. UTC | #3
On Tue, Jun 1, 2021 at 12:23 AM Jason Gunthorpe <jgg@ziepe.ca> wrote:
>
> On Mon, May 31, 2021 at 10:09:06PM +0530, Devesh Sharma wrote:
> > On Fri, May 28, 2021 at 3:09 PM Devesh Sharma
> > <devesh.sharma@broadcom.com> wrote:
> > >
> > > The focus of this patch series is to move SQ and RQ
> > > wqe posting indices from 128B fixed stride to 16B
> > > aligned stride. This allows more flexibility in choosing
> > > wqe size.
> > >
> > > v2 -> V3
> > >  - Split the ABI change into separate patch
> > >  - committed ABI patch using standard rdma-core script.
> > >
> > > Devesh Sharma (5):
> > >   Update kernel headers
> > >   bnxt_re/lib: Read wqe mode from the driver
> > >   bnxt_re/lib: add a function to initialize software queue
> > >   bnxt_re/lib: Use separate indices for shadow queue
> > >   bnxt_re/lib: Move hardware queue to 16B aligned indices
> > >
> > >  kernel-headers/rdma/bnxt_re-abi.h |   5 +-
> > >  providers/bnxt_re/bnxt_re-abi.h   |   5 +
> > >  providers/bnxt_re/db.c            |  10 +-
> > >  providers/bnxt_re/main.c          |   4 +
> > >  providers/bnxt_re/main.h          |  26 ++
> > >  providers/bnxt_re/memory.h        |  37 ++-
> > >  providers/bnxt_re/verbs.c         | 522 ++++++++++++++++++++----------
> > >  7 files changed, 431 insertions(+), 178 deletions(-)
> > >
> > >
> > Jason/Leon,
> >
> > Both user and kernel components related to these changes have been
> > posted quite a while ago. Could you please take these in if there are
> > further comments.
>
> Where is the kernel part here:
>
> https://patchwork.kernel.org/project/linux-rdma/list/
Yes it is there:

"[for-next,V3,3/3] RDMA/bnxt_re: update ABI to pass wqe-mode to user space"
https://patchwork.kernel.org/project/linux-rdma/patch/20210526153629.872796-4-devesh.sharma@broadcom.com

>
> ?
>
> Jason