Re: [PATCH] NX protection for kernel data : fix 32 bits S3 suspend

From: H. Peter Anvin
Date: Tue Feb 01 2011 - 11:31:39 EST


On 02/01/2011 05:25 AM, castet.matthieu@xxxxxxx wrote:

The proper fix will be done in 2.6.29, when S3 resume 32 bit trampoline will be
merged with 64 bit (ie call smp trampoline) [1]


That's insufficient; the actual unified lowmem trampoline allocator should take care of that bit per se.

However, the bottom line seems to be that this patch was done with some extreme lack of care. To really be proper, every allocation from the get go -- including the brk and memblock based ones -- should be tagged with their execution status, and that information should be used, not some magic ranges.

-hpa
--
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/