mbox series

[V2,rdma-core,0/4] Broadcom's rdma provider lib update

Message ID 20210505171056.514204-1-devesh.sharma@broadcom.com (mailing list archive)
Headers show
Series Broadcom's rdma provider lib update | expand

Message

Devesh Sharma May 5, 2021, 5:10 p.m. UTC
This patch series is a part of bigger feature submission which
changes the wqe posting logic. The current series adds the
ground work in the direction to change the wqe posting algorithm.

v1->v2
- added Fixes tag
- updated patch description
- dropped if() check before free.

Devesh Sharma (4):
  bnxt_re/lib: Check AH handler validity before use
  bnxt_re/lib: align base sq entry structure to 16B boundary
  bnxt_re/lib: consolidate hwque and swque in common structure
  bnxt_re/lib: query device attributes only once and store

 providers/bnxt_re/bnxt_re-abi.h |  24 ++---
 providers/bnxt_re/db.c          |   6 +-
 providers/bnxt_re/main.c        |  31 +++---
 providers/bnxt_re/main.h        |  15 ++-
 providers/bnxt_re/verbs.c       | 182 +++++++++++++++++---------------
 5 files changed, 138 insertions(+), 120 deletions(-)

Comments

Devesh Sharma May 10, 2021, 5:03 a.m. UTC | #1
On Wed, May 5, 2021 at 10:41 PM Devesh Sharma
<devesh.sharma@broadcom.com> wrote:
>
> This patch series is a part of bigger feature submission which
> changes the wqe posting logic. The current series adds the
> ground work in the direction to change the wqe posting algorithm.
>
> v1->v2
> - added Fixes tag
> - updated patch description
> - dropped if() check before free.
>
> Devesh Sharma (4):
>   bnxt_re/lib: Check AH handler validity before use
>   bnxt_re/lib: align base sq entry structure to 16B boundary
>   bnxt_re/lib: consolidate hwque and swque in common structure
>   bnxt_re/lib: query device attributes only once and store
>
>  providers/bnxt_re/bnxt_re-abi.h |  24 ++---
>  providers/bnxt_re/db.c          |   6 +-
>  providers/bnxt_re/main.c        |  31 +++---
>  providers/bnxt_re/main.h        |  15 ++-
>  providers/bnxt_re/verbs.c       | 182 +++++++++++++++++---------------
>  5 files changed, 138 insertions(+), 120 deletions(-)
>
> --
> 2.25.1
>
Hello Maintainers, Could you bless the V2 if there are no more
comments/suggestions...
Leon Romanovsky May 10, 2021, 5:41 a.m. UTC | #2
On Mon, May 10, 2021 at 10:33:48AM +0530, Devesh Sharma wrote:
> On Wed, May 5, 2021 at 10:41 PM Devesh Sharma
> <devesh.sharma@broadcom.com> wrote:
> >
> > This patch series is a part of bigger feature submission which
> > changes the wqe posting logic. The current series adds the
> > ground work in the direction to change the wqe posting algorithm.
> >
> > v1->v2
> > - added Fixes tag
> > - updated patch description
> > - dropped if() check before free.
> >
> > Devesh Sharma (4):
> >   bnxt_re/lib: Check AH handler validity before use
> >   bnxt_re/lib: align base sq entry structure to 16B boundary
> >   bnxt_re/lib: consolidate hwque and swque in common structure
> >   bnxt_re/lib: query device attributes only once and store
> >
> >  providers/bnxt_re/bnxt_re-abi.h |  24 ++---
> >  providers/bnxt_re/db.c          |   6 +-
> >  providers/bnxt_re/main.c        |  31 +++---
> >  providers/bnxt_re/main.h        |  15 ++-
> >  providers/bnxt_re/verbs.c       | 182 +++++++++++++++++---------------
> >  5 files changed, 138 insertions(+), 120 deletions(-)
> >
> > --
> > 2.25.1
> >
> Hello Maintainers, Could you bless the V2 if there are no more
> comments/suggestions...

I planned to take it after rdma-core release (today/tomorrow).

Thanks

> 
> -- 
> -Regards
> Devesh
Devesh Sharma May 10, 2021, 6:25 a.m. UTC | #3
On Mon, May 10, 2021 at 11:11 AM Leon Romanovsky <leon@kernel.org> wrote:
>
> On Mon, May 10, 2021 at 10:33:48AM +0530, Devesh Sharma wrote:
> > On Wed, May 5, 2021 at 10:41 PM Devesh Sharma
> > <devesh.sharma@broadcom.com> wrote:
> > >
> > > This patch series is a part of bigger feature submission which
> > > changes the wqe posting logic. The current series adds the
> > > ground work in the direction to change the wqe posting algorithm.
> > >
> > > v1->v2
> > > - added Fixes tag
> > > - updated patch description
> > > - dropped if() check before free.
> > >
> > > Devesh Sharma (4):
> > >   bnxt_re/lib: Check AH handler validity before use
> > >   bnxt_re/lib: align base sq entry structure to 16B boundary
> > >   bnxt_re/lib: consolidate hwque and swque in common structure
> > >   bnxt_re/lib: query device attributes only once and store
> > >
> > >  providers/bnxt_re/bnxt_re-abi.h |  24 ++---
> > >  providers/bnxt_re/db.c          |   6 +-
> > >  providers/bnxt_re/main.c        |  31 +++---
> > >  providers/bnxt_re/main.h        |  15 ++-
> > >  providers/bnxt_re/verbs.c       | 182 +++++++++++++++++---------------
> > >  5 files changed, 138 insertions(+), 120 deletions(-)
> > >
> > > --
> > > 2.25.1
> > >
> > Hello Maintainers, Could you bless the V2 if there are no more
> > comments/suggestions...
>
> I planned to take it after rdma-core release (today/tomorrow).
>
> Thanks
Sure, Thanks.
>
> >
> > --
> > -Regards
> > Devesh
>
>
Leon Romanovsky May 10, 2021, 9:50 a.m. UTC | #4
On Mon, May 10, 2021 at 11:55:24AM +0530, Devesh Sharma wrote:
> On Mon, May 10, 2021 at 11:11 AM Leon Romanovsky <leon@kernel.org> wrote:
> >
> > On Mon, May 10, 2021 at 10:33:48AM +0530, Devesh Sharma wrote:
> > > On Wed, May 5, 2021 at 10:41 PM Devesh Sharma
> > > <devesh.sharma@broadcom.com> wrote:
> > > >
> > > > This patch series is a part of bigger feature submission which
> > > > changes the wqe posting logic. The current series adds the
> > > > ground work in the direction to change the wqe posting algorithm.
> > > >
> > > > v1->v2
> > > > - added Fixes tag
> > > > - updated patch description
> > > > - dropped if() check before free.
> > > >
> > > > Devesh Sharma (4):
> > > >   bnxt_re/lib: Check AH handler validity before use
> > > >   bnxt_re/lib: align base sq entry structure to 16B boundary
> > > >   bnxt_re/lib: consolidate hwque and swque in common structure
> > > >   bnxt_re/lib: query device attributes only once and store
> > > >
> > > >  providers/bnxt_re/bnxt_re-abi.h |  24 ++---
> > > >  providers/bnxt_re/db.c          |   6 +-
> > > >  providers/bnxt_re/main.c        |  31 +++---
> > > >  providers/bnxt_re/main.h        |  15 ++-
> > > >  providers/bnxt_re/verbs.c       | 182 +++++++++++++++++---------------
> > > >  5 files changed, 138 insertions(+), 120 deletions(-)
> > > >
> > > > --
> > > > 2.25.1
> > > >
> > > Hello Maintainers, Could you bless the V2 if there are no more
> > > comments/suggestions...
> >
> > I planned to take it after rdma-core release (today/tomorrow).
> >
> > Thanks
> Sure, Thanks.

Applied, thanks

> >
> > >
> > > --
> > > -Regards
> > > Devesh
> >
> >