From patchwork Sun May 24 01:04:49 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lance Wang X-Patchwork-Id: 19826 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 67C7F449D53 for ; Sun, 24 May 2020 04:11:08 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 47E2B6882F0; Sun, 24 May 2020 04:11:08 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pl1-f196.google.com (mail-pl1-f196.google.com [209.85.214.196]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8062B6805A5 for ; Sun, 24 May 2020 04:11:01 +0300 (EEST) Received: by mail-pl1-f196.google.com with SMTP id bg4so660921plb.3 for ; Sat, 23 May 2020 18:11:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=00mBC02uOcfEJSH5LD/jKW2wcCCzjlZZTNpMijvOwhU=; b=Os7Ygd4KMQoQTdE5DB1IsKY/OoT193dv6WaLKMWvofvbA5QW+9FkMAf8xFDi23C8M8 A5F6EtfaVi/FY3WVSFE3YdLyf/ssnYsPRFy5VKyHFL9K8Hu5qsirP83gJNvFeotfKAke LmtC2WvuLyHibbsi2ORjV4ANgiNHLrT+d2JzM4aeV7eFIYZu1iw5tuOMQE9+x62kURW1 B5yrNfQB/GV41eyT7AJFGW7gJaMij7/DpEyQxmWiL8rItiTPJkQtF0j6zhkQrxcQi28C gN9YuE2FoWjgYaKLBhMaLg04n/u/SFUJXdUQPbnXVM/7dw2GzIu6wTJaB2nEw9UgSlPu pr+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=00mBC02uOcfEJSH5LD/jKW2wcCCzjlZZTNpMijvOwhU=; b=QwHRgkzpNx8zmt5e4GHg5gI5JDtF9hUgiYAvNm4HrEFUSNiEUOzqxUDToKXarGsq+w jujPEj6j7dbzUFNBQhzIepRqp+MzxaLi+3czUFCXl1PWLwXqF8Ot9VPIoJjP9RU/O/eM 4wNlrOaz1jdZCeXbxaKakijgRnmmCcZ2Al+CRmJMR1H9YraBEOd/ehnxTajrenPwnWdH CCtwzYQg9zu3d7i6hx4IWR4VMyJ1t9EBfSL7fBXd66/AZgCHL5mbmMzduf3s+MvhqJ6y x9lxAnDy90NopHW79If1W+kCMl5aN89JpLqzWL2EpV4bOAXupHXCyEEe3Ug0pV7ZQHtD yDSQ== X-Gm-Message-State: AOAM530B1EogaiN01gDApMgkCr7GuxDE3H/vUKpNjyiFS9WnPIJcXwuM vGYiuRb5Lxw/ES+D+V3vMDLAquV/ X-Google-Smtp-Source: ABdhPJxdUIPSoVhyOBq39l0JK4mbXj3yhakSS6aT5kePQtx6YCu7IG2xHfqhC//K4DrXmDL7Tn+iNA== X-Received: by 2002:a17:90a:1642:: with SMTP id x2mr7377598pje.149.1590282659352; Sat, 23 May 2020 18:10:59 -0700 (PDT) Received: from vpn2.localdomain ([161.117.202.209]) by smtp.gmail.com with ESMTPSA id gg8sm9731693pjb.39.2020.05.23.18.10.57 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 23 May 2020 18:10:58 -0700 (PDT) From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Date: Sun, 24 May 2020 09:04:49 +0800 Message-Id: <1590282291-20859-1-git-send-email-lance.lmwang@gmail.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1589900073-4183-1-git-send-email-lance.lmwang@gmail.com> References: <1589900073-4183-1-git-send-email-lance.lmwang@gmail.com> Subject: [FFmpeg-devel] [PATCH v6 1/3] avfilter/graphdump: support for the graph2dot function X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Cc: Limin Wang MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Limin Wang reuse the function from tools/graph2dot.c and add support for filter graph with graphviz DOT format for avfilter directly. So tools/graph2dot.c is duplicated and can be removed. In addition, make the options are pair-key format, now it support two key: fmt=DOT|dot filename=./outfile Signed-off-by: Limin Wang --- libavfilter/graphdump.c | 89 +++++++++++++++++++++ tools/graph2dot.c | 204 ------------------------------------------------ 2 files changed, 89 insertions(+), 204 deletions(-) delete mode 100644 tools/graph2dot.c diff --git a/libavfilter/graphdump.c b/libavfilter/graphdump.c index 79ef1a7..97d4f39 100644 --- a/libavfilter/graphdump.c +++ b/libavfilter/graphdump.c @@ -151,15 +151,104 @@ static void avfilter_graph_dump_to_buf(AVBPrint *buf, AVFilterGraph *graph) } } +static void avfilter_graph2dot_to_buf(AVBPrint *buf, AVFilterGraph *graph) +{ + int i, j; + + av_bprintf(buf, "digraph G {\n"); + av_bprintf(buf, "node [shape=box]\n"); + av_bprintf(buf, "rankdir=LR\n"); + + for (i = 0; i < graph->nb_filters; i++) { + char filter_ctx_label[128]; + const AVFilterContext *filter_ctx = graph->filters[i]; + + snprintf(filter_ctx_label, sizeof(filter_ctx_label), "%s\\n(%s)", + filter_ctx->name, + filter_ctx->filter->name); + + for (j = 0; j < filter_ctx->nb_outputs; j++) { + AVFilterLink *link = filter_ctx->outputs[j]; + if (link) { + char dst_filter_ctx_label[128]; + const AVFilterContext *dst_filter_ctx = link->dst; + + snprintf(dst_filter_ctx_label, sizeof(dst_filter_ctx_label), + "%s\\n(%s)", + dst_filter_ctx->name, + dst_filter_ctx->filter->name); + + av_bprintf(buf, "\"%s\" -> \"%s\" [ label= \"inpad:%s -> outpad:%s\\n", + filter_ctx_label, dst_filter_ctx_label, + avfilter_pad_get_name(link->srcpad, 0), + avfilter_pad_get_name(link->dstpad, 0)); + + if (link->type == AVMEDIA_TYPE_VIDEO) { + const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(link->format); + av_bprintf(buf, + "fmt:%s w:%d h:%d tb:%d/%d", + desc->name, + link->w, link->h, + link->time_base.num, link->time_base.den); + } else if (link->type == AVMEDIA_TYPE_AUDIO) { + char audio_buf[255]; + av_get_channel_layout_string(audio_buf, sizeof(audio_buf), -1, + link->channel_layout); + av_bprintf(buf, + "fmt:%s sr:%d cl:%s tb:%d/%d", + av_get_sample_fmt_name(link->format), + link->sample_rate, audio_buf, + link->time_base.num, link->time_base.den); + } + av_bprintf(buf, "\" ];\n"); + } + } + } + av_bprintf(buf, "}\n"); +} + char *avfilter_graph_dump(AVFilterGraph *graph, const char *options) { AVBPrint buf; char *dump = NULL; + int ret; + AVDictionary *dict = NULL; + AVDictionaryEntry *format = NULL; + AVDictionaryEntry *filename = NULL; + + ret = av_dict_parse_string(&dict, options, "=", ":", 0); + if (ret < 0) { + av_dict_free(&dict); + return NULL; + } + format = av_dict_get(dict, "fmt", NULL, 0); + if (format && !av_strcasecmp(format->value, "DOT")) { + av_bprint_init(&buf, 0, AV_BPRINT_SIZE_UNLIMITED); + avfilter_graph2dot_to_buf(&buf, graph); + av_bprint_finalize(&buf, &dump); + } else { av_bprint_init(&buf, 0, AV_BPRINT_SIZE_COUNT_ONLY); avfilter_graph_dump_to_buf(&buf, graph); av_bprint_init(&buf, buf.len + 1, buf.len + 1); avfilter_graph_dump_to_buf(&buf, graph); av_bprint_finalize(&buf, &dump); + } + + if (filename = av_dict_get(dict, "filename", NULL, 0)) { + FILE* file = fopen(filename->value, "w"); + if (!file) { + av_log(graph, AV_LOG_ERROR, "failed to open: %s \n", filename->value); + av_freep(&dump); + return NULL; + } + if (dump) { + fputs(dump, file); + fflush(file); + } + fclose(file); + } + + av_dict_free(&dict); return dump; } diff --git a/tools/graph2dot.c b/tools/graph2dot.c deleted file mode 100644 index d5c1e4e..0000000 --- a/tools/graph2dot.c +++ /dev/null @@ -1,204 +0,0 @@ -/* - * Copyright (c) 2008-2010 Stefano Sabatini - * - * This file is part of FFmpeg. - * - * FFmpeg is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 2.1 of the License, or (at your option) any later version. - * - * FFmpeg is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with FFmpeg; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA - */ - -#include "config.h" -#if HAVE_UNISTD_H -#include /* getopt */ -#endif -#include -#include - -#include "libavutil/channel_layout.h" -#include "libavutil/mem.h" -#include "libavutil/pixdesc.h" -#include "libavfilter/avfilter.h" - -#if !HAVE_GETOPT -#include "compat/getopt.c" -#endif - -static void usage(void) -{ - printf("Convert a libavfilter graph to a dot file.\n"); - printf("Usage: graph2dot [OPTIONS]\n"); - printf("\n" - "Options:\n" - "-i INFILE set INFILE as input file, stdin if omitted\n" - "-o OUTFILE set OUTFILE as output file, stdout if omitted\n" - "-h print this help\n"); -} - -struct line { - char data[256]; - struct line *next; -}; - -static void print_digraph(FILE *outfile, AVFilterGraph *graph) -{ - int i, j; - - fprintf(outfile, "digraph G {\n"); - fprintf(outfile, "node [shape=box]\n"); - fprintf(outfile, "rankdir=LR\n"); - - for (i = 0; i < graph->nb_filters; i++) { - char filter_ctx_label[128]; - const AVFilterContext *filter_ctx = graph->filters[i]; - - snprintf(filter_ctx_label, sizeof(filter_ctx_label), "%s\\n(%s)", - filter_ctx->name, - filter_ctx->filter->name); - - for (j = 0; j < filter_ctx->nb_outputs; j++) { - AVFilterLink *link = filter_ctx->outputs[j]; - if (link) { - char dst_filter_ctx_label[128]; - const AVFilterContext *dst_filter_ctx = link->dst; - - snprintf(dst_filter_ctx_label, sizeof(dst_filter_ctx_label), - "%s\\n(%s)", - dst_filter_ctx->name, - dst_filter_ctx->filter->name); - - fprintf(outfile, "\"%s\" -> \"%s\" [ label= \"inpad:%s -> outpad:%s\\n", - filter_ctx_label, dst_filter_ctx_label, - avfilter_pad_get_name(link->srcpad, 0), - avfilter_pad_get_name(link->dstpad, 0)); - - if (link->type == AVMEDIA_TYPE_VIDEO) { - const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(link->format); - fprintf(outfile, - "fmt:%s w:%d h:%d tb:%d/%d", - desc->name, - link->w, link->h, - link->time_base.num, link->time_base.den); - } else if (link->type == AVMEDIA_TYPE_AUDIO) { - char buf[255]; - av_get_channel_layout_string(buf, sizeof(buf), -1, - link->channel_layout); - fprintf(outfile, - "fmt:%s sr:%d cl:%s tb:%d/%d", - av_get_sample_fmt_name(link->format), - link->sample_rate, buf, - link->time_base.num, link->time_base.den); - } - fprintf(outfile, "\" ];\n"); - } - } - } - fprintf(outfile, "}\n"); -} - -int main(int argc, char **argv) -{ - const char *outfilename = NULL; - const char *infilename = NULL; - FILE *outfile = NULL; - FILE *infile = NULL; - char *graph_string = NULL; - AVFilterGraph *graph = av_mallocz(sizeof(AVFilterGraph)); - char c; - - av_log_set_level(AV_LOG_DEBUG); - - while ((c = getopt(argc, argv, "hi:o:")) != -1) { - switch (c) { - case 'h': - usage(); - return 0; - case 'i': - infilename = optarg; - break; - case 'o': - outfilename = optarg; - break; - case '?': - return 1; - } - } - - if (!infilename || !strcmp(infilename, "-")) - infilename = "/dev/stdin"; - infile = fopen(infilename, "r"); - if (!infile) { - fprintf(stderr, "Failed to open input file '%s': %s\n", - infilename, strerror(errno)); - return 1; - } - - if (!outfilename || !strcmp(outfilename, "-")) - outfilename = "/dev/stdout"; - outfile = fopen(outfilename, "w"); - if (!outfile) { - fprintf(stderr, "Failed to open output file '%s': %s\n", - outfilename, strerror(errno)); - return 1; - } - - /* read from infile and put it in a buffer */ - { - int64_t count = 0; - struct line *line, *last_line, *first_line; - char *p; - last_line = first_line = av_malloc(sizeof(struct line)); - if (!last_line) { - fprintf(stderr, "Memory allocation failure\n"); - return 1; - } - - while (fgets(last_line->data, sizeof(last_line->data), infile)) { - struct line *new_line = av_malloc(sizeof(struct line)); - if (!new_line) { - fprintf(stderr, "Memory allocation failure\n"); - return 1; - } - count += strlen(last_line->data); - last_line->next = new_line; - last_line = new_line; - } - last_line->next = NULL; - - graph_string = av_malloc(count + 1); - if (!graph_string) { - fprintf(stderr, "Memory allocation failure\n"); - return 1; - } - p = graph_string; - for (line = first_line; line->next; line = line->next) { - size_t l = strlen(line->data); - memcpy(p, line->data, l); - p += l; - } - *p = '\0'; - } - - if (avfilter_graph_parse(graph, graph_string, NULL, NULL, NULL) < 0) { - fprintf(stderr, "Failed to parse the graph description\n"); - return 1; - } - - if (avfilter_graph_config(graph, NULL) < 0) - return 1; - - print_digraph(outfile, graph); - fflush(outfile); - - return 0; -} From patchwork Sun May 24 01:04:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lance Wang X-Patchwork-Id: 19827 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 4037F449D53 for ; Sun, 24 May 2020 04:11:10 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 25E1A68A3AF; Sun, 24 May 2020 04:11:10 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pf1-f193.google.com (mail-pf1-f193.google.com [209.85.210.193]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 57DA46898AE for ; Sun, 24 May 2020 04:11:03 +0300 (EEST) Received: by mail-pf1-f193.google.com with SMTP id 23so7012898pfy.8 for ; Sat, 23 May 2020 18:11:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=swKGdIWRL0IzqgEumPj6uSxpVsNncP1yOVwqocsnQI0=; b=G/owOvl5m/mbTNwN/18cXqV+zJH2Q30mV5aNoesngsr/yDo6+dBP7ddzXz6oPasxUy /yD+v9iR3f2+3JSl25VkUfqp6MoADONzJnG2cpTyYmYZq2H6UP5CqAgIxMsGCYwUvW2K cYJjBXzjmsskWK8pOF18aEejmEIgqDD7eHR+MF2MszZZ5mccAwpUmcPJ4eKDz+Ym8yYL TA6idexz9Z2AfHkR8BFvdoECXR6QMGINZzwpavz6u/A+7robgKfldfjHQVhtQBONEPR9 F0wfE05SJ4nI7juHFWjnvZ3USmP9mbEyJPm+yJzQ8xqHRI+MEwb3bBbQDMgZiEDR4TM7 mTeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=swKGdIWRL0IzqgEumPj6uSxpVsNncP1yOVwqocsnQI0=; b=TvfHpNKhWIjdloZP6P0YKnoTOCWISM+4mbtDjrBtvtYWKDBTFhHdXXgovv3ecODRu8 1tM1gOZay06CHFL6a5IBu61bunBVL8ZFVgntxUKFSHpN2HWJzaSDcgtEU3fa/yJYalno zpYboyk+6wU8Uc8GopT/UCBbmQzGJd10czq4zIKNfg0iT15V4FfAGOVrpC6xluQbHznr BevQPUJXBnqpxP/7yuJb1UN6rs6bbP9Yul7Q5xcEVofKANP1VEs9oj4qsq1EaBRrleMe HKXTuogCkj/gJ3hRExI9iJX0MZTV5jMZ5cIptkXtzgiRmG6yfI3PlaflJL6vktd+CVQ/ 0EfA== X-Gm-Message-State: AOAM532qBviB28qrMVY787rh1FzqCroq49d2F/x9TkkmxoVwfDLsd9wA HCYiOa768EOrSlkzbtnZOKGDmx5E X-Google-Smtp-Source: ABdhPJxLoG2dTqHya15YC6Ydjnel2jw6YhJqR2Jd2lCMhEUNjt8Ko9l3m/Pniblb0a6vxCH249Ry3A== X-Received: by 2002:a63:4d0c:: with SMTP id a12mr19857510pgb.449.1590282661019; Sat, 23 May 2020 18:11:01 -0700 (PDT) Received: from vpn2.localdomain ([161.117.202.209]) by smtp.gmail.com with ESMTPSA id gg8sm9731693pjb.39.2020.05.23.18.10.59 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 23 May 2020 18:11:00 -0700 (PDT) From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Date: Sun, 24 May 2020 09:04:50 +0800 Message-Id: <1590282291-20859-2-git-send-email-lance.lmwang@gmail.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1590282291-20859-1-git-send-email-lance.lmwang@gmail.com> References: <1589900073-4183-1-git-send-email-lance.lmwang@gmail.com> <1590282291-20859-1-git-send-email-lance.lmwang@gmail.com> Subject: [FFmpeg-devel] [PATCH v6 2/3] avdevice/lavfi: support the dumpgraph with options X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Cc: Limin Wang MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Limin Wang Signed-off-by: Limin Wang --- doc/indevs.texi | 15 +++++++++++++-- libavdevice/lavfi.c | 2 +- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/doc/indevs.texi b/doc/indevs.texi index 6f5afaf..7ec7062 100644 --- a/doc/indevs.texi +++ b/doc/indevs.texi @@ -973,8 +973,13 @@ Set the filename of the filtergraph to be read and sent to the other filters. Syntax of the filtergraph is the same as the one specified by the option @var{graph}. -@item dumpgraph -Dump graph to stderr. +@item dumpgraph @var{options} +Dump graph to stderr with more options + +options is a ':'-separated list of @var{key=value} pairs. + +Set the graph with graphviz DOT format by @var{fmt=dot|DOT}, +set the filename of filtergraph to output by @var{filename=path}. @end table @@ -988,6 +993,12 @@ ffplay -f lavfi -graph "color=c=pink [out0]" dummy @end example @item +dump the filter graph with graphviz DOT output format to ./test.tmp +@example +ffplay -dumpgraph fmt=dot:filename=./test.tmp -f lavfi color=c=pink +@end example + +@item As the previous example, but use filename for specifying the graph description, and omit the "out0" label: @example diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index c949ff7..fa75fde 100644 --- a/libavdevice/lavfi.c +++ b/libavdevice/lavfi.c @@ -493,7 +493,7 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt) static const AVOption options[] = { { "graph", "set libavfilter graph", OFFSET(graph_str), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC }, { "graph_file","set libavfilter graph filename", OFFSET(graph_filename), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC}, - { "dumpgraph", "dump graph to stderr", OFFSET(dump_graph), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC }, + { "dumpgraph", "dump graph to stderr with more options", OFFSET(dump_graph), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC }, { NULL }, }; From patchwork Sun May 24 01:04:51 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lance Wang X-Patchwork-Id: 19828 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 234FF449D53 for ; Sun, 24 May 2020 04:11:12 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 0CC6168A460; Sun, 24 May 2020 04:11:12 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pj1-f65.google.com (mail-pj1-f65.google.com [209.85.216.65]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id CDC9168980D for ; Sun, 24 May 2020 04:11:04 +0300 (EEST) Received: by mail-pj1-f65.google.com with SMTP id nu7so6730538pjb.0 for ; Sat, 23 May 2020 18:11:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=Yg/+N/wIcETrRYDRnw8Ur5baIaHYH1uIWPvUrsCJ3w0=; b=HsJJPrIhSkJKPMJoWwPwjIJIRIRcF2o5bsAJC+qCiUTe3ILU3bhm9ds8Mm/66Jw2+J H2VboM8YYjXny0Xk/ac/iOlNq3ErQEi9BGmQy+udIexcXl4azNUeY8AmRhz+rv+/qwrT vK/G73rGLdtB/lJb8nFafztoxzl4DhPG9Lav8uJ/nnzzKvSczwGvZQOT4lk+NYv9lryA QhsaOlb6KA04U5c2WwaNCIg19VK0b/z6wsyItbKPSbY+J5Caygyxp1WziWiwTO3b0kMA NZpcGIizCUoASdxOhYXfRFFbb2TMvyXEeRqiRZDN1/SjGQWqA0Ib2Lt8TKzXWhZfOQ07 8SlA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=Yg/+N/wIcETrRYDRnw8Ur5baIaHYH1uIWPvUrsCJ3w0=; b=DpSKeoQlLZ6dhIbjEsQRgk6XZmB6la2yH/8ZtEfG05/gjE73WkCqPmHzwp9t1fOR4P OL9Rz2mdpcLos2j4J636zOzUW2VYsdycI0nPWWN7Ynl0rZo+qvDpRBF7Yz7iy3X4vJiX 5ZxdFZYVcanveIAoss4A0fPZ2EydwaBtY2FOVdDGRhnqvhsqpaNO4LR4Yzn2bqVemS2Q P1ayArPfr0ic8+71YkajX7b0gH8blXhbG5QfFaVnQ+ViZByshsEO9740++HjBi0X2h1A LtWXYKq5Od3ahHFsLHujSsixJJuCyaOir8BBx3L+YO/ZzP0m+A/wfzWYh493jonzU1wO M9bw== X-Gm-Message-State: AOAM533ifdiMXwQ8YTAXRV6oDfcGgUDQFdOWjlCu6fgZTiWsigtpgHLU kaCitohE+9+w8HMTbtL2lXt92m3z X-Google-Smtp-Source: ABdhPJwQmkzmhruP2WCKK27od3c07uuXDyKKVMBNkwqStJfKuGc7OBgEjCN5DZLZr6Wd8/0/o6ZSnQ== X-Received: by 2002:a17:902:6b47:: with SMTP id g7mr21861133plt.277.1590282662841; Sat, 23 May 2020 18:11:02 -0700 (PDT) Received: from vpn2.localdomain ([161.117.202.209]) by smtp.gmail.com with ESMTPSA id gg8sm9731693pjb.39.2020.05.23.18.11.01 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 23 May 2020 18:11:02 -0700 (PDT) From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Date: Sun, 24 May 2020 09:04:51 +0800 Message-Id: <1590282291-20859-3-git-send-email-lance.lmwang@gmail.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1590282291-20859-1-git-send-email-lance.lmwang@gmail.com> References: <1589900073-4183-1-git-send-email-lance.lmwang@gmail.com> <1590282291-20859-1-git-send-email-lance.lmwang@gmail.com> Subject: [FFmpeg-devel] [PATCH v6 3/3] fftools: add options to dump filter graph X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Cc: Limin Wang MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Limin Wang Signed-off-by: Limin Wang --- doc/ffmpeg.texi | 12 ++++++++++++ fftools/ffmpeg.c | 1 + fftools/ffmpeg.h | 1 + fftools/ffmpeg_filter.c | 11 +++++++++++ fftools/ffmpeg_opt.c | 10 ++++++++++ 5 files changed, 35 insertions(+) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index ed437bb..c7dd73b 100644 --- a/doc/ffmpeg.texi +++ b/doc/ffmpeg.texi @@ -735,6 +735,18 @@ Technical note -- attachments are implemented as codec extradata, so this option can actually be used to extract extradata from any stream, not just attachments. +@item dump_filtergraph @var{options} (@emph{global}) +Dump graph to stderr with more options + +options is a ':'-separated list of @var{key=value} pairs. + +Set the graph with graphviz DOT format by @var{fmt=dot|DOT}, +set the filename of filtergraph to output by @var{filename=path}. + +@example +ffmpeg -dump_filtergraph fmt=dot:filename=./test.tmp -i INPUT +@end example + @item -noautorotate Disable automatically rotating video based on file metadata. diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index ad95a0e..a51167c 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -625,6 +625,7 @@ static void ffmpeg_cleanup(int ret) av_err2str(AVERROR(errno))); } av_freep(&vstats_filename); + av_freep(&dump_filtergraph); av_freep(&input_streams); av_freep(&input_files); diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 38205a1..000c682 100644 --- a/fftools/ffmpeg.h +++ b/fftools/ffmpeg.h @@ -578,6 +578,7 @@ extern int nb_filtergraphs; extern char *vstats_filename; extern char *sdp_filename; +extern char *dump_filtergraph; extern float audio_drift_threshold; extern float dts_delta_threshold; diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c index 422e126..4a3be39 100644 --- a/fftools/ffmpeg_filter.c +++ b/fftools/ffmpeg_filter.c @@ -1107,6 +1107,17 @@ int configure_filtergraph(FilterGraph *fg) if ((ret = avfilter_graph_config(fg->graph, NULL)) < 0) goto fail; + if (dump_filtergraph) { + char *dump = avfilter_graph_dump(fg->graph, dump_filtergraph); + if (!dump) { + ret = AVERROR(ENOMEM); + goto fail; + } + fputs(dump, stderr); + fflush(stderr); + av_free(dump); + } + /* limit the lists of allowed formats to the ones selected, to * make sure they stay the same if the filtergraph is reconfigured later */ for (i = 0; i < fg->nb_outputs; i++) { diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index 60bb437..e4ebc88 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/ffmpeg_opt.c @@ -143,6 +143,7 @@ HWDevice *filter_hw_device; char *vstats_filename; char *sdp_filename; +char *dump_filtergraph; float audio_drift_threshold = 0.1; float dts_delta_threshold = 10; @@ -2914,6 +2915,13 @@ static int opt_vstats_file(void *optctx, const char *opt, const char *arg) return 0; } +static int opt_dumpgraph(void *optctx, const char *opt, const char *arg) +{ + av_free (dump_filtergraph); + dump_filtergraph = av_strdup(arg); + return 0; +} + static int opt_vstats(void *optctx, const char *opt, const char *arg) { char filename[40]; @@ -3548,6 +3556,8 @@ const OptionDef options[] = { { "dump_attachment", HAS_ARG | OPT_STRING | OPT_SPEC | OPT_EXPERT | OPT_INPUT, { .off = OFFSET(dump_attachment) }, "extract an attachment into a file", "filename" }, + { "dump_filtergraph",HAS_ARG | OPT_EXPERT, { .func_arg = opt_dumpgraph }, + "dump filter graph to stderr with options"}, { "stream_loop", OPT_INT | HAS_ARG | OPT_EXPERT | OPT_INPUT | OPT_OFFSET, { .off = OFFSET(loop) }, "set number of times input stream shall be looped", "loop count" }, { "debug_ts", OPT_BOOL | OPT_EXPERT, { &debug_ts },