AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

venus: increase busy wait order in vn_relax

Browse Source 
This is mainly to workaround a platform issue that has huge sleep
penalty, which could lead to a timeout if the small synchronous queries
are going to sleep.

This change adjusts the warn and abort order correspondingly so that to
match prior timing.

Signed-off-by: Yiwei Zhang <zzyiwei@chromium.org>
Reviewed-by: Ryan Neph <ryanneph@google.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17898>
This commit is contained in:
Yiwei Zhang
2022-08-04 19:06:17 +00:00
parent 1911dc5abb
commit ecd5509969
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/virtio/vulkan/vn_common.c
+5 -5
View File
@@ -120,10 +120,10 @@ vn_relax(uint32_t *iter, const char *reason)
* base_sleep_us microseconds for the same number of times. After that,
* keep doubling both sleep length and count.
*/
const uint32_t busy_wait_order = 4;
const uint32_t busy_wait_order = 8;
const uint32_t base_sleep_us = vn_env.relax_base_sleep_us;
const uint32_t warn_order = 12;
const uint32_t abort_order = 14;
const uint32_t warn_order = 14;
const uint32_t abort_order = 16;
(*iter)++;
if (*iter < (1 << busy_wait_order)) {
@@ -131,8 +131,8 @@ vn_relax(uint32_t *iter, const char *reason)
return;
}
/* warn occasionally if we have slept at least 1.28ms for 2048 times (plus
* another 2047 shorter sleeps)
/* warn occasionally if we have slept at least 0.32ms for 8192 times (plus
* another 8191 shorter sleeps)
*/
if (unlikely(*iter % (1 << warn_order) == 0)) {
vn_log(NULL, "stuck in %s wait with iter at %d", reason, *iter);