qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Alex K <vip-ak47@yandex.ru>
Cc: qemu-devel@nongnu.org, aliguori@amazon.com
Subject: Re: [Qemu-devel] [PATCH] Video and sound capture to a videofile through ffmpeg
Date: Thu, 04 May 2017 16:01:52 +0200	[thread overview]
Message-ID: <1493906512.371.19.camel@redhat.com> (raw)
In-Reply-To: <1493894491-26130-1-git-send-email-vip-ak47@yandex.ru>

On Do, 2017-05-04 at 13:41 +0300, Alex K wrote:
> Hello everyone,
> 
> I've made a patch that adds ability to record video of what's going on
> inside qemu. It uses ffmpeg libraries. Basically, the patch adds
> 2 new commands to the console:
> capture_start path framerate
> capture_stop
> 
> path is required
> framerate could be 24, 25, 30 or 60. Default is 60
> video codec is always h264
> 
> The patch uses ffmpeg so you will need to install these packages:
> ffmpeg libavformat-dev libavcodec-dev libavutil-dev libswscale-dev
> 
> This is my first time posting here, so please correct me if I'm doing
> something wrong

First, as others already mentioned, it would be better to do this
outside qemu, as vnc or spice client.  spice is probably the better
choice as spice supports passing the guest display as dma-buf, which
saves the encode-as-spice and decode-spice steps.  And the dma-bufs can
be passed to a hardware encoder.

Second, this must be a compile time option.  Breaking the build in case
ffmpeg is not installed isn't an option.

Third, supporting H.264 only is a non-starter too.  Due to the patent
situation you can't expect distros shipping ffmpeg with H.264 support
enabled.

cheers,
  Gerd

  parent reply	other threads:[~2017-05-04 14:01 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-04 10:41 [Qemu-devel] [PATCH] Video and sound capture to a videofile through ffmpeg Alex K
2017-05-04 10:45 ` no-reply
2017-05-04 10:48 ` no-reply
2017-05-04 10:50 ` Marc-André Lureau
2017-05-04 10:51 ` Daniel P. Berrange
2017-05-04 11:35 ` Fam Zheng
2017-05-04 14:01 ` Gerd Hoffmann [this message]
  -- strict thread matches above, loose matches on Subject: below --
2017-04-23 14:10 Alex K
2017-04-23 14:16 ` no-reply

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=1493906512.371.19.camel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=aliguori@amazon.com \
    --cc=qemu-devel@nongnu.org \
    --cc=vip-ak47@yandex.ru \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).