[PowerPC]Disable randomized address space on Linux ppc64le

[lei]

Summary

Recent versions of Ubuntu (17.04 and 18.04) on PowerPC have introduced changes to Address Space Layout Randomization (ASLR) that is causing 500+ sanitizer failures. This patch disables ASLR when running the sanitizers on PowerPC 64bit LE.

We are continuing to investigate a long-term solution that will allow the sanitizers to run with ASLR enabled. However, in the meantime this will allow the sanitizers to function correctly on these systems.

[seurer]

The similar change for aarch64 was done in

compiler-rt/lib/tsan/rtl/tsan_platform_linux.cc

which seems a more "natural" location for this. Will that location not work for powerpc64?

Also, the wording of the warning is terrible (same as in the aarch64 one).

[nemanjai]

In D52900#1258162, @seurer wrote:

The similar change for aarch64 was done in

compiler-rt/lib/tsan/rtl/tsan_platform_linux.cc

which seems a more "natural" location for this. Will that location not work for powerpc64?

I thought this needed to be done for more than just tsan on PPC. Lei can correct me if I'm wrong here.

Also, the wording of the warning is terrible (same as in the aarch64 one).

Feel free to suggest a different wording.

compiler-rt/lib/sanitizer_common/sanitizer_linux.cc

1998

The comment does not match the macro. I expect that SANITIZER_PPC64V2 means:
"Sanitizer turned on, platform is PPC, ELFv2 ABI". The ELFv2 ABI does not necessarily suggest Linux on little endian. For example, FreeBSD big endian uses the same ABI.

[lei]

For PPC this needs to be done for both tsan and msan.

compiler-rt/lib/sanitizer_common/sanitizer_linux.cc

1998

FreeBSD defines SANITIZER_FREEBSD. This is only for PPCLE.

[seurer]

TSAN/MSAN: Ahh, got it.

As for the wording, how about something like this.

WARNING: This program is being run with address space layout randomization (ASLR) enabled which prevents the thread and memory sanitizers from working on powerpc64le. ASLR will be disabled and the program re-executed.

[nemanjai]

As for the wording, how about something like this.

WARNING: This program is being run with address space layout randomization (ASLR) enabled which prevents the thread and memory sanitizers from working on powerpc64le. ASLR will be disabled and the program re-executed.

I like it! And it somehow comes up highlighted in Phabricator :)

[tstellar]

Do you have more information about the change in Ubuntu that caused this to stop working?

[seurer]

It is not specific to Ubuntu but is changes in how wide a range of addresses that the kernel chooses when ASLR is enabled. After some updates in the 4.x kernels (sorry, I am not sure which ones) the range is hundreds of times bigger which makes choosing the constants that the sanitizers use impossible.

[seurer]

As examples on a 4.4 kernel memory was very roughly in this range (sampled from many test runs):
0x0000000100000984 to 0x000000013FFF0984 which is a range of about 3FFF0000

While on a 4.15 kernel it was this:
0x0000000100980984 to 0x00001000F7E90984 which is a range of FFFF7510000

(both systems were Ubuntu)

[lei]

Warning message updated.

[lei]

Any other concerns about committing this patch?

[seurer]

FWIW it looks good to me

[nemanjai]

Since this fixes the sanitizers on PPC for the newer kernels and the fix seems perfectly reasonable and there are no concerns brought up by anyone, let's go ahead with this fix.

compiler-rt/lib/sanitizer_common/sanitizer_linux.cc

if (UNLIKELY(internal_sysctl(mib, 3, &paxflags, &len, NULL, 0) == -1)) {
     Printf("sysctl failed\n");
     Die();
   }
 
   if (UNLIKELY(paxflags & CTL_PROC_PAXFLAGS_ASLR)) {
     Printf("This sanitizer is not compatible with enabled ASLR\n");
     Die();
   }
+#elif SANITIZER_PPC64V2
+  // Disable ASLR for Linux PPC64LE.
+  int old_personality = personality(0xffffffff);
+  if (old_personality != -1 && (old_personality & ADDR_NO_RANDOMIZE) == 0) {
+    VReport(1, "WARNING: Program is run with randomized virtual address "
+              "space, which wouldn't work with linux ppc64le sanitizer.\n"
+              "Re-execing with fixed virtual address space.\n");
+    CHECK_NE(personality(old_personality | ADDR_NO_RANDOMIZE), -1);
+    ReExec();
+  }
 #else
   // Do nothing
 #endif
 }
 
 void PrintModuleMap() { }
 
 void CheckNoDeepBind(const char *filename, int flag) {