| Message ID | 20210324190453.8171-1-bharat@chelsio.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 3408be145a5d6418ff955fe5badde652be90e700 |
| Delegated to: | Jason Gunthorpe |
| Headers | show |
| Series | [v2,for-rc] RDMA/cxgb4: Fix adapter LE hash errors while destroying ipv6 listening server | expand |
On Thu, Mar 25, 2021 at 12:34:53AM +0530, Potnuri Bharat Teja wrote: > Not setting ipv6 bit while destroying ipv6 listening servers may result in > potential fatal adapter errors due to lookup engine memory hash errors. > Therefore always set ipv6 field while destroying ipv6 listening servers. > > Fixes: 830662f6f032 ("RDMA/cxgb4: Add support for active and passive open connection with IPv6 address") > Signed-off-by: Potnuri Bharat Teja <bharat@chelsio.com> > --- > Changes since v0: > - modified commit description to inform the severity of patch. > Changes since v1: > - removed extra variable as per Leon. > --- > --- > drivers/infiniband/hw/cxgb4/cm.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Thanks, Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
On Thu, Mar 25, 2021 at 12:34:53AM +0530, Potnuri Bharat Teja wrote: > Not setting ipv6 bit while destroying ipv6 listening servers may result in > potential fatal adapter errors due to lookup engine memory hash errors. > Therefore always set ipv6 field while destroying ipv6 listening servers. > > Fixes: 830662f6f032 ("RDMA/cxgb4: Add support for active and passive open connection with IPv6 address") > Signed-off-by: Potnuri Bharat Teja <bharat@chelsio.com> > Reviewed-by: Leon Romanovsky <leonro@nvidia.com> > --- > Changes since v0: > - modified commit description to inform the severity of patch. > Changes since v1: > - removed extra variable as per Leon. > --- > --- > drivers/infiniband/hw/cxgb4/cm.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied to for-rc, thanks Jason
diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c index 8769e7aa097f..81903749d241 100644 --- a/drivers/infiniband/hw/cxgb4/cm.c +++ b/drivers/infiniband/hw/cxgb4/cm.c @@ -3610,13 +3610,13 @@ int c4iw_destroy_listen(struct iw_cm_id *cm_id) ep->com.local_addr.ss_family == AF_INET) { err = cxgb4_remove_server_filter( ep->com.dev->rdev.lldi.ports[0], ep->stid, - ep->com.dev->rdev.lldi.rxq_ids[0], 0); + ep->com.dev->rdev.lldi.rxq_ids[0], false); } else { struct sockaddr_in6 *sin6; c4iw_init_wr_wait(ep->com.wr_waitp); err = cxgb4_remove_server( ep->com.dev->rdev.lldi.ports[0], ep->stid, - ep->com.dev->rdev.lldi.rxq_ids[0], 0); + ep->com.dev->rdev.lldi.rxq_ids[0], true); if (err) goto done; err = c4iw_wait_for_reply(&ep->com.dev->rdev, ep->com.wr_waitp,
Not setting ipv6 bit while destroying ipv6 listening servers may result in potential fatal adapter errors due to lookup engine memory hash errors. Therefore always set ipv6 field while destroying ipv6 listening servers. Fixes: 830662f6f032 ("RDMA/cxgb4: Add support for active and passive open connection with IPv6 address") Signed-off-by: Potnuri Bharat Teja <bharat@chelsio.com> --- Changes since v0: - modified commit description to inform the severity of patch. Changes since v1: - removed extra variable as per Leon. --- --- drivers/infiniband/hw/cxgb4/cm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)