* Resolve and test race between OnReadDone(ok=false) and IsCancelled
* Fix retry case
* Fix health check case
* Address reviewer comments.
* ADD TODO requested by markdroth
This is a trivial change and results in better code generation,
because it marks the trace path unlikely.
The changes are all mechanical, except I did some minor edits in
two macros in call_stack.h and http to apply best practices.
alignment options (for cache-alignment).
We shrink by:
1) Removing an unnecessary zone pointer.
2) Replacing gpr_mu (40 bytes when using pthread_mutex_t) with
std::atomic_flag.
We also header-inline the fastpath alloc (ie. when not doing a zone
alloc) and move the malloc() for a zone alloc outside of the mutex
critical zone, which allows us to replace the mutex with a spinlock.
We also cache-align created arenas.
src/core. exec_ctx is now a thread_local pointer of type ExecCtx instead of
grpc_exec_ctx which is initialized whenever ExecCtx is instantiated. ExecCtx
also keeps track of the previous exec_ctx so that nesting of exec_ctx is
allowed. This means that there is only one exec_ctx being used at any
time. Also, grpc_exec_ctx_finish is called in the destructor of the
object, and the previous exec_ctx is restored to avoid breaking current
functionality. The code still explicitly calls grpc_exec_ctx_finish
because removing all such instances causes the code to break.
- trace system is now thread safe when run with TSAN
- fix a race in client_auth_filter.c
- allow timer manager to run in single threaded mode for fuzzers
Vijay Pai
Esun Kim
Mark D. Roth
Yash Tibrewal
Soheil Hassas Yeganeh
Arjun Roy
Yihua Zhang
ncteisen
Hope Casey-Allen
Juanli Shen
Juanli Shen
Alexander Polcyn
Muxi Yan
Craig Tiller
yang-g
Jan Tattermusch
David Garcia Quintas
murgatroid99