On Tue, Jun 22, 2021 at 09:32:27AM +0000, Haakon Bugge wrote:
>
>
> > On 22 Jun 2021, at 10:18, Leon Romanovsky <leon@xxxxxxxxxx> wrote:
> >
> > On Mon, Jun 21, 2021 at 03:18:57PM +0200, Håkon Bugge wrote:
> >> An approximation for the PacketLifeTime is half the local ACK timeout.
> >> The encoding for both timers are logarithmic. The PacketLifeTime
> >> calculation is wrong when local ACK timeout is zero. In that case,
> >> PacketLifeTime is set to the incorrect value 255.
> >>
> >> Fixed by explicitly testing for timeout being zero.
> >>
> >> Fixes: e1ee1e62bec4 ("RDMA/cma: Use ACK timeout for RoCE packetLifeTime")
> >> Signed-off-by: Håkon Bugge <haakon.bugge@xxxxxxxxxx>
> >>
> >> ---
> >>
> >> * Note: This commit must be merged after ("RDMA/cma: Replace
> >> RMW with atomic bit-ops")
> >> ---
> >> drivers/infiniband/core/cma.c | 8 +++++---
> >> 1 file changed, 5 insertions(+), 3 deletions(-)
> >>
> >
> > Thanks,
> > Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxx>
>
> Thanks for the review, Leon! I have to rebase on the tip of for-next, since the ("RDMA/cma: Replace RMW with atomic bit-ops") will not have the get_bit() stuff in cma_resolve_iboe_route() anymore. I assume I can retain your r-b after the rebase?
Yes, please.
>
>
> Thxs, Håkon
>
>
>
>
