fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-03 18:38:11 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Revert "radeonsi/vcn: AV1 skip the redundant bs resize"

Browse source 
Currently ffmpeg has a bug in VAAPI AV1 decode that in some cases
it submits the same slice data buffer as many times as there is tiles.
However, in other cases it behaves correctly and all slice data buffers
contain different parts of bitstream to decode which this change breaks.

Now that the va frontend is passing correct offsets, this fixes decoding
AV1-TEST-VECTORS/av1-1-b8-22-svc-L1T2 with ffmpeg.

This reverts commit e6701f7231.

Cc: mesa-stable
Reviewed-by: Leo Liu <leo.liu@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28960>
(cherry picked from commit 88dfe04b08)
This commit is contained in:
David Rosca 2024-04-27 19:21:40 +02:00 • committed by Eric Engestrom
parent e8f360c32c
commit 8b6095c17e
2 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.pick_status.json
View file

@ -414,7 +414,7 @@
"description": "Revert \"radeonsi/vcn: AV1 skip the redundant bs resize\"",
"nominated": true,
"nomination_type": 0,
"resolution": 0,
"resolution": 1,
"main_sha": null,
"because_sha": null,
"notes": null

3
src/gallium/drivers/radeonsi/radeon_vcn_dec.c
View file

@ -2873,9 +2873,6 @@ static void radeon_dec_decode_bitstream(struct pipe_video_codec *decoder,
if (!dec->bs_ptr)
return;
if (dec->bs_size && dec->stream_type == RDECODE_CODEC_AV1)
return;
unsigned long total_bs_size = dec->bs_size;
for (i = 0; i < num_buffers; ++i)
total_bs_size += sizes[i];