zcfh opened a new pull request, #3284: URL: https://github.com/apache/brpc/pull/3284
### What problem does this PR solve? Issue Number: Problem Summary: ### What is changed and the side effects? 在编译器，开启pgo优化后，task_group.cpp 中，`tls_unique_user_ptr`/`tls_task_group` 的访问，还会有异常。 #### 具体原因 在task_group 中，包含下面的调用链，其中 ready_to_run_in_worker 是间接调用 `task_runner -> ending_sched -> sched_to -*-> ready_to_run_in_worker` 在开启pgo优化的场景下，存在这样的问题， 1. 当编译器识别到 `ending_sched`\`sched_to`，是个热函数时，会调高这两个函数的inline阈值，此时 sched_to 和 ending_sched 会被inline，此时 `void* saved_unique_user_ptr = tls_unique_user_ptr;` 这个关于 tls_unique_user_ptr 的访问就会有异常。 2. 在pgo的作用下，识别的 ready_to_run_in_worker 的间接调用是热的，会进行间接调用提升的优化，此时 ready_to_run_in_worker 也会被inline到 sched_to，相应的 ready_to_run_in_worker 里包含的tls变量的访问也会触发一次。 #### 复现途径 由于Autofdo优化，需要profile，我们的复现场景profile较为复杂无法直接提供。一个简单的复现方式是，使用 `-mllvm --inline-threshold=10000`的编译选项，手动调高inline的阈值。 编译器版本：17.0.6 ### 总结 对于tls变量的访问，原先在函数的"**开头**"部分，在函数不被inline的情况下，是不会发生tls的访问异常的。但是类似Autofdo这类的 [profile-guided-optimization](https://clang.llvm.org/docs/UsersManual.html#profile-guided-optimization)，会挖掘一些潜在的inline机会。所以这些发生在函数"开头"部分的，tls变量访问，仍需通过宏来禁用优化。 ### 补充 此外，还发现了一处调用链， 这里也会出现把tls基地址提前缓存起来的问题，不过没对我们的服务产生影响，可能需要你们评估一下，是否需要禁用优化。 ``` g->_control->_nbthreads << -1; inline Reducer<T, Op, InvOp>& Reducer<T, Op, InvOp>::operator<<( get_or_create_tls_agent get_tls_agent ``` Changed: `ready_to_run_in_worker` \ `ready_to_run_in_worker_ignoresignal` \ `sched_to` 中对tls变量的访问，都改为宏`BAIDU_GET_VOLATILE_THREAD_LOCAL` --- ### Check List: - Please make sure your changes are compilable. - When providing us with a new feature, it is best to add related tests. - Please follow [Contributor Covenant Code of Conduct](https://github.com/apache/brpc/blob/master/CODE_OF_CONDUCT.md). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
