On Thu, Aug 25, 2022 at 05:14:47PM -0500, Bob Pearson wrote:
> Currently in resize_finish() in rxe_queue.c there is a loop which
> copies the entries in the original queue into a newly allocated queue.
> The termination logic for this loop is incorrect. The call to
> queue_next_index() updates cons but has no effect on whether the
> queue is empty. So if the queue starts out empty nothing is copied
> but if it is not then the loop will run forever. This patch changes
> the loop to compare the value of cons to the original producer index.
>
> Fixes: ae6e843fe08d0 ("RDMA/rxe: Add memory barriers to kernel queues")
> Signed-off-by: Bob Pearson <rpearsonhpe@xxxxxxxxx>
> Reviewed-by: Li Zhijian <lizhijian@xxxxxxxxxxx>
> ---
> v2
> Fixed typo. Should have replaced all original 'prod' by 'new_prod'
> ---
> drivers/infiniband/sw/rxe/rxe_queue.c | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
Applied to for-next, thanks
Jason
