Re: [PATCH] rds: Fix inaccurate accounting of unsignaled wrs

From: Santosh Shilimkar
Date: Tue Oct 24 2017 - 12:05:28 EST


On 10/24/2017 7:16 AM, HÃkon Bugge wrote:
The number of unsignaled work-requests posted to the IB send queue is
tracked by a counter in the rds_ib_connection struct. When it reaches
zero, or the caller explicitly asks for it, the send-signaled bit is
set in send_flags and the counter is reset. This is performed by the
rds_ib_set_wr_signal_state() function.

However, this function is not always used which yields inaccurate
accounting. This commit fixes this, re-factors a code bloat related to
the matter, and makes the actual parameter type to the function
consistent.

Signed-off-by: HÃkon Bugge <haakon.bugge@xxxxxxxxxx>
---
Instead of partially doing changes inside/outside helper,
can also add inline helper for solicited state like
rds_ib_set_wr_solicited_state() and use that along
with this change.

Regards,
Santosh