[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] hw/rdma: Remove unused parameter from rdma_
From: |
Yuval Shaia |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] hw/rdma: Remove unused parameter from rdma_poll_cq() |
Date: |
Wed, 13 Mar 2019 11:25:14 +0200 |
User-agent: |
Mutt/1.11.3 (2019-02-01) |
On Wed, Mar 13, 2019 at 10:46:46AM +0200, Kamal Heib wrote:
> The 'rdma_dev_res' parameter is not used in rdma_poll_cq(), so remove it.
>
> Signed-off-by: Kamal Heib <address@hidden>
> ---
> hw/rdma/rdma_backend.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/hw/rdma/rdma_backend.c b/hw/rdma/rdma_backend.c
> index 66185bd487b6..78bafc13642a 100644
> --- a/hw/rdma/rdma_backend.c
> +++ b/hw/rdma/rdma_backend.c
> @@ -94,8 +94,7 @@ static void clean_recv_mads(RdmaBackendDev *backend_dev)
> } while (cqe_ctx_id != -ENOENT);
> }
>
> -static int rdma_poll_cq(RdmaBackendDev *backend_dev,
> - RdmaDeviceResources *rdma_dev_res, struct ibv_cq
> *ibcq)
> +static int rdma_poll_cq(RdmaDeviceResources *rdma_dev_res, struct ibv_cq
> *ibcq)
> {
> int i, ne, total_ne = 0;
> BackendCtx *bctx;
> @@ -181,7 +180,7 @@ static void *comp_handler_thread(void *arg)
> }
>
> backend_dev->rdma_dev_res->stats.poll_cq_from_bk++;
> - rdma_poll_cq(backend_dev, backend_dev->rdma_dev_res, ev_cq);
> + rdma_poll_cq(backend_dev->rdma_dev_res, ev_cq);
>
> ibv_ack_cq_events(ev_cq, 1);
> }
> @@ -315,7 +314,7 @@ void rdma_backend_poll_cq(RdmaDeviceResources
> *rdma_dev_res, RdmaBackendCQ *cq)
> int polled;
>
> rdma_dev_res->stats.poll_cq_from_guest++;
> - polled = rdma_poll_cq(cq->backend_dev, rdma_dev_res, cq->ibcq);
> + polled = rdma_poll_cq(rdma_dev_res, cq->ibcq);
> if (!polled) {
> rdma_dev_res->stats.poll_cq_from_guest_empty++;
> }
> --
Reviewed-by: Yuval Shaia <address@hidden>
> 2.20.1
>
>
- [Qemu-devel] [PATCH 3/4] hw/rdma: Use {} instead of {0}, (continued)