Re: [PATCH 10/10] IB: remove the unused usecnt field from struct ib_mr

Or Gerlitz
2015-12-20T11:28:02Z

On 12/20/2015 11:31 AM, Sagi Grimberg wrote:
> However, I think that the patch from Shani 6b52a12bc3fc ("IB/uverbs:
> Implement memory windows support in uverbs") is wrong. A memory
> window allocation should really reference the MR and not the PD (which
> is referenced by the MR). Otherwise the MR deregistration is allowed
> with windows bound to it. If this is the case then this field is not
> useless and we should fix ib_uverbs_alloc_mw()?

I tend to agree, lets not remove the field till this is clarified with
Haggai and Co

>
> Haggai, can you shed some light here? It's Shani's and your code.