From patchwork Sun Aug 20 22:41:37 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Thompson X-Patchwork-Id: 4779 Delivered-To: ffmpegpatchwork@gmail.com Received: by 10.2.37.132 with SMTP id g126csp2343911jag; Sun, 20 Aug 2017 15:44:12 -0700 (PDT) X-Received: by 10.28.52.207 with SMTP id b198mr4706810wma.31.1503269052901; Sun, 20 Aug 2017 15:44:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1503269052; cv=none; d=google.com; s=arc-20160816; b=UGTqisF8tD4M+XD22wjCIVNmL+GR3RSSZ3VAeroCsZ/9968fLRxDKjDdAzYoDgVpYk FgW5EFVt9XWoJA0K5t5HTCbLiDEJFMLiU4DwEKk0NBTCPq5nfXynzdXk4sc6n/RY7UKR qKHdJOn18ol/atwYI2sEY4OCGQqWjXFsFUStXc1EderGC1CvmdXD/I7w7s3L5qNrb7de 8nLwafLCwkzrU4IbStalC2vBa5sopR/8LXE2cUjPNZYf+pk4jAv78RD92edX+PEG1qn9 P+eAQwnb8QmkbpMmTf565QvCRJ3UUxrhutXyiyx5BOt+X/X9HYmDAWDfgXASa1VtNzbr CpGw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:references:in-reply-to:message-id:date :to:from:dkim-signature:delivered-to:arc-authentication-results; bh=6DJWhYjLJBwzfTbgwDlvXL1ibfZn1sofpxf+u4TYxIg=; b=GjAHmNrpTE50mS+UTLjgU+R+R9KAfKKL2sYEIMmPnJIQHyPwXhQqIQQf0eo/V4tlsb WsVLWVQszK/xX4a5bY5EgpiZOF/46G00aeUvRKFDEQdAVKxX88X76mMd5nFDOVf0N6r3 FE/Gq57DpXrWTac4FH4ClGZ0kU+oDRamugq8ngQ6VFVmWIbc7HpXRApFOY9a+EmT3MYm TNnT4Z2USua2Uv84dr6OIehEAszS+o+v09MoTswxXdM/ZPRKHOkQIa6V4DAGfwA2PY+O cTwzsBQMY70Z1Gy2Xanhny+DI9m0ga/qrYWir+iZQqifTYyFec5tuhX25DIKkKvM4MPu KzUw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@jkqxz-net.20150623.gappssmtp.com header.s=20150623 header.b=luiloqFj; 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 c13si8256669wrd.22.2017.08.20.15.44.12; Sun, 20 Aug 2017 15:44:12 -0700 (PDT) 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=@jkqxz-net.20150623.gappssmtp.com header.s=20150623 header.b=luiloqFj; 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 19C82689B2D; Mon, 21 Aug 2017 01:42:12 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wr0-f171.google.com (mail-wr0-f171.google.com [209.85.128.171]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D8B726898AC for ; Mon, 21 Aug 2017 01:42:01 +0300 (EEST) Received: by mail-wr0-f171.google.com with SMTP id f8so48166980wrf.3 for ; Sun, 20 Aug 2017 15:42:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=jkqxz-net.20150623.gappssmtp.com; s=20150623; h=from:to:subject:date:message-id:in-reply-to:references; bh=5XTa5jLPQ3AT4jn/wkfzZtUsQLm5BDmRBO5LtZqbVGg=; b=luiloqFjjF6xbLavU0jIn65roBr1h4ySfrb0q/12UbVIQmH0PO0+A7o28Idtd6OqlT M8ICSUfkvEyc2oK4a1n99tLV25WXKrJi7E406vwUNxYf7K724mIR71hEhFwHFbe+Rbzw KBLuzMfYqZJnJ0BatuPnrM2lpZxhJ9H197wvFntn0mU5PSGPmb2c0/qkpK5fJ2wjP9oQ v032zqyqY3tFBz0x0Ih+rn5yOwi8Yop37VKMclkiRrPfQAPCIBCLXqGMfU+2uniM6tTX AbSIiSoKZUi1eH6uQpO5Nbyk/8XJMiUTD1YABQecUIAxNjg+mrcuauw+J5vi+ekS/hIo 35eg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=5XTa5jLPQ3AT4jn/wkfzZtUsQLm5BDmRBO5LtZqbVGg=; b=dULwqVp9hJsBEorn5m56x/4CGMxPyFwUIWHZmlVRHD0yJ8rVh3gmkbGUr5Inzhw6JI qujY8n5uaJy0Vq5ADh4ffMUh7H/ULuL2ookiOMPzVWxmoNWyNnXSPuyRXPzncrEzyLwd NLNRJOaY1aqQDiTpVTf87Ws//5jKw2Z+lZUqAHxmTVjXmKfVZSo5Ki8IFSobWecm02mg NyNLwdK74PAPkIVvgH4bwaAuPxPXdOOOrU44rZJ3mdyrtAcSUMAadq0K4WA287qkmuCN kF35/dGy3kUoJU9Gi+Kv0qBN4M89pyh/NEdzR7aNCOhUKJfDHny13yYmKCyo5Jn3AcMM phKQ== X-Gm-Message-State: AHYfb5i9K9zLpm021T8jZUjEMaLdmcb8ONkHG2ELL77l0ClqmKV/lAXN 0dSAkPXktaNc+3LW8/o= X-Received: by 10.28.74.7 with SMTP id x7mr4750948wma.112.1503268928054; Sun, 20 Aug 2017 15:42:08 -0700 (PDT) Received: from rywe.jkqxz.net (cpc91242-cmbg18-2-0-cust650.5-4.cable.virginm.net. [82.8.130.139]) by smtp.gmail.com with ESMTPSA id f9sm5391116wmf.18.2017.08.20.15.42.07 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 20 Aug 2017 15:42:07 -0700 (PDT) From: Mark Thompson To: ffmpeg-devel@ffmpeg.org Date: Sun, 20 Aug 2017 23:41:37 +0100 Message-Id: <20170820224146.21289-10-sw@jkqxz.net> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170820224146.21289-1-sw@jkqxz.net> References: <20170820224146.21289-1-sw@jkqxz.net> Subject: [FFmpeg-devel] [PATCH 09/18] lavc: Add trace_headers bitstream filter 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 MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Supports all streams that the coded bitstream infrastructure does (currently H.264, H.265 and MPEG-2). (cherry picked from commit f11d8a5e8b185340cc50fcbc8a1437b0fbe7e931) --- configure | 1 + doc/bitstream_filters.texi | 8 +++ libavcodec/Makefile | 1 + libavcodec/bitstream_filters.c | 1 + libavcodec/trace_headers_bsf.c | 126 +++++++++++++++++++++++++++++++++++++++++ 5 files changed, 137 insertions(+) create mode 100644 libavcodec/trace_headers_bsf.c diff --git a/configure b/configure index 8d9a8b0d03..41e23c2780 100755 --- a/configure +++ b/configure @@ -2833,6 +2833,7 @@ vc1_parser_select="vc1dsp" # bitstream_filters mjpeg2jpeg_bsf_select="jpegtables" +trace_headers_bsf_select="cbs_h264 cbs_h265 cbs_mpeg2" # external libraries aac_at_decoder_deps="audiotoolbox" diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index 2dffe021f9..990b4f3c58 100644 --- a/doc/bitstream_filters.texi +++ b/doc/bitstream_filters.texi @@ -275,6 +275,14 @@ codec) with metadata headers. See also the @ref{mov2textsub} filter. +@section trace_headers + +Log trace output containing all syntax elements in the coded stream +headers (everything above the level of individual coded blocks). +This can be useful for debugging low-level stream issues. + +Supports H.264, H.265 and MPEG-2. + @section vp9_superframe Merge VP9 invisible (alt-ref) frames back into VP9 superframes. This diff --git a/libavcodec/Makefile b/libavcodec/Makefile index ddb56ae907..4916dfc4cb 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -1003,6 +1003,7 @@ OBJS-$(CONFIG_NOISE_BSF) += noise_bsf.o OBJS-$(CONFIG_NULL_BSF) += null_bsf.o OBJS-$(CONFIG_REMOVE_EXTRADATA_BSF) += remove_extradata_bsf.o OBJS-$(CONFIG_TEXT2MOVSUB_BSF) += movsub_bsf.o +OBJS-$(CONFIG_TRACE_HEADERS_BSF) += trace_headers_bsf.o OBJS-$(CONFIG_VP9_RAW_REORDER_BSF) += vp9_raw_reorder_bsf.o OBJS-$(CONFIG_VP9_SUPERFRAME_BSF) += vp9_superframe_bsf.o OBJS-$(CONFIG_VP9_SUPERFRAME_SPLIT_BSF) += vp9_superframe_split_bsf.o diff --git a/libavcodec/bitstream_filters.c b/libavcodec/bitstream_filters.c index ce34de640d..dd0247182b 100644 --- a/libavcodec/bitstream_filters.c +++ b/libavcodec/bitstream_filters.c @@ -41,6 +41,7 @@ extern const AVBitStreamFilter ff_noise_bsf; extern const AVBitStreamFilter ff_null_bsf; extern const AVBitStreamFilter ff_remove_extradata_bsf; extern const AVBitStreamFilter ff_text2movsub_bsf; +extern const AVBitStreamFilter ff_trace_headers_bsf; extern const AVBitStreamFilter ff_vp9_raw_reorder_bsf; extern const AVBitStreamFilter ff_vp9_superframe_bsf; extern const AVBitStreamFilter ff_vp9_superframe_split_bsf; diff --git a/libavcodec/trace_headers_bsf.c b/libavcodec/trace_headers_bsf.c new file mode 100644 index 0000000000..9ea6308a61 --- /dev/null +++ b/libavcodec/trace_headers_bsf.c @@ -0,0 +1,126 @@ +/* + * 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 + +#include "libavutil/avstring.h" +#include "libavutil/common.h" +#include "libavutil/log.h" + +#include "bsf.h" +#include "cbs.h" + + +typedef struct TraceHeadersContext { + CodedBitstreamContext cbc; +} TraceHeadersContext; + + +static int trace_headers_init(AVBSFContext *bsf) +{ + TraceHeadersContext *ctx = bsf->priv_data; + int err; + + err = ff_cbs_init(&ctx->cbc, bsf->par_in->codec_id, bsf); + if (err < 0) + return err; + + ctx->cbc.trace_enable = 1; + ctx->cbc.trace_level = AV_LOG_INFO; + + if (bsf->par_in->extradata) { + CodedBitstreamFragment ps; + + av_log(bsf, AV_LOG_INFO, "Extradata\n"); + + err = ff_cbs_read_extradata(&ctx->cbc, &ps, bsf->par_in); + if (err < 0) { + av_log(bsf, AV_LOG_ERROR, "Failed to read extradata.\n"); + return err; + } + + ff_cbs_fragment_uninit(&ctx->cbc, &ps); + } + + return 0; +} + +static void trace_headers_close(AVBSFContext *bsf) +{ + TraceHeadersContext *ctx = bsf->priv_data; + + ff_cbs_close(&ctx->cbc); +} + +static int trace_headers(AVBSFContext *bsf, AVPacket *out) +{ + TraceHeadersContext *ctx = bsf->priv_data; + CodedBitstreamFragment au; + AVPacket *in; + char tmp[256] = { 0 }; + int err; + + err = ff_bsf_get_packet(bsf, &in); + if (err < 0) + return err; + + if (in->flags & AV_PKT_FLAG_KEY) + av_strlcat(tmp, ", key frame", sizeof(tmp)); + if (in->flags & AV_PKT_FLAG_CORRUPT) + av_strlcat(tmp, ", corrupt", sizeof(tmp)); + + if (in->pts != AV_NOPTS_VALUE) + av_strlcatf(tmp, sizeof(tmp), ", pts %"PRId64, in->pts); + else + av_strlcat(tmp, ", no pts", sizeof(tmp)); + if (in->dts != AV_NOPTS_VALUE) + av_strlcatf(tmp, sizeof(tmp), ", dts %"PRId64, in->dts); + else + av_strlcat(tmp, ", no dts", sizeof(tmp)); + if (in->duration > 0) + av_strlcatf(tmp, sizeof(tmp), ", duration %"PRId64, in->duration); + + av_log(bsf, AV_LOG_INFO, "Packet: %d bytes%s.\n", in->size, tmp); + + err = ff_cbs_read_packet(&ctx->cbc, &au, in); + if (err < 0) + return err; + + ff_cbs_fragment_uninit(&ctx->cbc, &au); + + av_packet_move_ref(out, in); + av_packet_free(&in); + + return 0; +} + +static const enum AVCodecID trace_headers_codec_ids[] = { + AV_CODEC_ID_H264, + AV_CODEC_ID_HEVC, + AV_CODEC_ID_MPEG2VIDEO, + AV_CODEC_ID_NONE, +}; + +const AVBitStreamFilter ff_trace_headers_bsf = { + .name = "trace_headers", + .priv_data_size = sizeof(TraceHeadersContext), + .init = &trace_headers_init, + .close = &trace_headers_close, + .filter = &trace_headers, + .codec_ids = trace_headers_codec_ids, +};