record more email comments
git-svn-id: svn://svn.valgrind.org/valgrind/trunk@5156 a5019735-40e9-0310-863c-91ae7b9d1cf9
diff --git a/coregrind/m_main.c b/coregrind/m_main.c
index b381a06..94345ec 100644
--- a/coregrind/m_main.c
+++ b/coregrind/m_main.c
@@ -658,7 +658,10 @@
vg_assert(ok);
/* We make the data segment (heap) executable because LinuxThreads on
- ppc32 creates trampolines in this area. */
+ ppc32 creates trampolines in this area. Also, on x86/Linux the data
+ segment is RWX natively, at least according to /proc/self/maps.
+ Also, having a non-executable data seg would kill any program which
+ tried to create code in the data seg and then run it. */
sres = VG_(am_mmap_anon_fixed_client)(
anon_start,
anon_size,