iii marked an inline comment as done.
iii added inline comments.

================
Comment at: compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp:781
+      // TSAN. Act as if we ran out of memory.
+      internal_munmap(res, sz);
+      errno = errno_ENOMEM;
----------------
dvyukov wrote:
> TSan runtime is supposed to mprotect all ranges that are inaccessible to the 
> application. How does it happen that kernel still returns such an address? Do 
> we fail to mprotect all ranges on s390? If so should that be fixed instead?
> I am concerned that if kernel can return unsupported addresses, it probably 
> can returns them very early before exhausting all supported addresses (e.g. 
> always return unsupported from the start).
> Or is it that we mprotect everything, but kernel still somehow stomps on it? 
> If so can this address overlap with, say, tsan shadow? If yes, them the 
> munmap will unmap our shadow which is not good.
The common code in CheckAndProtect() didn't mprotect the address space "tail" - 
I've added a special case for that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105629/new/

https://reviews.llvm.org/D105629

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to