[v2] pvrdma: Add support for SRQ

[Qemu-devel] [PATCH v2 0/4] pvrdma: Add support for SRQ

Kamal Heib posted 4 patches 5 years ago

Diff against v1 v3
Download series mbox

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test asan passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190326125433.475-1-kamalheib1@gmail.com

Maintainers: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Yuval Shaia <yuval.shaia@oracle.com>

There is a newer version of this series

hw/rdma/rdma_backend.c      | 125 +++++++++++++++++++++-
hw/rdma/rdma_backend.h      |  18 +++-
hw/rdma/rdma_backend_defs.h |   5 +
hw/rdma/rdma_rm.c           | 106 +++++++++++++++++-
hw/rdma/rdma_rm.h           |  13 ++-
hw/rdma/rdma_rm_defs.h      |   9 ++
hw/rdma/vmw/pvrdma_cmd.c    | 208 ++++++++++++++++++++++++++++++++----
hw/rdma/vmw/pvrdma_main.c   |  16 +++
hw/rdma/vmw/pvrdma_qp_ops.c |  46 +++++++-
hw/rdma/vmw/pvrdma_qp_ops.h |   1 +
10 files changed, 513 insertions(+), 34 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v2 0/4] pvrdma: Add support for SRQ

Posted by Kamal Heib 5 years ago

This series implements the SRQ (Shared Receive Queue) for the pvrdma
device, It also includes all the needed functions and definitions for
support SRQ in the backend and resource management layers.

Changes from v1->v2:
- Handle checkpatch.pl warnings. 

Kamal Heib (4):
  hw/rdma: Add SRQ support to backend layer
  hw/rdma: Add support for managing SRQ resource
  hw/rdma: Modify create/destroy QP to support SRQ
  hw/pvrdma: Add support for SRQ

 hw/rdma/rdma_backend.c      | 125 +++++++++++++++++++++-
 hw/rdma/rdma_backend.h      |  18 +++-
 hw/rdma/rdma_backend_defs.h |   5 +
 hw/rdma/rdma_rm.c           | 106 +++++++++++++++++-
 hw/rdma/rdma_rm.h           |  13 ++-
 hw/rdma/rdma_rm_defs.h      |   9 ++
 hw/rdma/vmw/pvrdma_cmd.c    | 208 ++++++++++++++++++++++++++++++++----
 hw/rdma/vmw/pvrdma_main.c   |  16 +++
 hw/rdma/vmw/pvrdma_qp_ops.c |  46 +++++++-
 hw/rdma/vmw/pvrdma_qp_ops.h |   1 +
 10 files changed, 513 insertions(+), 34 deletions(-)

-- 
2.20.1

Re: [Qemu-devel] [PATCH v2 0/4] pvrdma: Add support for SRQ

Posted by Yuval Shaia 5 years ago

On Tue, Mar 26, 2019 at 02:54:29PM +0200, Kamal Heib wrote:
> This series implements the SRQ (Shared Receive Queue) for the pvrdma
> device, It also includes all the needed functions and definitions for
> support SRQ in the backend and resource management layers.
> 
> Changes from v1->v2:
> - Handle checkpatch.pl warnings. 

Hi Kamal,
I'm done with the review, please check my comments.

Thanks for doing it, this is an advance feature that i wanted to do for
long time.

> 
> Kamal Heib (4):
>   hw/rdma: Add SRQ support to backend layer
>   hw/rdma: Add support for managing SRQ resource
>   hw/rdma: Modify create/destroy QP to support SRQ
>   hw/pvrdma: Add support for SRQ
> 
>  hw/rdma/rdma_backend.c      | 125 +++++++++++++++++++++-
>  hw/rdma/rdma_backend.h      |  18 +++-
>  hw/rdma/rdma_backend_defs.h |   5 +
>  hw/rdma/rdma_rm.c           | 106 +++++++++++++++++-
>  hw/rdma/rdma_rm.h           |  13 ++-
>  hw/rdma/rdma_rm_defs.h      |   9 ++
>  hw/rdma/vmw/pvrdma_cmd.c    | 208 ++++++++++++++++++++++++++++++++----
>  hw/rdma/vmw/pvrdma_main.c   |  16 +++
>  hw/rdma/vmw/pvrdma_qp_ops.c |  46 +++++++-
>  hw/rdma/vmw/pvrdma_qp_ops.h |   1 +
>  10 files changed, 513 insertions(+), 34 deletions(-)
> 
> -- 
> 2.20.1
> 
>

Re: [Qemu-devel] [PATCH v2 0/4] pvrdma: Add support for SRQ

Posted by Kamal Heib 5 years ago


On 3/27/19 6:18 PM, Yuval Shaia wrote:
> On Tue, Mar 26, 2019 at 02:54:29PM +0200, Kamal Heib wrote:
>> This series implements the SRQ (Shared Receive Queue) for the pvrdma
>> device, It also includes all the needed functions and definitions for
>> support SRQ in the backend and resource management layers.
>>
>> Changes from v1->v2:
>> - Handle checkpatch.pl warnings. 
> 
> Hi Kamal,

Hi Yuval,

> I'm done with the review, please check my comments.
> 

Thank you for reviewing the patch set, I'll address the comments in v3.

> Thanks for doing it, this is an advance feature that i wanted to do for
> long time.
> 
I'm very happy to contribute to this project :-)

Thanks,
Kamal

>>
>> Kamal Heib (4):
>>   hw/rdma: Add SRQ support to backend layer
>>   hw/rdma: Add support for managing SRQ resource
>>   hw/rdma: Modify create/destroy QP to support SRQ
>>   hw/pvrdma: Add support for SRQ
>>
>>  hw/rdma/rdma_backend.c      | 125 +++++++++++++++++++++-
>>  hw/rdma/rdma_backend.h      |  18 +++-
>>  hw/rdma/rdma_backend_defs.h |   5 +
>>  hw/rdma/rdma_rm.c           | 106 +++++++++++++++++-
>>  hw/rdma/rdma_rm.h           |  13 ++-
>>  hw/rdma/rdma_rm_defs.h      |   9 ++
>>  hw/rdma/vmw/pvrdma_cmd.c    | 208 ++++++++++++++++++++++++++++++++----
>>  hw/rdma/vmw/pvrdma_main.c   |  16 +++
>>  hw/rdma/vmw/pvrdma_qp_ops.c |  46 +++++++-
>>  hw/rdma/vmw/pvrdma_qp_ops.h |   1 +
>>  10 files changed, 513 insertions(+), 34 deletions(-)
>>
>> -- 
>> 2.20.1
>>
>>