Mesa (master): radeon/vcn : Corrected dpb_size calculation for VP9_2

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Nov 9 19:36:02 UTC 2020


Module: Mesa
Branch: master
Commit: 956228da3a11a83d938c929415efb375ffc2ca1f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=956228da3a11a83d938c929415efb375ffc2ca1f

Author: SureshGuttula <suresh.guttula at amd.com>
Date:   Fri Nov  6 23:53:28 2020 +0530

radeon/vcn : Corrected dpb_size calculation for VP9_2

Currently dpb_size for VP9 profile0 and profile2 is same eventhough
for profile2 dpb_size is  multiplied by extra 3/2 and we are
seeing VM_L2_PROTECTION_FAULT error and ring vcn_dec timeout because
of less dpb_size for VP9_2.

This patch will correct dpb_size for VP9_2 and fixes the issue.

Signed-off-by: SureshGuttula <suresh.guttula at amd.com>
Reviewed-by: Leo Liu <leo.liu at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7480>

---

 src/gallium/drivers/radeon/radeon_vcn_dec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/drivers/radeon/radeon_vcn_dec.c b/src/gallium/drivers/radeon/radeon_vcn_dec.c
index 15781d3036e..18b64433dc9 100644
--- a/src/gallium/drivers/radeon/radeon_vcn_dec.c
+++ b/src/gallium/drivers/radeon/radeon_vcn_dec.c
@@ -1401,7 +1401,7 @@ static unsigned calc_dpb_size(struct radeon_decoder *dec)
                     : (4096 * 3000 * 3 / 2) * max_references;
 
       if (dec->base.profile == PIPE_VIDEO_PROFILE_VP9_PROFILE2)
-         dpb_size *= (3 / 2);
+         dpb_size = dpb_size * 3 / 2;
       break;
 
    case PIPE_VIDEO_FORMAT_JPEG:



More information about the mesa-commit mailing list