> On Jun 19, 2016, at 3:38 PM, Or Gerlitz <gerlitz.or@xxxxxxxxx> wrote:
>
> On Sun, Jun 19, 2016 at 1:04 PM, Sagi Grimberg <sagigrim@xxxxxxxxx> wrote:
>
>>> I am able to reproduce the Local Protection Errors with
>>> this patch applied and SLUB debugging disabled.
>
>> Thanks Chuck for proving that the dma alignment is not the issue here.
>>
>> I suggest that we go with my dma coherent patch for now until Leon and
>> the Mellanox team can debug this one with the HW/FW folks and find out
>> what is going on.
>>
>> Leon, I had my share of debugging this area on mlx4/mlx5 areas. If you
>> want I can help with debugging this one.
>
> Hi Sagi, Leon and Co,
>
> From quick reading of the patch I got the impression that some scheme
> which used to work is now broken, did we get a bisection result
> pointing to the upstream commit which introduce the regression?
Fixes: 1b2cd0fc673c ('IB/mlx4: Support the new memory registration API')
The problem was introduced by the new FR API. I reported
this issue back in late April:
http://marc.info/?l=linux-rdma&m=146194706501705&w=2
and bisected the appearance of symptoms to:
commit d86bd1bece6fc41d59253002db5441fe960a37f6
Author: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>
Date: Tue Mar 15 14:55:12 2016 -0700
mm/slub: support left redzone
The left redzone changes the alignment characteristics
of regions returned by kmalloc. Further diagnosis showed
the problem was with mlx4_alloc_priv_pages(), and the
WR flush occurred only when mr->pages happened to contain
a page boundary.
What we don't understand is why a page boundary in that
array is a problem.
> I
> didn't see such note along the thread, basically, I think this is
> where we should be starting, thoughts? I also added the mlx4 core/IB
> maintainer.
Yishai was notified about this issue on May 25:
http://marc.info/?l=linux-rdma&m=146419192913960&w=2
--
Chuck Lever
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
