diff mbox series

[FFmpeg-devel,V2,3/3] lavfi/dnn: add error info for TF backend filling task failure

Message ID 20230324073040.16310-3-ting.fu@intel.com
State New
Headers show
Series [FFmpeg-devel,V2,1/3] lavfi/dnn: fix corruption when TF backend infer failed | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Ting Fu March 24, 2023, 7:30 a.m. UTC 
Signed-off-by: Ting Fu <ting.fu@intel.com>
---
 libavfilter/dnn/dnn_backend_tf.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Guo, Yejun March 25, 2023, 10:54 a.m. UTC | #1 
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Ting Fu
> Sent: Friday, March 24, 2023 3:31 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH V2 3/3] lavfi/dnn: add error info for TF
> backend filling task failure
> 
> Signed-off-by: Ting Fu <ting.fu@intel.com>
> ---
>  libavfilter/dnn/dnn_backend_tf.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavfilter/dnn/dnn_backend_tf.c
> b/libavfilter/dnn/dnn_backend_tf.c
> index fe1baa2e5e..3d372a5628 100644
> --- a/libavfilter/dnn/dnn_backend_tf.c
> +++ b/libavfilter/dnn/dnn_backend_tf.c
> @@ -1152,6 +1152,7 @@ int ff_dnn_execute_model_tf(const DNNModel
> *model, DNNExecBaseParams *exec_param
> 
>      ret = ff_dnn_fill_task(task, exec_params, tf_model, ctx->options.async, 1);
>      if (ret != 0) {
> +        av_log(ctx, AV_LOG_ERROR, "Fill task with invalid
> + parameter(s).\n");
>          av_freep(&task);
>          return ret;
>      }
> --
This patch set LGTM, will push, thanks.
diff mbox series

Patch

diff --git a/libavfilter/dnn/dnn_backend_tf.c b/libavfilter/dnn/dnn_backend_tf.c
index fe1baa2e5e..3d372a5628 100644
--- a/libavfilter/dnn/dnn_backend_tf.c
+++ b/libavfilter/dnn/dnn_backend_tf.c
@@ -1152,6 +1152,7 @@  int ff_dnn_execute_model_tf(const DNNModel *model, DNNExecBaseParams *exec_param
 
     ret = ff_dnn_fill_task(task, exec_params, tf_model, ctx->options.async, 1);
     if (ret != 0) {
+        av_log(ctx, AV_LOG_ERROR, "Fill task with invalid parameter(s).\n");
         av_freep(&task);
         return ret;
     }