On Mon, Jul 11, 2022 at 06:11:12PM -0300, Leonardo Bras wrote: > Signed-off-by: Leonardo Bras <leob...@redhat.com> > Acked-by: Markus Armbruster <arm...@redhat.com> > Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> > --- > qapi/migration.json | 7 ++++++- > migration/migration.c | 2 ++ > monitor/hmp-cmds.c | 5 +++++ > 3 files changed, 13 insertions(+), 1 deletion(-) > > diff --git a/qapi/migration.json b/qapi/migration.json > index 7102e474a6..4a03e8f173 100644 > --- a/qapi/migration.json > +++ b/qapi/migration.json > @@ -55,6 +55,10 @@ > # @postcopy-bytes: The number of bytes sent during the post-copy phase > # (since 7.0). > # > +# @dirty-sync-missed-zero-copy: Number of times dirty RAM synchronization > could > +# not avoid copying dirty pages. This is > between > +# 0 and @dirty-sync-count * @multifd-channels. > +# (since 7.1) > # Since: 0.14 > ## > { 'struct': 'MigrationStats', > @@ -65,7 +69,8 @@ > 'postcopy-requests' : 'int', 'page-size' : 'int', > 'multifd-bytes' : 'uint64', 'pages-per-second' : 'uint64', > 'precopy-bytes' : 'uint64', 'downtime-bytes' : 'uint64', > - 'postcopy-bytes' : 'uint64' } } > + 'postcopy-bytes' : 'uint64', > + 'dirty-sync-missed-zero-copy' : 'uint64' } } > > ## > # @XBZRLECacheStats: > diff --git a/migration/migration.c b/migration/migration.c > index 78f5057373..048f7f8bdb 100644 > --- a/migration/migration.c > +++ b/migration/migration.c > @@ -1027,6 +1027,8 @@ static void populate_ram_info(MigrationInfo *info, > MigrationState *s) > info->ram->normal_bytes = ram_counters.normal * page_size; > info->ram->mbps = s->mbps; > info->ram->dirty_sync_count = ram_counters.dirty_sync_count; > + info->ram->dirty_sync_missed_zero_copy = > + ram_counters.dirty_sync_missed_zero_copy; > info->ram->postcopy_requests = ram_counters.postcopy_requests; > info->ram->page_size = page_size; > info->ram->multifd_bytes = ram_counters.multifd_bytes; > diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c > index ca98df0495..a6dc79e0d5 100644 > --- a/monitor/hmp-cmds.c > +++ b/monitor/hmp-cmds.c > @@ -307,6 +307,11 @@ void hmp_info_migrate(Monitor *mon, const QDict *qdict) > monitor_printf(mon, "postcopy ram: %" PRIu64 " kbytes\n", > info->ram->postcopy_bytes >> 10); > } > + if (info->ram->dirty_sync_missed_zero_copy) { > + monitor_printf(mon, > + "Zero-copy-send fallbacks happened: %" PRIu64 " > times\n", > + info->ram->dirty_sync_missed_zero_copy);
Thanks, this looks better. Though I think all the "dirty-sync" wordings are still kept there, assuming flush() is bound to dirty sync even it's not yet. Not a big deal, but let's still keep an eye on the follow up patches.. Acked-by: Peter Xu <pet...@redhat.com> -- Peter Xu