From patchwork Wed Dec 27 04:16:57 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Chen, Wenbin" X-Patchwork-Id: 45336 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:6623:b0:194:e134:edd4 with SMTP id n35csp3406857pzh; Tue, 26 Dec 2023 20:17:28 -0800 (PST) X-Google-Smtp-Source: AGHT+IGIhlxsMqbTiC6tfJGH+qtawmfKZ5F7fuLoBp6Hr1xuj3i2eLCiMkotJ3P9RtnWIZA/rEDl X-Received: by 2002:a17:906:c183:b0:a23:619b:d324 with SMTP id g3-20020a170906c18300b00a23619bd324mr2738619ejz.85.1703650648726; Tue, 26 Dec 2023 20:17:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703650648; cv=none; d=google.com; s=arc-20160816; b=A0KaMT30C7kpmjrqIBfhrMUF5PJcyrToV4f3Q1NYnsNGGjV2Yp/IgqMfDXRGDyi+5s 8021i/gSUp0eNqyeq15NR94mcC5wKYYJAn2eQghizEBaVXQ2R43K8MgmNerroPU4Wx44 LFXFgDYUBCwvm5wowgRdM/Hxh9x7FpNiYQvyLNTmR82yGOhvARYP/MY5tBx9KCqjBBXH 1EEX3630ME0Piyre+KLZOg2fNluFxpoI7IGadATi1QvuFcTH1azMfOwNjX1jH8I8JPnf I1e/iZm1F9rh0Z+j5Eu/hooilRnWuLWnbNNlpiVBPR2aSKu52+D82h38Fk+DdUQBB18s uceg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:mime-version:message-id:date:to:from :dkim-signature:delivered-to; bh=NEAyPn4UDzqa0/ekOJYRYmT5ssV1nhtGUCP4wA2dpG0=; fh=YOA8vD9MJZuwZ71F/05pj6KdCjf6jQRmzLS+CATXUQk=; b=nz44DQK+pOZ8s82k0+waCQFWPC/COVo+P3CjGuM8fTDGs303La2W1YMg7X3ME7m20G /sOu9dFpMUvcN6qExDtM+RX7qWRK7z3defJP9Nc0QnK2vfswuDHgjm4AjIKZOvsYJ49+ dTZ4iu3nX9hL3lFWJpCbQjaSCrBIMF2dYJ5Fv1aPMPV/EkQSIM9ajx8aG4K5jIMqKlfb l1qB9mS2kWDHmrxXU6k2Yh5U/rcovM9guYhGW7wCbJvGpFUGe4wJ82SWgobp4Rla04cL rA0csTMxj3g8B+dTHTZKvo36O/zthnUwGuraER/i1IQBLXzJUQbs2+BFEa8Lyg5dL2TQ 2HZg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@intel.com header.s=Intel header.b=a1clnx5+; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id kg25-20020a17090776f900b00a1f801c0c4bsi5819318ejc.893.2023.12.26.20.17.28; Tue, 26 Dec 2023 20:17:28 -0800 (PST) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@intel.com header.s=Intel header.b=a1clnx5+; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 38B6F68CB2D; Wed, 27 Dec 2023 06:17:24 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D7D2968CB75 for ; Wed, 27 Dec 2023 06:17:16 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1703650643; x=1735186643; h=from:to:subject:date:message-id:mime-version: content-transfer-encoding; bh=YekNQE4vUqFQktnfM5QzSGmMXqjPGHirrwZIQV92gQw=; b=a1clnx5+Fl4pauz+CqmUwUA6yb8Fa9/PrZ+jQtZyEReKLhDXMKIX+8IL Yl/DuaCk/bR8qoM/pqhtxsenAd5nZqqpZpINQH6wFl1yh2vAgBK3ereV/ lS1R2EL0Gh0Xr1LkcndTjyPvR0D7huzTMcVwt3rASYBiTMSNo9J7OCeJb t+5Srl1t78jkt/9nipnA924hTfs3xlZX9LS3OreMLW5O00dQbfdaPaKIV gahAUCyuIuXK5Z9L2Qb9JxNz6KL5wFmSqD8KMXrtN3ePA53XYcoiDh1kL N2XJIoNb1kMbcF5XESOkM6Xie5Y2GP/kMuieM5FHRBlZqnKgCHEmAxY51 Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10935"; a="15082288" X-IronPort-AV: E=Sophos;i="6.04,308,1695711600"; d="scan'208";a="15082288" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Dec 2023 20:17:02 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10935"; a="848581532" X-IronPort-AV: E=Sophos;i="6.04,308,1695711600"; d="scan'208";a="848581532" Received: from wenbin-z390-aorus-ultra.sh.intel.com ([10.239.156.43]) by fmsmga004.fm.intel.com with ESMTP; 26 Dec 2023 20:16:59 -0800 From: wenbin.chen-at-intel.com@ffmpeg.org To: ffmpeg-devel@ffmpeg.org Date: Wed, 27 Dec 2023 12:16:57 +0800 Message-Id: <20231227041658.392174-1-wenbin.chen@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 1/2] libavfilter/dnn_backend_openvino: Add dynamic output support X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: 8FvDNbwZxOfi From: Wenbin Chen Add dynamic outputs support. Some models don't have fixed output size. Its size changes according to result. Now openvino can run these kinds of models. Signed-off-by: Wenbin Chen --- libavfilter/dnn/dnn_backend_openvino.c | 134 +++++++++++-------------- 1 file changed, 59 insertions(+), 75 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c index 671a995c70..e207d44584 100644 --- a/libavfilter/dnn/dnn_backend_openvino.c +++ b/libavfilter/dnn/dnn_backend_openvino.c @@ -219,31 +219,26 @@ static int fill_model_input_ov(OVModel *ov_model, OVRequestItem *request) task = lltask->task; #if HAVE_OPENVINO2 - if (!ov_model_is_dynamic(ov_model->ov_model)) { - if (ov_model->input_port) { - ov_output_const_port_free(ov_model->input_port); - ov_model->input_port = NULL; - } - status = ov_model_const_input_by_name(ov_model->ov_model, task->input_name, &ov_model->input_port); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); - return ov2_map_error(status, NULL); - } - status = ov_const_port_get_shape(ov_model->input_port, &input_shape); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); - return ov2_map_error(status, NULL); - } - dims = input_shape.dims; - status = ov_port_get_element_type(ov_model->input_port, &precision); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port data type.\n"); - ov_shape_free(&input_shape); - return ov2_map_error(status, NULL); - } - } else { - avpriv_report_missing_feature(ctx, "Do not support dynamic model."); - return AVERROR(ENOSYS); + if (ov_model->input_port) { + ov_output_const_port_free(ov_model->input_port); + ov_model->input_port = NULL; + } + status = ov_model_const_input_by_name(ov_model->ov_model, task->input_name, &ov_model->input_port); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); + return ov2_map_error(status, NULL); + } + status = ov_const_port_get_shape(ov_model->input_port, &input_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); + return ov2_map_error(status, NULL); + } + dims = input_shape.dims; + status = ov_port_get_element_type(ov_model->input_port, &precision); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port data type.\n"); + ov_shape_free(&input_shape); + return ov2_map_error(status, NULL); } input.height = dims[1]; input.width = dims[2]; @@ -1049,30 +1044,22 @@ static int get_input_ov(void *model, DNNData *input, const char *input_name) ov_element_type_e precision; int64_t* dims; ov_status_e status; - if (!ov_model_is_dynamic(ov_model->ov_model)) { - status = ov_model_const_input_by_name(ov_model->ov_model, input_name, &ov_model->input_port); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); - return ov2_map_error(status, NULL); - } - - status = ov_const_port_get_shape(ov_model->input_port, &input_shape); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); - return ov2_map_error(status, NULL); - } - dims = input_shape.dims; - - status = ov_port_get_element_type(ov_model->input_port, &precision); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to get input port data type.\n"); - return ov2_map_error(status, NULL); - } - } else { - avpriv_report_missing_feature(ctx, "Do not support dynamic model now."); - return AVERROR(ENOSYS); + status = ov_model_const_input_by_name(ov_model->ov_model, input_name, &ov_model->input_port); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); + return ov2_map_error(status, NULL); } - + status = ov_port_get_element_type(ov_model->input_port, &precision); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port data type.\n"); + return ov2_map_error(status, NULL); + } + status = ov_const_port_get_shape(ov_model->input_port, &input_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to get input port shape.\n"); + return ov2_map_error(status, NULL); + } + dims = input_shape.dims; if (dims[1] <= 3) { // NCHW input->channels = dims[1]; input->height = input_resizable ? -1 : dims[2]; @@ -1083,7 +1070,7 @@ static int get_input_ov(void *model, DNNData *input, const char *input_name) input->channels = dims[3]; } input->dt = precision_to_datatype(precision); - + ov_shape_free(&input_shape); return 0; #else char *model_input_name = NULL; @@ -1267,34 +1254,31 @@ static int get_output_ov(void *model, const char *input_name, int input_width, i #if HAVE_OPENVINO2 if (ctx->options.input_resizable) { - if (!ov_model_is_dynamic(ov_model->ov_model)) { - status = ov_partial_shape_create(4, dims, &partial_shape); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed create partial shape.\n"); - return ov2_map_error(status, NULL); - } - status = ov_const_port_get_shape(ov_model->input_port, &input_shape); - input_shape.dims[2] = input_height; - input_shape.dims[3] = input_width; - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed create shape for model input resize.\n"); - return ov2_map_error(status, NULL); - } + status = ov_partial_shape_create(4, dims, &partial_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to create partial shape.\n"); + return ov2_map_error(status, NULL); + } + status = ov_const_port_get_shape(ov_model->input_port, &input_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to create shape for model input resize.\n"); + return ov2_map_error(status, NULL); + } + input_shape.dims[2] = input_height; + input_shape.dims[3] = input_width; - status = ov_shape_to_partial_shape(input_shape, &partial_shape); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed create partial shape for model input resize.\n"); - return ov2_map_error(status, NULL); - } + status = ov_shape_to_partial_shape(input_shape, &partial_shape); + ov_shape_free(&input_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to create partial shape for model input resize.\n"); + return ov2_map_error(status, NULL); + } - status = ov_model_reshape_single_input(ov_model->ov_model, partial_shape); - if (status != OK) { - av_log(ctx, AV_LOG_ERROR, "Failed to reszie model input.\n"); - return ov2_map_error(status, NULL); - } - } else { - avpriv_report_missing_feature(ctx, "Do not support dynamic model."); - return AVERROR(ENOTSUP); + status = ov_model_reshape_single_input(ov_model->ov_model, partial_shape); + ov_partial_shape_free(&partial_shape); + if (status != OK) { + av_log(ctx, AV_LOG_ERROR, "Failed to reszie model input.\n"); + return ov2_map_error(status, NULL); } }