Re: [BUG] Bad page map in process ibv_devinfo

[Lukas Razik <linux@razik.name>]

David Miller <davem@davemloft.net> wrote:
>>  That explains everything.  The problem is that we don't do the sparc64
>>  PTE handling code patching in modules.  So it's left at the default 4U
>>  versions.
> ...
>>  I'll work on a fix for this.
> 
> Ok, please test this out, thanks!
> 
> --------------------
> [PATCH] sparc64: Patch sun4v code sequences properly on module load.
> 
> Some of the sun4v code patching occurs in inline functions visible
> to, and usable by, modules.
> 
> Therefore we have to patch them up during module load.
> 
> Signed-off-by: David S. Miller <davem@davemloft.net>
> ---
> arch/sparc/kernel/entry.h    |    7 ++++++
> arch/sparc/kernel/module.c   |   27 +++++++++++++++++++++++
> arch/sparc/kernel/setup_64.c |   48 +++++++++++++++++++++++++----------------
> 3 files changed, 63 insertions(+), 19 deletions(-)
> 

Hi David,

sorry that it took so long but I had to sleep to avoid mistakes - it was 6am here.
Thanks again for this fast patch!!!

Your first patch I could apply against linux-2.6.39.4. Your second patch I can't.
I've tried to apply it manually but I've seen too many differences and gave up.

Therefore I applied _both_ patches against linux-3.1.1.
Then I've tested ibv_devinfo with the builtin mlx drivers from linux-3.1.1 and it seems that the BUG message is away.

But I can't do further testing because I would need the mlx modules from OFED-1.5.4-rc4 (newest version) which doesn't compile with linux-3.1.1:
http://thread.gmane.org/gmane.linux.drivers.rdma/10192

Hence I would need your patch against 2.6.39.4 to say for sure if it works now. :-/
Please don't feel forced by me! It would be nice to have a patch but I see that you work on many areas needing improvement. Then I must wait for an OFED version that is compatible with linux-3.1.1 ...

Regards,
Lukas
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html