Module: Mesa
Branch: main
Commit: fd1e36a044bb2a23356df025fff16f015ad34130
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=fd1e36a044bb2a23356df025fff16f015ad34130

Author: Marek Olšák <marek.ol...@amd.com>
Date:   Sun Jan  7 17:00:35 2024 -0500

winsys/amdgpu: represent IB_MAX_SUBMIT_DWORDS in bytes

Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-pra...@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27060>

---

 src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 6 +++---
 src/gallium/winsys/amdgpu/drm/amdgpu_cs.h | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c 
b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
index 7243ff75dc9..9218887c502 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
@@ -793,8 +793,8 @@ static bool amdgpu_get_new_ib(struct amdgpu_winsys *ws,
 
    if (!cs->has_chaining) {
       ib_size = MAX2(ib_size,
-                     4 * MIN2(util_next_power_of_two(main_ib->max_ib_size_dw),
-                              IB_MAX_SUBMIT_DWORDS));
+                     MIN2(4 * util_next_power_of_two(main_ib->max_ib_size_dw),
+                          IB_MAX_SUBMIT_BYTES));
    }
 
    main_ib->max_ib_size_dw = main_ib->max_ib_size_dw - main_ib->max_ib_size_dw 
/ 32;
@@ -1074,7 +1074,7 @@ static bool amdgpu_cs_check_space(struct radeon_cmdbuf 
*rcs, unsigned dw)
 
    unsigned projected_size_dw = rcs->prev_dw + rcs->current.cdw + dw;
 
-   if (projected_size_dw > IB_MAX_SUBMIT_DWORDS)
+   if (projected_size_dw * 4 > IB_MAX_SUBMIT_BYTES)
       return false;
 
    if (rcs->current.max_dw - rcs->current.cdw >= dw)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h 
b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h
index 0ad4e62831b..2084a48d6b7 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h
@@ -16,7 +16,7 @@
  * waiting for buffers and fences. Proof:
  *   http://www.phoronix.com/scan.php?page=article&item=mesa-111-si&num=1
  */
-#define IB_MAX_SUBMIT_DWORDS (20 * 1024)
+#define IB_MAX_SUBMIT_BYTES (80 * 1024)
 
 struct amdgpu_ctx {
    struct pipe_reference reference;

Reply via email to