[PATCH] media: verisilicon: AV1: Fix tile info buffer size
Nicolas Dufresne
nicolas.dufresne at collabora.com
Sun Jan 18 14:34:24 PST 2026
Le mercredi 14 janvier 2026 à 10:07 +0100, Benjamin Gaignard a écrit :
> Each tile info is composed of: row_sb, col_sb, start_pos
> and end_pos (4 bytes each). So the total required memory
> is AV1_MAX_TILES * 16 bytes.
> Use the correct #define to allocate the buffer and avoid
> writing tile info in non-allocated memory.
>
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard at collabora.com>
> Fixes: 727a400686a2c ("media: verisilicon: Add Rockchip AV1 decoder")
> ---
> .../media/platform/verisilicon/rockchip_vpu981_hw_av1_dec.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/platform/verisilicon/rockchip_vpu981_hw_av1_dec.c b/drivers/media/platform/verisilicon/rockchip_vpu981_hw_av1_dec.c
> index e4703bb6be7c..af854b149f35 100644
> --- a/drivers/media/platform/verisilicon/rockchip_vpu981_hw_av1_dec.c
> +++ b/drivers/media/platform/verisilicon/rockchip_vpu981_hw_av1_dec.c
> @@ -373,12 +373,12 @@ int rockchip_vpu981_av1_dec_init(struct hantro_ctx *ctx)
> return -ENOMEM;
> av1_dec->global_model.size = GLOBAL_MODEL_SIZE;
>
> - av1_dec->tile_info.cpu = dma_alloc_coherent(vpu->dev, AV1_MAX_TILES,
> + av1_dec->tile_info.cpu = dma_alloc_coherent(vpu->dev, AV1_TILE_INFO_SIZE,
> &av1_dec->tile_info.dma,
> GFP_KERNEL);
> if (!av1_dec->tile_info.cpu)
> return -ENOMEM;
> - av1_dec->tile_info.size = AV1_MAX_TILES;
> + av1_dec->tile_info.size = AV1_TILE_INFO_SIZE;
After checking the sizes, I can only conclude the we rarely have more then 8
tiles in our tests.
Reviewed-by: Nicolas Dufresne <nicolas.dufresne at collabora.com>
>
> av1_dec->film_grain.cpu = dma_alloc_coherent(vpu->dev,
> ALIGN(sizeof(struct rockchip_av1_film_grain), 2048),
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20260118/500e5b9c/attachment.sig>
More information about the linux-arm-kernel
mailing list