From patchwork Sun Jul 8 09:46:26 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Karsten Otto X-Patchwork-Id: 9649 Delivered-To: ffmpegpatchwork@gmail.com Received: by 2002:a02:104:0:0:0:0:0 with SMTP id c4-v6csp1388622jad; Sun, 8 Jul 2018 02:46:35 -0700 (PDT) X-Google-Smtp-Source: AAOMgpcb+9iTzymrlrzgtFN8ZVg3E6neJM4pyaur8sE7LaPDCw4/UMLZcnIVGdADONNg5eX4oiXR X-Received: by 2002:adf:f210:: with SMTP id p16-v6mr12288626wro.184.1531043195691; Sun, 08 Jul 2018 02:46:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531043195; cv=none; d=google.com; s=arc-20160816; b=ha9uNK/EMH3uL+EzeDYSfGeiNR5sXXv1XbHfJrrA/BROAywNKLoCgeebhd9+Ztt0ZU AR3hayQOgDce2TTb2sfZWaQh29+GKhFY6QX8myDYAhLnok9/aMlCVEwnsVcjFc9mX8hp ELIyYpfuBaLo7+KTEYQoGP92J1KWM7hRUFd7unPXQ6+nHvoYhT6/S9tyE36NoueZTUlX HMDMF5KTsTUc3o1ba7LoyVt2JvFIRz7c2tOLNhusJMCXpSKtXCoNDGhNA1C5NxFKmzlh swAXhKA77reunDzbdKevPbP7sZiQJaSepkegP/t8ZeVBFAmbJrPwCJovnZ37UpNtSKar aZgg== 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:message-id:date:to:from:dkim-signature :delivered-to:arc-authentication-results; bh=7dHwe7pkST6nB+YYWrX/8HVkHOIpWt0yLLEBjCLRa+k=; b=iVii+vFWv3JcEX7QrLy8DsRz5zrRNclE3t/jqwYk+WsW1Tje6RQHu3SX477+P4mJ+r dPdB9TF2XJGKgdor24oGRbqCgqXYgZkaS7b46i8Iz7aEmwQqJl8SQo7tXFDaDvwYmSgz /uD4+d1Y8tXyPvyGvp6lC/jJ4/s1gcjjyRTwhH132x/gOrnA3io0ay4cpJ8pE/Q1chDz ymy1VI6tDaJghXuwGy8mTtrDh7Hz+knaRIEg+3qWewkuEGWIHE+USxzh4rUfZCGWJcCM +TaZiXpYPWp1ZOG4SYJ9v2DmeTYIpipd5k7qE0GrVWEimuxTZ+imP/Zx12DFrHm2ym8Q bseg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@posteo.de header.s=2017 header.b=CmYh2Ajv; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=posteo.de Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id o4-v6si2973719wrg.441.2018.07.08.02.46.35; Sun, 08 Jul 2018 02:46:35 -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=@posteo.de header.s=2017 header.b=CmYh2Ajv; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=posteo.de Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 46BFC68A180; Sun, 8 Jul 2018 12:46:28 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mout01.posteo.de (mout01.posteo.de [185.67.36.65]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id DC103689C50 for ; Sun, 8 Jul 2018 12:46:21 +0300 (EEST) Received: from submission (posteo.de [89.146.220.130]) by mout01.posteo.de (Postfix) with ESMTPS id BD67720008 for ; Sun, 8 Jul 2018 11:46:27 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1531043187; bh=zExjD6uOYzSrjcWDY851WVGdbW3eeJkXhWUUt6/o+RA=; h=From:To:Subject:Date:From; b=CmYh2AjvtPn4FDhHAUe6tvst4dJE/NrfuFKO1XvTuCtteb/qnXTJ7HEhj6PBKNDBR G+S5FhnDO/Zj0ZjILJtRha1cngcP40ERKTogMvi/R2Uw6roOId+iWRfhrwnvPZg6cS kQTWbC+NL/9C8l2p6AtsEX2Ydh9dSPNKUo8A3uv0JMUTDJUfvddpbOPbzUemzmsTLM f1qHxl/MnQDzIvWrB5KIstqQw2KrJ6Sl7C/z942C4CPy7mpAMDW79IUsTrvMACGqFP PT1Lc2/gV7dBg99NSGjszglRzOzYzYSN8GsrV4h5LdzxblU34lGEVpIdX/4/Q+0mES cJy7wH8xTBSLQ== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 41NkBC2Hsgz9rxG for ; Sun, 8 Jul 2018 11:46:27 +0200 (CEST) From: Karsten Otto To: ffmpeg-devel@ffmpeg.org Date: Sun, 8 Jul 2018 11:46:26 +0200 Message-Id: <20180708094626.40407-1-ottoka@posteo.de> X-Mailer: git-send-email 2.14.3 (Apple Git-98) Subject: [FFmpeg-devel] [PATCH v5 2/3] aadec: add chapters and seeking 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" read_packet reads content in chunks. Thus seek must be clamped to valid chunk positions in the file, which in turn are relative to chapter start positions. So in read_header, scan for chapter headers once by skipping through the content. Set stream time_base based on bitrate in bytes/s, for easy timestamp to position conversion. Then in read_seek, find the chapter containing the seek position, calculate the nearest chunk position, and reinit the read_seek state accordingly. --- libavformat/aadec.c | 85 ++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 81 insertions(+), 4 deletions(-) diff --git a/libavformat/aadec.c b/libavformat/aadec.c index 4db71b1939..17ad20686b 100644 --- a/libavformat/aadec.c +++ b/libavformat/aadec.c @@ -35,6 +35,8 @@ #define MAX_TOC_ENTRIES 16 #define MAX_DICTIONARY_ENTRIES 128 #define TEA_BLOCK_SIZE 8 +#define CHAPTER_HEADER_SIZE 8 +#define TIMEPREC 1000 typedef struct AADemuxContext { AVClass *class; @@ -46,6 +48,7 @@ typedef struct AADemuxContext { struct AVTEA *tea_ctx; uint8_t file_key[16]; int64_t current_chapter_size; + int64_t content_start; int64_t content_end; } AADemuxContext; @@ -70,7 +73,7 @@ static int aa_read_header(AVFormatContext *s) uint32_t nkey, nval, toc_size, npairs, header_seed = 0, start; char key[128], val[128], codec_name[64] = {0}; uint8_t output[24], dst[8], src[8]; - int64_t largest_size = -1, current_size = -1; + int64_t largest_size = -1, current_size = -1, chapter_pos; struct toc_entry { uint32_t offset; uint32_t size; @@ -173,19 +176,23 @@ static int aa_read_header(AVFormatContext *s) st->codecpar->codec_id = AV_CODEC_ID_MP3; st->codecpar->sample_rate = 22050; st->need_parsing = AVSTREAM_PARSE_FULL_RAW; - st->start_time = 0; + avpriv_set_pts_info(st, 64, 8, 32000 * TIMEPREC); } else if (!strcmp(codec_name, "acelp85")) { st->codecpar->codec_id = AV_CODEC_ID_SIPR; st->codecpar->block_align = 19; st->codecpar->channels = 1; st->codecpar->sample_rate = 8500; + st->codecpar->bit_rate = 8500; st->need_parsing = AVSTREAM_PARSE_FULL_RAW; + avpriv_set_pts_info(st, 64, 8, 8500 * TIMEPREC); } else if (!strcmp(codec_name, "acelp16")) { st->codecpar->codec_id = AV_CODEC_ID_SIPR; st->codecpar->block_align = 20; st->codecpar->channels = 1; st->codecpar->sample_rate = 16000; + st->codecpar->bit_rate = 16000; st->need_parsing = AVSTREAM_PARSE_FULL_RAW; + avpriv_set_pts_info(st, 64, 8, 16000 * TIMEPREC); } /* determine, and jump to audio start offset */ @@ -198,7 +205,28 @@ static int aa_read_header(AVFormatContext *s) } start = TOC[largest_idx].offset; avio_seek(pb, start, SEEK_SET); + + // extract chapter positions. since all formats have constant bit rate, use it + // as time base in bytes/s, for easy stream position <-> timestamp conversion + st->start_time = 0; + c->content_start = start; c->content_end = start + largest_size; + + while ((chapter_pos = avio_tell(pb)) >= 0 && chapter_pos < c->content_end) { + int chapter_idx = s->nb_chapters; + uint32_t chapter_size = avio_rb32(pb); + if (chapter_size == 0) break; + chapter_pos -= start + CHAPTER_HEADER_SIZE * chapter_idx; + avio_skip(pb, 4 + chapter_size); + if (!avpriv_new_chapter(s, chapter_idx, st->time_base, + chapter_pos * TIMEPREC, (chapter_pos + chapter_size) * TIMEPREC, NULL)) + return AVERROR(ENOMEM); + } + + st->duration = (largest_size - CHAPTER_HEADER_SIZE * s->nb_chapters) * TIMEPREC; + + ff_update_cur_dts(s, st, 0); + avio_seek(pb, start, SEEK_SET); c->current_chapter_size = 0; return 0; @@ -215,9 +243,10 @@ static int aa_read_packet(AVFormatContext *s, AVPacket *pkt) int written = 0; int ret; AADemuxContext *c = s->priv_data; + uint64_t pos = avio_tell(s->pb); // are we at the end of the audio content? - if (avio_tell(s->pb) >= c->content_end) { + if (pos >= c->content_end) { return AVERROR_EOF; } @@ -230,6 +259,7 @@ static int aa_read_packet(AVFormatContext *s, AVPacket *pkt) av_log(s, AV_LOG_DEBUG, "Chapter %d (%" PRId64 " bytes)\n", c->chapter_idx, c->current_chapter_size); c->chapter_idx = c->chapter_idx + 1; avio_skip(s->pb, 4); // data start offset + pos += 8; c->current_codec_second_size = c->codec_second_size; } @@ -267,10 +297,56 @@ static int aa_read_packet(AVFormatContext *s, AVPacket *pkt) if (ret < 0) return ret; memcpy(pkt->data, buf, written); + pkt->pos = pos; return 0; } +static int aa_read_seek(AVFormatContext *s, + int stream_index, int64_t timestamp, int flags) +{ + AADemuxContext *c = s->priv_data; + AVChapter *ch; + int64_t chapter_pos, chapter_start, chapter_size; + int chapter_idx = 0; + + // find chapter containing seek timestamp + if (timestamp < 0) + timestamp = 0; + + while (chapter_idx < s->nb_chapters && timestamp >= s->chapters[chapter_idx]->end) { + ++chapter_idx; + } + + if (chapter_idx >= s->nb_chapters) { + chapter_idx = s->nb_chapters - 1; + if (chapter_idx < 0) return -1; // there is no chapter. + timestamp = s->chapters[chapter_idx]->end; + } + + ch = s->chapters[chapter_idx]; + + // sync by clamping timestamp to nearest valid block position in its chapter + chapter_size = ch->end / TIMEPREC - ch->start / TIMEPREC; + chapter_pos = av_rescale_rnd((timestamp - ch->start) / TIMEPREC, + 1, c->codec_second_size, + (flags & AVSEEK_FLAG_BACKWARD) ? AV_ROUND_DOWN : AV_ROUND_UP) + * c->codec_second_size; + if (chapter_pos >= chapter_size) + chapter_pos = chapter_size; + chapter_start = c->content_start + (ch->start / TIMEPREC) + CHAPTER_HEADER_SIZE * (1 + chapter_idx); + + // reinit read state + avio_seek(s->pb, chapter_start + chapter_pos, SEEK_SET); + c->current_codec_second_size = c->codec_second_size; + c->current_chapter_size = chapter_size - chapter_pos; + c->chapter_idx = 1 + chapter_idx; + + ff_update_cur_dts(s, s->streams[0], ch->start + chapter_pos * TIMEPREC); + + return 1; +} + static int aa_probe(AVProbeData *p) { uint8_t *buf = p->buf; @@ -316,6 +392,7 @@ AVInputFormat ff_aa_demuxer = { .read_probe = aa_probe, .read_header = aa_read_header, .read_packet = aa_read_packet, + .read_seek = aa_read_seek, .read_close = aa_read_close, - .flags = AVFMT_GENERIC_INDEX, + .flags = AVFMT_NO_BYTE_SEEK | AVFMT_NOGENSEARCH, };