From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: "Daniel P. Berrangé" <berrange@redhat.com>
Cc: Jim Fehlig <jfehlig@suse.com>,
Juan Quintela <quintela@redhat.com>,
Claudio Fontana <cfontana@suse.de>,
qemu-devel <qemu-devel@nongnu.org>
Subject: Re: bad virsh save /dev/null performance (600 MiB/s max)
Date: Wed, 9 Mar 2022 18:39:44 +0000 [thread overview]
Message-ID: <Yij0cJ3ioN6aiKgA@work-vm> (raw)
In-Reply-To: <Yijz9hzJFJoMo7vE@redhat.com>
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Wed, Mar 09, 2022 at 07:27:12PM +0100, Claudio Fontana wrote:
> >
> > One difference I could see looking at the qmp commands issued by libvirt in the "virsh save" case,
> > is "detach:true" in the migration command (which seems to have no effect in qemu),
>
> That is a bug in libvirt - it should not be setting that in QMP.
>
> To quote the QAPI spec for 'migrate'
>
> # @detach: this argument exists only for compatibility reasons and
> # is ignored by QEMU
>
>
> >
> >
> > and maybe more interestingly this stuff about the "fd":
> >
> >
> > 2022-03-09 17:29:34.247+0000: 20390: info : qemuMonitorSend:995 : QEMU_MONITOR_SEND_MSG: mon=0x7faa9003ebf0 msg={"execute":"getfd","arguments":{"fdname":"migrate"},"id":"libvirt-390"}^M
> > fd=34
> > 2022-03-09 17:29:34.247+0000: 20387: info : qemuMonitorIOWrite:452 : QEMU_MONITOR_IO_WRITE: mon=0x7faa9003ebf0 buf={"execute":"getfd","arguments":{"fdname":"migrate"},"id":"libvirt-390"}^M
> > len=73 ret=73 errno=0
> > 2022-03-09 17:29:34.247+0000: 20387: info : qemuMonitorIOWrite:457 : QEMU_MONITOR_IO_SEND_FD: mon=0x7faa9003ebf0 fd=34 ret=73 errno=0
> > 2022-03-09 17:29:34.248+0000: 20387: info : qemuMonitorJSONIOProcessLine:240 : QEMU_MONITOR_RECV_REPLY: mon=0x7faa9003ebf0 reply={"return": {}, "id": "libvirt-390"}
> > 2022-03-09 17:29:34.249+0000: 20390: info : qemuMonitorSend:995 : QEMU_MONITOR_SEND_MSG: mon=0x7faa9003ebf0 msg={"execute":"migrate","arguments":{"detach":true,"blk":false,"inc":false,"uri":"fd:migrate"},"id":"libvirt-391"}^M
> > fd=-1
> > 2022-03-09 17:29:34.249+0000: 20387: info : qemuMonitorIOWrite:452 : QEMU_MONITOR_IO_WRITE: mon=0x7faa9003ebf0 buf={"execute":"migrate","arguments":{"detach":true,"blk":false,"inc":false,"uri":"fd:migrate"},"id":"libvirt-391"}^M
> > len=113 ret=113 errno=0
> >
> >
> > in qemu I am currently looking at the code in migration/socket.c
> > vs the code in migration/fd.c , wonder if the difference would
> > stem from there..
>
> When saving to a file, libvirt passes in a pre-opened FD for
> QEU to use. IIRC this should always be a pipe FD connected to
> libvirt's iohelper program, sometimes indirectly via a compression
> program.
In which case, what is a simple 'top' showing on the system?
Dave
> Regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2022-03-09 18:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <8826b03d-e5e9-0e65-cab7-ea1829f48e6c@suse.de>
[not found] ` <YiXQHIWtHx5BocxK@redhat.com>
[not found] ` <62ba8b1e-d641-5b10-c1b3-54b7d5a652e7@suse.de>
[not found] ` <YiXVh1P4oJNuEtFM@redhat.com>
2022-03-07 11:19 ` starting to look at qemu savevm performance, a first regression detected Claudio Fontana
2022-03-07 12:00 ` Daniel P. Berrangé
2022-03-07 12:09 ` Claudio Fontana
2022-03-07 12:20 ` Daniel P. Berrangé
2022-03-07 12:26 ` Claudio Fontana
2022-03-07 12:28 ` Dr. David Alan Gilbert
2022-03-09 11:15 ` bad qemu savevm to /dev/null performance (600 MiB/s max) (Was: Re: starting to look at qemu savevm performance, a first regression detected) Claudio Fontana
2022-03-09 11:43 ` Dr. David Alan Gilbert
2022-03-09 11:51 ` Daniel P. Berrangé
2022-03-09 14:22 ` Claudio Fontana
2022-03-09 18:27 ` bad virsh save /dev/null performance (600 MiB/s max) Claudio Fontana
2022-03-09 18:37 ` Daniel P. Berrangé
2022-03-09 18:39 ` Dr. David Alan Gilbert [this message]
2022-03-09 18:47 ` Claudio Fontana
2022-03-09 18:53 ` Dr. David Alan Gilbert
2022-03-09 18:46 ` Daniel P. Berrangé
2022-03-10 15:25 ` Claudio Fontana
2022-03-09 13:16 ` bad qemu savevm to /dev/null performance (600 MiB/s max) (Was: Re: starting to look at qemu savevm performance, a first regression detected) Claudio Fontana
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=Yij0cJ3ioN6aiKgA@work-vm \
--to=dgilbert@redhat.com \
--cc=berrange@redhat.com \
--cc=cfontana@suse.de \
--cc=jfehlig@suse.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/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).