This patch adds support for dfsan on aarch64-linux with 42-bit VMA
(current default config for 64K pagesize kernels). The support is
enabled by defining the SANITIZER_AARCH64_VMA to 42 at build time
for both clang/llvm and compiler-rt. The default VMA is 39 bits.
Details
Details
Diff Detail
Diff Detail
Event Timeline
lib/dfsan/dfsan.cc | ||
---|---|---|
132 | This looks wrong with the table above? |
lib/dfsan/dfsan.cc | ||
---|---|---|
132 | In fact is the contraty, the comment needs update. I'll change it. |
lib/dfsan/dfsan.h | ||
---|---|---|
52 | Do you need the same trick with defining it to 39 when undef? Or is this getting from the same header as the other sans? |
Comment Actions
The SANITIZER_AARCH64_VMA is defined in a common header (lib/sanitizer_common/sanitizer_platform.h) used by all sanitizers.
Comment Actions
LGTM after updating the comment. Thanks!
lib/dfsan/dfsan.cc | ||
---|---|---|
132 | makes sense, this is 3 bits to the left. :) |
Do you need the same trick with defining it to 39 when undef? Or is this getting from the same header as the other sans?