From patchwork Thu May 28 00:27:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: A G X-Patchwork-Id: 19906 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 6FE3C4496DD for ; Thu, 28 May 2020 03:28:00 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 49051689AE3; Thu, 28 May 2020 03:28:00 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11olkn2075.outbound.protection.outlook.com [40.92.20.75]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E6D4E680890 for ; Thu, 28 May 2020 03:27:52 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=A45BVbzS6SE5lJgsAqjeEm/e+vVwhvfYkc+XmbV0gTDTM54yhiOji00QNXDmmyvd9oV3avH3YqAmAtH68QnnpN17w/aDBoh/mNlpXA6RFY4Bv8go3WJyXrRExQTjHvp5k+9Jri1/Yn+YAzMbGRyje+iuKWjYozOt0LhV2Cq8d46dnjDxJF67dmhJXF06PScU8kgzBImSD8VeoserL6OnS4gAnuTCWQrrKhN9EeswdFmJ/vZ5vc8/sw6bPvNt1sa4pROX7TM3BPraY1VIypjibV2eiVg2CgUUSLe37GDbe2OPmZlaJdpv0vLqp7xVGg5Hxj0PmyqTOACERKsok8EREg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=yjvd5FB191Z4nfcP+et48r39UiXT6COlNKEaIRHmdvY=; b=Os4Sgh5WFW//K0vJCRbpexP5673wy53Itx7dWDryDXdPvQHN+lNKtusOAWxxpqLFPYNbnW9BdfiVpIp/8UUr8bYUWyJUfqkwlfm5FlQey5g+91H8B9hdN33ay5ct3V/MuDFUsNUcJyynqAYEWR4f6h1hPM6pbVfzpn6uMrPtq4kI0J0cACvK3r/Za97kGLQA+5LldB5ofEV/9SDZ3Q+ycY3/+u8OQEsiFasLj1Rjo0tasBRDGMaA9bbYtRIWBYrG5nisO0bpEM/q1h54C446DCrvygnY3mCDmviiCPP0Nxh92X+U1tUgC8mSL/ADQ2RYgZOLtf/PJnZwOkN4QHkP4Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=yjvd5FB191Z4nfcP+et48r39UiXT6COlNKEaIRHmdvY=; b=L4t465T2kixoBXDTqJexCWZGIDEz0VQ3PEQbeEOLYP7z0v6uZGQDdVWpMpp5git9snA17BuFaxulCM2m/CpAZXCeGGTDaldJrTbrQCNIIJqzdAitRdyVIe2WrOvkyZMLEb8PEoqJILraHqkC+fygF3dg6Hsf3mdeW6P/uiCrqd3vNTrWxuoEo9S/DwrAZflORuKmOzYpl/Wx2ykg0Dkw1Eqf8UwyccW6+E1mpxCpi9bi72io8QzQGolrV3ECIwdNMr9Vx+qFvDex9ljop7ebh4SZjAKdolVdqYOhAPFsGD9+PAe4YZadeeL7kH5TRW2Ut9Ewq71NEFnebGB/FQv0qg== Received: from CO1NAM11FT063.eop-nam11.prod.protection.outlook.com (2a01:111:e400:3861::4e) by CO1NAM11HT004.eop-nam11.prod.protection.outlook.com (2a01:111:e400:3861::335) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3021.23; Thu, 28 May 2020 00:27:50 +0000 Received: from BYAPR15MB2647.namprd15.prod.outlook.com (2a01:111:e400:3861::41) by CO1NAM11FT063.mail.protection.outlook.com (2a01:111:e400:3861::293) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3021.23 via Frontend Transport; Thu, 28 May 2020 00:27:50 +0000 Received: from BYAPR15MB2647.namprd15.prod.outlook.com ([fe80::f083:ed57:da58:e2a1]) by BYAPR15MB2647.namprd15.prod.outlook.com ([fe80::f083:ed57:da58:e2a1%6]) with mapi id 15.20.3021.029; Thu, 28 May 2020 00:27:50 +0000 From: A G To: FFmpeg development discussions and patches Thread-Topic: [PATCH] avformat/oggenc: Add partial support for OggPCM muxing Thread-Index: AQHWNIXPWi37IadvKkuwRVdhP7HyIQ== Date: Thu, 28 May 2020 00:27:50 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-incomingtopheadermarker: OriginalChecksum:A73F63340A69A81B7224B9E09972155A91FDBDA2481596DDCC169DEEB8655221; UpperCasedChecksum:D109C93F61DE9DD6B580F931CD3A9E2822A15A46E2164C164153A08FF196AA24; SizeAsReceived:6807; Count:42 x-tmn: [tVhFYtw67AWjOiyDfY8cqXqaWHwYoKkMUDEZS1FLACA=] x-ms-publictraffictype: Email x-incomingheadercount: 42 x-eopattributedmessage: 0 x-ms-office365-filtering-correlation-id: dbe1eedc-7b47-4a63-0ee2-08d8029df474 x-ms-traffictypediagnostic: CO1NAM11HT004: x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: urcHaW7BuuJ+aYhm1LWn4lpKkESbtUd4QJVmYM0gEe3P8N3j18Px3ACC8+fc+HZU7t6r81H18aeEfD2x/WwfHCz+fhESim3OVc3yvMp3PSQ8+O9h7esk97RDM+n5QkSTHbMtadDma/qOvQzXXVc5fG16IG9YLJv7pcfCX9PuoCmXoA9aMspIpoTW4S/Pnwp2bSQPu7SzidKWdB6LCoYSKElf3X3dYXF/76YiwHztETPOETK7wJdRBt4rAUFBBToS x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:0; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR15MB2647.namprd15.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:; DIR:OUT; SFP:1901; x-ms-exchange-antispam-messagedata: GVzhBLdZiKJE/c3CQRJVFOnnYJeHfsKSgeYqjdtKxk3o2OaSdFDqkiF9rv9I+OkTOS/25GrsImD+fMKTsNEVLbKfOkAMml8LkGKl0DzL6R0VxbVG4rAi4lhoDKc+50bCgccjI7UO4TnAnnoV8iznlnp5ujVZp3TolBCnf9c4/ns= x-ms-exchange-transport-forked: True MIME-Version: 1.0 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: dbe1eedc-7b47-4a63-0ee2-08d8029df474 X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-originalarrivaltime: 28 May 2020 00:27:50.6353 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1NAM11HT004 X-Content-Filtered-By: Mailman/MimeDel 2.1.20 Subject: [FFmpeg-devel] [PATCH] avformat/oggenc: Add partial support for OggPCM muxing 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From 81cf2038424e483e60ed9747f965ec452d357773 Mon Sep 17 00:00:00 2001 From: oreo639 <31916379+Oreo639@users.noreply.github.com> Date: Wed, 27 May 2020 17:18:03 -0700 Subject: [PATCH] avformat/oggenc: Add partial support for OggPCM muxing This adds partial support for OggPCM demuxing. Heavily based on the work here: https://ffmpeg.org/pipermail/ffmpeg-devel/2013-July/145556.html and here: http://www.on2.com/media/gpl/mplayer/ --- libavformat/oggenc.c | 85 ++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 79 insertions(+), 6 deletions(-) -- 2.26.2 diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c index f5032759a6..5f03b4d9b7 100644 --- a/libavformat/oggenc.c +++ b/libavformat/oggenc.c @@ -99,6 +99,36 @@ static const AVClass flavor ## _muxer_class = {\ .version = LIBAVUTIL_VERSION_INT,\ }; + +static const struct ogg_pcm_codec { + uint32_t codec_id; + uint32_t format_id; +} ogg_pcm_codecs[] = { + { AV_CODEC_ID_PCM_S8, 0x00 }, + { AV_CODEC_ID_PCM_U8, 0x01 }, + { AV_CODEC_ID_PCM_S16LE, 0x02 }, + { AV_CODEC_ID_PCM_S16BE, 0x03 }, + { AV_CODEC_ID_PCM_S24LE, 0x04 }, + { AV_CODEC_ID_PCM_S24BE, 0x05 }, + { AV_CODEC_ID_PCM_S32LE, 0x06 }, + { AV_CODEC_ID_PCM_S32BE, 0x07 }, + { AV_CODEC_ID_PCM_F32LE, 0x20 }, + { AV_CODEC_ID_PCM_F32BE, 0x21 }, + { AV_CODEC_ID_PCM_F64LE, 0x22 }, + { AV_CODEC_ID_PCM_F64BE, 0x23 }, +}; + +static inline uint32_t ogg_get_pcm_format_id(uint32_t codec_id) +{ + int i; + + for (i = 0; i < FF_ARRAY_ELEMS(ogg_pcm_codecs); i++) + if (ogg_pcm_codecs[i].codec_id == codec_id) + return ogg_pcm_codecs[i].format_id; + + return 0; +} + static void ogg_write_page(AVFormatContext *s, OGGPage *page, int extra_flags) { OGGStreamContext *oggstream = s->streams[page->stream_index]->priv_data; @@ -363,6 +393,39 @@ static int ogg_build_speex_headers(AVCodecParameters *par, return 0; } +#define OGGPCM_HEADER_SIZE 224 + +static int ogg_build_pcm_headers(AVCodecParameters *par, + OGGStreamContext *oggstream, int bitexact, + AVDictionary **m) +{ + uint8_t *p; + + // first packet: OggPCM header + p = av_mallocz(OGGPCM_HEADER_SIZE); + if (!p) + return AVERROR(ENOMEM); + oggstream->header[0] = p; + oggstream->header_len[0] = OGGPCM_HEADER_SIZE; + bytestream_put_buffer(&p, "PCM ", 8); // Identifier + bytestream_put_be16(&p, 0x00); // VMAJ + bytestream_put_be16(&p, 0x00); // VMIN + bytestream_put_be32(&p, ogg_get_pcm_format_id(par->codec_id)); // PCM fmt + bytestream_put_be32(&p, par->sample_rate); // Sample rate + bytestream_put_byte(&p, 0); // Significant bits (0 == Same as fmt) + bytestream_put_byte(&p, par->channels); // Channels + bytestream_put_be16(&p, 0); // Max frames per packet TODO:// Actually calculate max frames per packet + bytestream_put_be32(&p, 0); // Number of extra headers + + // second packet: VorbisComment + p = ogg_write_vorbiscomment(0, bitexact, &oggstream->header_len[1], m, 0, NULL, 0); + if (!p) + return AVERROR(ENOMEM); + oggstream->header[1] = p; + + return 0; +} + #define OPUS_HEADER_SIZE 19 static int ogg_build_opus_headers(AVCodecParameters *par, @@ -487,18 +550,20 @@ static int ogg_init(AVFormatContext *s) avpriv_set_pts_info(st, 64, 1, st->codecpar->sample_rate); } - if (st->codecpar->codec_id != AV_CODEC_ID_VORBIS && - st->codecpar->codec_id != AV_CODEC_ID_THEORA && - st->codecpar->codec_id != AV_CODEC_ID_SPEEX && - st->codecpar->codec_id != AV_CODEC_ID_FLAC && - st->codecpar->codec_id != AV_CODEC_ID_OPUS && + if (st->codecpar->codec_id != AV_CODEC_ID_VORBIS && + st->codecpar->codec_id != AV_CODEC_ID_THEORA && + st->codecpar->codec_id != AV_CODEC_ID_SPEEX && + st->codecpar->codec_id != AV_CODEC_ID_FLAC && + st->codecpar->codec_id != AV_CODEC_ID_OPUS && + !ogg_get_pcm_format_id(st->codecpar->codec_id) && st->codecpar->codec_id != AV_CODEC_ID_VP8) { av_log(s, AV_LOG_ERROR, "Unsupported codec id in stream %d\n", i); return AVERROR(EINVAL); } if ((!st->codecpar->extradata || !st->codecpar->extradata_size) && - st->codecpar->codec_id != AV_CODEC_ID_VP8) { + st->codecpar->codec_id != AV_CODEC_ID_VP8 && + !ogg_get_pcm_format_id(st->codecpar->codec_id)) { av_log(s, AV_LOG_ERROR, "No extradata present\n"); return AVERROR_INVALIDDATA; } @@ -553,6 +618,14 @@ static int ogg_init(AVFormatContext *s) av_log(s, AV_LOG_ERROR, "Error writing VP8 headers\n"); return err; } + } else if (ogg_get_pcm_format_id(st->codecpar->codec_id)) { + int err = ogg_build_pcm_headers(st->codecpar, oggstream, + s->flags & AVFMT_FLAG_BITEXACT, + &st->metadata); + if (err) { + av_log(s, AV_LOG_ERROR, "Error writing OggPCM headers\n"); + return err; + } } else { uint8_t *p; const char *cstr = st->codecpar->codec_id == AV_CODEC_ID_VORBIS ? "vorbis" : "theora";