Skip to content

Commit 5b04b6f

Browse files
authored
[HWASan] [compiler-rt] support non-4k pages on Android (#95069)
1 parent 7620fe0 commit 5b04b6f

10 files changed

+35
-29
lines changed

compiler-rt/lib/asan/asan_linux.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,8 @@ uptr FindDynamicShadowStart() {
9393
# endif
9494

9595
return MapDynamicShadow(shadow_size_bytes, ASAN_SHADOW_SCALE,
96-
/*min_shadow_base_alignment*/ 0, kHighMemEnd);
96+
/*min_shadow_base_alignment*/ 0, kHighMemEnd,
97+
GetMmapGranularity());
9798
}
9899

99100
void AsanApplyToGlobals(globals_op_fptr op, const void *needle) {

compiler-rt/lib/asan/asan_mac.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ bool IsSystemHeapAddress (uptr addr) { return false; }
5151

5252
uptr FindDynamicShadowStart() {
5353
return MapDynamicShadow(MemToShadowSize(kHighMemEnd), ASAN_SHADOW_SCALE,
54-
/*min_shadow_base_alignment*/ 0, kHighMemEnd);
54+
/*min_shadow_base_alignment*/ 0, kHighMemEnd,
55+
GetMmapGranularity());
5556
}
5657

5758
// No-op. Mac does not support static linkage anyway.

compiler-rt/lib/asan/asan_premap_shadow.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ uptr PremapShadowSize() {
3333
// PremapShadowSize() bytes on the right of it are mapped r/o.
3434
uptr PremapShadow() {
3535
return MapDynamicShadow(PremapShadowSize(), /*mmap_alignment_scale*/ 3,
36-
/*min_shadow_base_alignment*/ 0, kHighMemEnd);
36+
/*min_shadow_base_alignment*/ 0, kHighMemEnd,
37+
GetMmapGranularity());
3738
}
3839

3940
bool PremapShadowFailed() {

compiler-rt/lib/hwasan/hwasan_dynamic_shadow.cpp

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,20 @@ decltype(__hwasan_shadow)* __hwasan_premap_shadow();
3636

3737
namespace __hwasan {
3838

39+
// We cannot call anything in libc here (see comment above), so we need to
40+
// assume the biggest allowed page size.
41+
// Android max page size is defined as 16k here:
42+
// https://android.googlesource.com/platform/bionic/+/main/libc/platform/bionic/page.h#41
43+
static constexpr uptr kMaxGranularity = 16384;
44+
3945
// Conservative upper limit.
4046
static uptr PremapShadowSize() {
41-
return RoundUpTo(GetMaxVirtualAddress() >> kShadowScale,
42-
GetMmapGranularity());
47+
return RoundUpTo(GetMaxVirtualAddress() >> kShadowScale, kMaxGranularity);
4348
}
4449

4550
static uptr PremapShadow() {
4651
return MapDynamicShadow(PremapShadowSize(), kShadowScale,
47-
kShadowBaseAlignment, kHighMemEnd);
52+
kShadowBaseAlignment, kHighMemEnd, kMaxGranularity);
4853
}
4954

5055
static bool IsPremapShadowAvailable() {
@@ -56,7 +61,7 @@ static bool IsPremapShadowAvailable() {
5661
}
5762

5863
static uptr FindPremappedShadowStart(uptr shadow_size_bytes) {
59-
const uptr granularity = GetMmapGranularity();
64+
const uptr granularity = kMaxGranularity;
6065
const uptr shadow_start = reinterpret_cast<uptr>(&__hwasan_shadow);
6166
const uptr premap_shadow_size = PremapShadowSize();
6267
const uptr shadow_size = RoundUpTo(shadow_size_bytes, granularity);
@@ -109,7 +114,7 @@ uptr FindDynamicShadowStart(uptr shadow_size_bytes) {
109114
if (IsPremapShadowAvailable())
110115
return FindPremappedShadowStart(shadow_size_bytes);
111116
return MapDynamicShadow(shadow_size_bytes, kShadowScale, kShadowBaseAlignment,
112-
kHighMemEnd);
117+
kHighMemEnd, kMaxGranularity);
113118
}
114119

115120
} // namespace __hwasan
@@ -135,7 +140,7 @@ uptr FindDynamicShadowStart(uptr shadow_size_bytes) {
135140
RingBufferSize());
136141
# endif
137142
return MapDynamicShadow(shadow_size_bytes, kShadowScale, kShadowBaseAlignment,
138-
kHighMemEnd);
143+
kHighMemEnd, GetMmapGranularity());
139144
}
140145

141146
} // namespace __hwasan

compiler-rt/lib/memprof/memprof_linux.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,8 @@ void InitializePlatformExceptionHandlers() {}
5858
uptr FindDynamicShadowStart() {
5959
uptr shadow_size_bytes = MemToShadowSize(kHighMemEnd);
6060
return MapDynamicShadow(shadow_size_bytes, SHADOW_SCALE,
61-
/*min_shadow_base_alignment*/ 0, kHighMemEnd);
61+
/*min_shadow_base_alignment*/ 0, kHighMemEnd,
62+
GetMmapGranularity());
6263
}
6364

6465
void *MemprofDlSymNext(const char *sym) { return dlsym(RTLD_NEXT, sym); }

compiler-rt/lib/sanitizer_common/sanitizer_common.h

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,10 @@ inline int Verbosity() {
6060
return atomic_load(&current_verbosity, memory_order_relaxed);
6161
}
6262

63-
#if SANITIZER_ANDROID
64-
inline uptr GetPageSize() {
65-
// Android post-M sysconf(_SC_PAGESIZE) crashes if called from .preinit_array.
66-
return 4096;
67-
}
68-
inline uptr GetPageSizeCached() {
69-
return 4096;
70-
}
63+
#if SANITIZER_ANDROID && !defined(__aarch64__)
64+
// 32-bit Android only has 4k pages.
65+
inline uptr GetPageSize() { return 4096; }
66+
inline uptr GetPageSizeCached() { return 4096; }
7167
#else
7268
uptr GetPageSize();
7369
extern uptr PageSizeCached;
@@ -77,6 +73,7 @@ inline uptr GetPageSizeCached() {
7773
return PageSizeCached;
7874
}
7975
#endif
76+
8077
uptr GetMmapGranularity();
8178
uptr GetMaxVirtualAddress();
8279
uptr GetMaxUserVirtualAddress();
@@ -91,6 +88,7 @@ void GetThreadStackAndTls(bool main, uptr *stk_addr, uptr *stk_size,
9188

9289
// Memory management
9390
void *MmapOrDie(uptr size, const char *mem_type, bool raw_report = false);
91+
9492
inline void *MmapOrDieQuietly(uptr size, const char *mem_type) {
9593
return MmapOrDie(size, mem_type, /*raw_report*/ true);
9694
}
@@ -139,7 +137,8 @@ void UnmapFromTo(uptr from, uptr to);
139137
// shadow_size_bytes bytes on the right, which on linux is mapped no access.
140138
// The high_mem_end may be updated if the original shadow size doesn't fit.
141139
uptr MapDynamicShadow(uptr shadow_size_bytes, uptr shadow_scale,
142-
uptr min_shadow_base_alignment, uptr &high_mem_end);
140+
uptr min_shadow_base_alignment, uptr &high_mem_end,
141+
uptr granularity);
143142

144143
// Let S = max(shadow_size, num_aliases * alias_size, ring_buffer_size).
145144
// Reserves 2*S bytes of address space to the right of the returned address and

compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1136,7 +1136,7 @@ uptr GetMaxUserVirtualAddress() {
11361136
return addr;
11371137
}
11381138

1139-
# if !SANITIZER_ANDROID
1139+
# if !SANITIZER_ANDROID || defined(__aarch64__)
11401140
uptr GetPageSize() {
11411141
# if SANITIZER_LINUX && (defined(__x86_64__) || defined(__i386__)) && \
11421142
defined(EXEC_PAGESIZE)
@@ -1155,7 +1155,7 @@ uptr GetPageSize() {
11551155
return sysconf(_SC_PAGESIZE); // EXEC_PAGESIZE may not be trustworthy.
11561156
# endif
11571157
}
1158-
# endif // !SANITIZER_ANDROID
1158+
# endif
11591159

11601160
uptr ReadBinaryName(/*out*/ char *buf, uptr buf_len) {
11611161
# if SANITIZER_SOLARIS

compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -995,9 +995,8 @@ void UnmapFromTo(uptr from, uptr to) {
995995
}
996996

997997
uptr MapDynamicShadow(uptr shadow_size_bytes, uptr shadow_scale,
998-
uptr min_shadow_base_alignment,
999-
UNUSED uptr &high_mem_end) {
1000-
const uptr granularity = GetMmapGranularity();
998+
uptr min_shadow_base_alignment, UNUSED uptr &high_mem_end,
999+
uptr granularity) {
10011000
const uptr alignment =
10021001
Max<uptr>(granularity << shadow_scale, 1ULL << min_shadow_base_alignment);
10031002
const uptr left_padding =

compiler-rt/lib/sanitizer_common/sanitizer_mac.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1188,8 +1188,8 @@ uptr GetMaxVirtualAddress() {
11881188
}
11891189

11901190
uptr MapDynamicShadow(uptr shadow_size_bytes, uptr shadow_scale,
1191-
uptr min_shadow_base_alignment, uptr &high_mem_end) {
1192-
const uptr granularity = GetMmapGranularity();
1191+
uptr min_shadow_base_alignment, uptr &high_mem_end,
1192+
uptr granularity) {
11931193
const uptr alignment =
11941194
Max<uptr>(granularity << shadow_scale, 1ULL << min_shadow_base_alignment);
11951195
const uptr left_padding =

compiler-rt/lib/sanitizer_common/sanitizer_win.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -384,9 +384,8 @@ bool DontDumpShadowMemory(uptr addr, uptr length) {
384384
}
385385

386386
uptr MapDynamicShadow(uptr shadow_size_bytes, uptr shadow_scale,
387-
uptr min_shadow_base_alignment,
388-
UNUSED uptr &high_mem_end) {
389-
const uptr granularity = GetMmapGranularity();
387+
uptr min_shadow_base_alignment, UNUSED uptr &high_mem_end,
388+
uptr granularity) {
390389
const uptr alignment =
391390
Max<uptr>(granularity << shadow_scale, 1ULL << min_shadow_base_alignment);
392391
const uptr left_padding =

0 commit comments

Comments
 (0)