[Tsan] Do not sanitize memset() and other functions during initialization

[kutuzov.viktor.84]

Summary

We need these to suppress to pass initialization on FreeBSD 11.

[kcc]

Is this fix testable? or is it covered by existing tests?

lib/tsan/rtl/tsan_interceptors.cc

667

I would prefer to define COMMON_INTERCEPTOR_NOTHING_IS_INITIALIZED above this line and use COMMON_INTERCEPTOR_NOTHING_IS_INITIALIZED here
and below instead of if(cur_thread()->is_inited)

[kutuzov.viktor.84]

Updated in D8544.

lib/tsan/rtl/tsan_interceptors.cc

 TSAN_INTERCEPTOR(uptr, strlen, const char *s) {
   SCOPED_TSAN_INTERCEPTOR(strlen, s);
   uptr len = internal_strlen(s);
   MemoryAccessRange(thr, pc, (uptr)s, len + 1, false);
   return len;
 }
 
 TSAN_INTERCEPTOR(void*, memset, void *dst, int v, uptr size) {
+  // On FreeBSD we get here from libthr internals on thread initialization.
+  if (cur_thread()->is_inited) {
     SCOPED_TSAN_INTERCEPTOR(memset, dst, v, size);
     MemoryAccessRange(thr, pc, (uptr)dst, size, true);
+  }
   return internal_memset(dst, v, size);
 }
 
 TSAN_INTERCEPTOR(void*, memcpy, void *dst, const void *src, uptr size) {
   // On FreeBSD we get here from libthr internals on thread initialization.
   if (cur_thread()->is_inited) {
     SCOPED_TSAN_INTERCEPTOR(memcpy, dst, src, size);
     MemoryAccessRange(thr, pc, (uptr)dst, size, true);

 #define COMMON_INTERCEPTOR_MUTEX_REPAIR(ctx, m) \
   MutexRepair(((TsanInterceptorContext *)ctx)->thr, \
             ((TsanInterceptorContext *)ctx)->pc, (uptr)m)
 
 #define COMMON_INTERCEPTOR_HANDLE_RECVMSG(ctx, msg) \
   HandleRecvmsg(((TsanInterceptorContext *)ctx)->thr, \
       ((TsanInterceptorContext *)ctx)->pc, msg)
 
+#define COMMON_INTERCEPTOR_NOTHING_IS_INITIALIZED \
+  (!cur_thread()->is_inited)
+
 #include "sanitizer_common/sanitizer_common_interceptors.inc"
 
 #define TSAN_SYSCALL() \
   ThreadState *thr = cur_thread(); \
   if (thr->ignore_interceptors) \
     return; \
   ScopedSyscall scoped_syscall(thr) \
 /**/