Re: [PATCH 6.1.y] media: mediatek: vcodec: Fix H264 multi stateless decoder smatch warning

[Sasha Levin <sashal@kernel.org>]

[ Sasha's backport helper bot ]

Hi,

✅ All tests passed successfully. No issues detected.
No action required from the submitter.

The upstream commit SHA1 provided is correct: 9be85491619f1953b8a29590ca630be571941ffa

WARNING: Author mismatch between patch and upstream commit:
Backport author: jetlan9@163.com
Commit author: Yunfei Dong<yunfei.dong@mediatek.com>


Status in newer kernel trees:
6.13.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Present (different SHA1: 588bcce9e64c)

Note: The patch differs from the upstream commit:
---
1:  9be85491619f1 ! 1:  dffd0505449e8 media: mediatek: vcodec: Fix H264 multi stateless decoder smatch warning
    @@ Metadata
      ## Commit message ##
         media: mediatek: vcodec: Fix H264 multi stateless decoder smatch warning
     
    +    [ Upstream commit 9be85491619f1953b8a29590ca630be571941ffa ]
    +
         Fix a smatch static checker warning on vdec_h264_req_multi_if.c.
         Which leads to a kernel crash when fb is NULL.
     
    @@ Commit message
         Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
         Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>
         Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
    +    [ drivers/media/platform/mediatek/vcodec/decoder/vdec/vdec_h264_req_multi_if.c
    +    is renamed from drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c
    +    since 0934d3759615 ("media: mediatek: vcodec: separate decoder and encoder").
    +    The path is changed accordingly to apply the patch on 6.1.y. ]
    +    Signed-off-by: Wenshan Lan <jetlan9@163.com>
     
    - ## drivers/media/platform/mediatek/vcodec/decoder/vdec/vdec_h264_req_multi_if.c ##
    -@@ drivers/media/platform/mediatek/vcodec/decoder/vdec/vdec_h264_req_multi_if.c: static int vdec_h264_slice_single_decode(void *h_vdec, struct mtk_vcodec_mem *bs
    + ## drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c ##
    +@@ drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c: static int vdec_h264_slice_single_decode(void *h_vdec, struct mtk_vcodec_mem *bs
      		return vpu_dec_reset(vpu);
      
      	fb = inst->ctx->dev->vdec_pdata->get_cap_buffer(inst->ctx);
     +	if (!fb) {
    -+		mtk_vdec_err(inst->ctx, "fb buffer is NULL");
    ++		mtk_vcodec_err(inst, "fb buffer is NULL");
     +		return -ENOMEM;
     +	}
     +
    @@ drivers/media/platform/mediatek/vcodec/decoder/vdec/vdec_h264_req_multi_if.c: st
     -	c_fb_dma = fb ? (u64)fb->base_c.dma_addr : 0;
     +	y_fb_dma = fb->base_y.dma_addr;
     +	c_fb_dma = fb->base_c.dma_addr;
    - 	mtk_vdec_debug(inst->ctx, "[h264-dec] [%d] y_dma=%llx c_dma=%llx",
    - 		       inst->ctx->decoded_frame_cnt, y_fb_dma, c_fb_dma);
    + 	mtk_vcodec_debug(inst, "[h264-dec] [%d] y_dma=%llx c_dma=%llx",
    + 			 inst->ctx->decoded_frame_cnt, y_fb_dma, c_fb_dma);
      
---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-6.1.y        |  Success    |  Success   |