Re: [PATCH] [3/8] CPA: Test the correct mapping alias on x86-64

From: Ingo Molnar
Date: Mon Feb 11 2008 - 06:49:53 EST



* Andi Kleen <ak@xxxxxxx> wrote:

> static_protections previously would test against the x86-64 kernel
> mapping twice. First against the unchanged symbol directly from the
> linker (which always points into the kernel mapping) and then again it
> would manually relocate the address into the kernel mapping and test
> again.
>
> This patch reverses the second test instead to test against the direct
> mapping (low) aliases virtual addresses which was probably intended in
> the first place.
>
> Simply use __pa and __va for that.

thanks, applied.

( the practical implications of this are low because we do not utilize
the low direct aliases for execution. It needs to be fixed
nevertheless (will be needed for PAT later on anyway) and your cleanup
and #ifdef reduction is nice to have as well. )

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/