sanity: remove /proc/sys/vm/mmap_min_addr check
authorRoman I Khimov <khimov@altell.ru>
Mon, 8 Mar 2010 14:00:58 +0000 (17:00 +0300)
committerRoman I Khimov <khimov@altell.ru>
Tue, 9 Mar 2010 17:28:17 +0000 (20:28 +0300)
Binary locale generation fails on openSUSE 11.2 and probably would fail
on some other hosts with 2.6.31+ kernels since mmap-ing page zero may be
forbidden for security reasons even with /proc/sys/vm/mmap_min_addr set
to 0 (this also affects Ubuntu 9.10, although it's been fixed there:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/423513
).

This is not a problem for recently introduced qemu 0.12.3 since it can
relocate binaries now, which is tested on openSUSE 2.6.31 kernel with
mmap_min_addr=65536 and Debian lenny 2.6.26 with mmap_min_addr set to
0, 4096, 65536.

Given that, this check is just useless. Unfortunately, it also means
that for reliable builds we have to remove qemu 0.10 which can't work
with some kernels. Patch will follow to do that.

Signed-off-by: Roman I Khimov <khimov@altell.ru>
Acked-by: Tom Rini <tom_rini@mentor.com>
Acked-by: Koen Kooi <koen@openembedded.org>
Acked-by: Marcin Juszkiewicz <marcin@juszkiewicz.com.pl>
classes/sanity.bbclass

index 5f0e724..19310cb 100644 (file)
@@ -99,15 +99,6 @@ def check_sanity(e):
                if "qemu-native" in assume_provided:
                        required_utilities += " %s" % (qemu_target_binary(e.data))
 
-               try:
-                       if os.path.exists("/proc/sys/vm/mmap_min_addr"):
-                               f = file("/proc/sys/vm/mmap_min_addr", "r")
-                               if (f.read().strip() != "0"):
-                                       messages = messages + "/proc/sys/vm/mmap_min_addr is not 0. This will cause problems with qemu so please fix the value (as root).\n\nTo fix this in later reboots, set vm.mmap_min_addr = 0 in /etc/sysctl.conf.\n"
-                               f.close()
-               except:
-                       pass
-
        for util in required_utilities.split():
                if not check_app_exists( util, e.data ):
                        missing = missing + "%s," % util