[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/3] Add zero-copy-copied migration stat
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH v2 2/3] Add zero-copy-copied migration stat |
Date: |
Mon, 4 Jul 2022 12:40:09 +0100 |
User-agent: |
Mutt/2.2.6 (2022-06-05) |
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Mon, Jul 04, 2022 at 08:18:54AM +0200, Markus Armbruster wrote:
> > Leonardo Bras <leobras@redhat.com> writes:
> >
> > > Signed-off-by: Leonardo Bras <leobras@redhat.com>
> > > ---
> > > qapi/migration.json | 5 ++++-
> > > migration/migration.c | 1 +
> > > monitor/hmp-cmds.c | 4 ++++
> > > 3 files changed, 9 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/qapi/migration.json b/qapi/migration.json
> > > index 7102e474a6..925f009868 100644
> > > --- a/qapi/migration.json
> > > +++ b/qapi/migration.json
> > > @@ -55,6 +55,9 @@
> > > # @postcopy-bytes: The number of bytes sent during the post-copy phase
> > > # (since 7.0).
> > > #
> > > +# @zero-copy-copied: The number of zero-copy flushes that reported data
> > > sent
> > > +# using zero-copy that ended up being copied. (since
> > > 7.2)
> >
> > The description feels awkward. What's a "zero-copy flush", and why
> > should the user care? I figure what users care about is the number of
> > all-zero pages we had to "copy", i.e. send the bulky way. Is this what
> > @zero-copy-copied reports?
>
> MigrationCapability field @zero-copy-send instructs QEMU to try to
> avoid copying data between userspace and kernel space when transmitting
> RAM region.
>
> Even if the kernel supports zero copy, it is not guaranteed to happen,
> it is merely a request to try.
>
> QEMU periodically (once per migration iteration) flushes outstanding
> zero-copy requests and gets an indication back of whether any copies
> took place or not.
>
> So this counter is a reflection of how many iterations resulted in
> zero-copy not being fully honoured.
>
> IOW, ideally this counter will always be zero. If it is non-zero,
> then the magnitude gives a very very very rough guide to what's
> going on. If it is '1' then it was just a transient limitation.
> If it matches the number of migration iterations, then it is a
> more systemic limitation.
>
> Incidentally, do we report the migration iteration count ? I
> thought we did, but i'm not finding it now that I look.
Yes we do; it's dirty-sync-count
Dave
>
> With 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
- [PATCH v2 0/3] Zero copy improvements (QIOChannel + multifd), Leonardo Bras, 2022/07/01
- [PATCH v2 1/3] QIOChannelSocket: Fix zero-copy flush returning code 1 when nothing sent, Leonardo Bras, 2022/07/01
- [PATCH v2 2/3] Add zero-copy-copied migration stat, Leonardo Bras, 2022/07/01
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Markus Armbruster, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Daniel P . Berrangé, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Markus Armbruster, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat,
Dr. David Alan Gilbert <=
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Markus Armbruster, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Daniel P . Berrangé, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Markus Armbruster, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Daniel P . Berrangé, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Leonardo Brás, 2022/07/04
- Re: [PATCH v2 2/3] Add zero-copy-copied migration stat, Markus Armbruster, 2022/07/05
[PATCH v2 3/3] migration/multifd: Warn user when zerocopy not working, Leonardo Bras, 2022/07/01