Re: [PATCH] UV: fix incorrect tlb flush all issue

From: Ingo Molnar
Date: Wed Sep 26 2012 - 07:23:26 EST



* Cliff Wickman <cpw@xxxxxxx> wrote:

> From: Cliff Wickman <cpw@xxxxxxx>
>
> (this was sent as an ack on 9/13, but with incorrect title and sign-off)
>
> Ack.
> But with the adjustment below. The 'end' argument was not declared long.
>
> I tested the patch on a UV.
> It has the effect of either clearing 1 or all TLBs in a cpu.
> I added some debugging to test for the cases when clearing all TLBs is
> overkill, and in practice it happens very seldom.
>
> Reported-by: Jan Beulich <jbeulich@xxxxxxxx>
> Signed-off-by: Alex Shi <alex.shi@xxxxxxxxx>
> Signed-off-by: Cliff Wickman <cpw@xxxxxxx>

If this is still substantially the original patch then I suspect
this should be From: Alex Shi, and the small fix you made can be
added as:

Signed-off-by: Alex Shi <alex.shi@xxxxxxxxx>
[ fixed the 'end' argument to be declared 'long'. ]
Signed-off-by: Cliff Wickman <cpw@xxxxxxx>

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/