From: Cameron Gutman <aicommander@gmail.com> To: Andriy Gelman <andriy.gelman@gmail.com>, FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_dec: dequeue frame if input isn't ready Date: Mon, 27 Dec 2021 03:18:10 -0600 Message-ID: <246eb92f-61c3-7158-5265-feb0cb0252af@gmail.com> (raw) In-Reply-To: <20211227061738.mbkxlmm4deveyggf@jackie> On 12/27/21 00:17, Andriy Gelman wrote: > On Tue, 14. Dec 02:12, Cameron Gutman wrote: >> The V4L2M2M API operates asynchronously, so multiple packets can >> be enqueued before getting a batch of frames back. Since it was >> only possible to receive a frame by submitting another packet, >> there wasn't a way to drain those excess output frames from when >> avcodec_receive_frame() returned AVERROR(EAGAIN). >> > >> In my testing, this change reduced decode latency of a real-time >> 60 FPS H.264 stream by approximately 10x (200ms -> 20ms) on a >> Raspberry Pi 4. > > I was doing some more tests today, but didn't have any luck dequeuing a frame > if ff_decode_get_packet() returned EAGAIN. Hmm, maybe there is something different about your test harness? I'm receiving 720p 60 FPS H.264 ES in real-time from the network. For each H.264 encoded frame I receive off the network, my basic approach is like this (simplified for brevity): avcodec_send_packet(&pkt); do { frame = av_frame_alloc(); if ((err = avcodec_receive_frame(frame)) == 0) { render_frame(frame); } } while (err == 0); I'll usually get EAGAIN immediately for the first few frames I submit (so no output frame yet), but then I'll get a batch of output frames back after the first completed decode. That drains the excess latency from the pipeline to avoid always being behind. For cases where we want to prioritize latency over throughput, I've had success with this approach too: avcodec_send_packet(&pkt); while (avcodec_receive_frame(frame) == AVERROR(EAGAIN)) { msleep(1); } render_frame(frame); In this case, we can retry avcodec_receive_frame() until we get the frame back that we just submitted for decoding. The patch here enables both of these use-cases by allowing V4L2M2M to retry getting a decoded frame without new input data. Both of these also work with MMAL after the recent decoupled dataflow patch. > Could you share the dataset? > It is 720p60.h264 from here: https://onedrive.live.com/?authkey=%21ALoKfcPfFeKyhzs&id=C15BF9770619F56%21165617&cid=0C15BF9770619F56 > Thanks, > Thank you _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2021-12-27 9:18 UTC|newest] Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <20211214021215.456493-1-aicommander@gmail.com> 2021-12-25 7:28 ` Andriy Gelman 2021-12-27 6:17 ` Andriy Gelman 2021-12-27 9:18 ` Cameron Gutman [this message] 2021-12-28 23:41 ` Andriy Gelman 2022-01-02 16:56 ` Andriy Gelman
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=246eb92f-61c3-7158-5265-feb0cb0252af@gmail.com \ --to=aicommander@gmail.com \ --cc=andriy.gelman@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git