Hi Mike,
will take a look at it tomorrow, on which kernel version is the patch
based?
Thomas
On 28 Jun 2017, at 19:46, Marciniszyn, Mike wrote:
Hi Mike, for the patch itself, it addresses a bug so,
Acked-by: Sagi Grimberg <sagi@xxxxxxxxxxx>
But,
As for not having any max pages for FMR it's really annoying.
What do you think about exposing max_fast_reg_page_list_len to
limit also for FMRs? We can also rename it to max_reg_page_list_len
to get rid of the FRWR association.
Thoughts?
I have no issue with your proposal, but I would rather get this fixed
and adjust the API with an additional fix. I suspect we probably
want to mark this fix stable.
I did audit the kernel ULPs for FMR vs. FRMR usage and the only one
that looked at max_fast_reg_page_list_len unconditionally was iser.
Thomas, can you verify this patch and reply with a Tested-by:
Meanwhile, I will audit kernel providers for which ones besides qib do
not support the extensions and FMR.
Mike
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
