linux-kernel - Re: [PATCH v2] media: mediatek: vcodec: add decoder command to support stateless decoder

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2aefd938-f70a-4133-a3a8-9c96464c9f5f@collabora.com>
Date: Thu, 9 May 2024 11:54:31 +0200
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
To: Yunfei Dong <yunfei.dong@...iatek.com>,
 Nícolas F . R . A . Prado <nfraprado@...labora.com>,
 Nicolas Dufresne <nicolas.dufresne@...labora.com>,
 Hans Verkuil <hverkuil-cisco@...all.nl>,
 Benjamin Gaignard <benjamin.gaignard@...labora.com>,
 Nathan Hebert <nhebert@...omium.org>,
 Sebastian Fricke <sebastian.fricke@...labora.com>
Cc: Hsin-Yi Wang <hsinyi@...omium.org>, Fritz Koenig <frkoenig@...omium.org>,
 Daniel Vetter <daniel@...ll.ch>, Steve Cho <stevecho@...omium.org>,
 linux-media@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-mediatek@...ts.infradead.org,
 Project_Global_Chrome_Upstream_Group@...iatek.com
Subject: Re: [PATCH v2] media: mediatek: vcodec: add decoder command to
 support stateless decoder

Il 16/03/24 09:13, Yunfei Dong ha scritto:
> The supported decoder commands are different for stateless and
> stateful architecture. Add stateless decoder commands to fix
> the v4l2-compliance test error below.
> 
> Codec ioctls:
>      VIDIOC_ENCODER_CMD returned -1 (Inappropriate ioctl for device)
>      VIDIOC_TRY_ENCODER_CMD returned -1 (Inappropriate ioctl for device)
>   test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
>      VIDIOC_G_ENC_INDEX returned -1 (Inappropriate ioctl for device)
>   test VIDIOC_G_ENC_INDEX: OK (Not Supported)
>      VIDIOC_DECODER_CMD returned -1 (Invalid argument)
>      VIDIOC_TRY_DECODER_CMD returned -1 (Invalid argument)
>      VIDIOC_TRY_DECODER_CMD returned -1 (Invalid argument)
>      fail: v4l2-test-codecs.cpp(126): ret
>   test VIDIOC_(TRY_)DECODER_CMD: FAIL
> 
> Signed-off-by: Yunfei Dong <yunfei.dong@...iatek.com>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>

Nicolas, Sebastian, anything to add?

Cheers,
Angelo

> ---
> changed with v1:
> - remove the static function prefix.
> - fix some messages not reasonable.
> ---
>   .../mediatek/vcodec/decoder/mtk_vcodec_dec.c  | 60 +++++++++++++++++--
>   1 file changed, 54 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c b/drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c
> index ba742f0e391d..c2b64a528028 100644
> --- a/drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c
> +++ b/drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c
> @@ -80,21 +80,18 @@ static struct mtk_q_data *mtk_vdec_get_q_data(struct mtk_vcodec_dec_ctx *ctx,
>   	return &ctx->q_data[MTK_Q_DATA_DST];
>   }
>   
> -static int vidioc_try_decoder_cmd(struct file *file, void *priv,
> -				struct v4l2_decoder_cmd *cmd)
> +static int stateful_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
>   {
>   	return v4l2_m2m_ioctl_try_decoder_cmd(file, priv, cmd);
>   }
>   
> -
> -static int vidioc_decoder_cmd(struct file *file, void *priv,
> -				struct v4l2_decoder_cmd *cmd)
> +static int stateful_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
>   {
>   	struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
>   	struct vb2_queue *src_vq, *dst_vq;
>   	int ret;
>   
> -	ret = vidioc_try_decoder_cmd(file, priv, cmd);
> +	ret = stateful_try_decoder_cmd(file, priv, cmd);
>   	if (ret)
>   		return ret;
>   
> @@ -128,6 +125,57 @@ static int vidioc_decoder_cmd(struct file *file, void *priv,
>   	return 0;
>   }
>   
> +static int stateless_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
> +{
> +	return v4l2_m2m_ioctl_stateless_try_decoder_cmd(file, priv, cmd);
> +}
> +
> +static int stateless_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
> +{
> +	struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
> +	int ret;
> +
> +	ret = v4l2_m2m_ioctl_stateless_try_decoder_cmd(file, priv, cmd);
> +	if (ret)
> +		return ret;
> +
> +	mtk_v4l2_vdec_dbg(3, ctx, "decoder cmd=%u", cmd->cmd);
> +	switch (cmd->cmd) {
> +	case V4L2_DEC_CMD_FLUSH:
> +		/*
> +		 * If the flag of the output buffer is equals V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF,
> +		 * this command will prevent dequeueing the capture buffer containing the last
> +		 * decoded frame. Or do nothing
> +		 */
> +		break;
> +	default:
> +		mtk_v4l2_vdec_err(ctx, "invalid stateless decoder cmd=%u", cmd->cmd);
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> +
> +static int vidioc_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
> +{
> +	struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
> +
> +	if (ctx->dev->vdec_pdata->uses_stateless_api)
> +		return stateless_try_decoder_cmd(file, priv, cmd);
> +
> +	return stateful_try_decoder_cmd(file, priv, cmd);
> +}
> +
> +static int vidioc_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
> +{
> +	struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
> +
> +	if (ctx->dev->vdec_pdata->uses_stateless_api)
> +		return stateless_decoder_cmd(file, priv, cmd);
> +
> +	return stateful_decoder_cmd(file, priv, cmd);
> +}
> +
>   void mtk_vdec_unlock(struct mtk_vcodec_dec_ctx *ctx)
>   {
>   	mutex_unlock(&ctx->dev->dec_mutex[ctx->hw_id]);