[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 22/23] hw/rdma: Do not call rdma_backend_del_
From: |
Yuval Shaia |
Subject: |
Re: [Qemu-devel] [PATCH v3 22/23] hw/rdma: Do not call rdma_backend_del_gid on an empty gid |
Date: |
Sun, 18 Nov 2018 11:42:01 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Sat, Nov 17, 2018 at 02:25:55PM +0200, Marcel Apfelbaum wrote:
>
>
> On 11/13/18 9:13 AM, Yuval Shaia wrote:
> > When device goes down the function fini_ports loops over all entries in
> > gid table regardless of the fact whether entry is valid or not. In case
> > that entry is not valid we'd like to skip from any further processing in
> > backend device.
> >
> > Signed-off-by: Yuval Shaia <address@hidden>
> > ---
> > hw/rdma/rdma_rm.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/hw/rdma/rdma_rm.c b/hw/rdma/rdma_rm.c
> > index 35a96d9a64..e3f6b2f6ea 100644
> > --- a/hw/rdma/rdma_rm.c
> > +++ b/hw/rdma/rdma_rm.c
> > @@ -555,6 +555,10 @@ int rdma_rm_del_gid(RdmaDeviceResources *dev_res,
> > RdmaBackendDev *backend_dev,
> > {
> > int rc;
> > + if (!dev_res->port.gid_tbl[gid_idx].gid.global.interface_id) {
> > + return 0;
> > + }
> > +
> > rc = rdma_backend_del_gid(backend_dev, ifname,
> > &dev_res->port.gid_tbl[gid_idx].gid);
> > if (rc < 0) {
>
> Reviewed-by: Marcel Apfelbaum<address@hidden>
There seems to be a missing space separator between "Apfelbaum" and "<". Is
that ok?
>
> Thanks,
> Marcel
>
- [Qemu-devel] [PATCH v3 19/23] vl: Introduce shutdown_notifiers, (continued)
- [Qemu-devel] [PATCH v3 19/23] vl: Introduce shutdown_notifiers, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 18/23] hw/rdma: Remove unneeded code that handles more that one port, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 20/23] hw/pvrdma: Clean device's resource when system is shutdown, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 21/23] hw/rdma: Do not use bitmap_zero_extend to free bitmap, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 22/23] hw/rdma: Do not call rdma_backend_del_gid on an empty gid, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 00/23] Add support for RDMA MAD, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 23/23] docs: Update pvrdma device documentation, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 03/23] hw/rdma: Return qpn 1 if ibqp is NULL, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 05/23] hw/rdma: Add support for MAD packets, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 04/23] hw/rdma: Abort send-op if fail to create addr handler, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 07/23] hw/pvrdma: Make default pkey 0xFFFF, Yuval Shaia, 2018/11/13
- [Qemu-devel] [PATCH v3 01/23] contrib/rdmacm-mux: Add implementation of RDMA User MAD multiplexer, Yuval Shaia, 2018/11/13