On Thu, Feb 29, 2024 at 11:28 PM Markus Armbruster <arm...@redhat.com> wrote:
>
> Hao Xiang <hao.xi...@bytedance.com> writes:
>
> > 1. Add zero_pages field in MultiFDPacket_t.
> > 2. Implements the zero page detection and handling on the multifd
> > threads for non-compression, zlib and zstd compression backends.
> > 3. Added a new value 'multifd' in ZeroPageDetection enumeration.
> > 4. Handle migration QEMU9.0 -> QEMU8.2 compatibility.
> > 5. Adds zero page counters and updates multifd send/receive tracing
> > format to track the newly added counters.
> >
> > Signed-off-by: Hao Xiang <hao.xi...@bytedance.com>
>
> [...]
>
> > diff --git a/qapi/migration.json b/qapi/migration.json
> > index 8da05dba47..846d0411d5 100644
> > --- a/qapi/migration.json
> > +++ b/qapi/migration.json
> > @@ -671,10 +671,15 @@
> > #
> > # @legacy: Perform zero page checking in main migration thread.
> > #
> > +# @multifd: Perform zero page checking in multifd sender thread.
> > +# This option only takes effect if migration capability multifd
> > +# is set. Otherwise, it will have the same effect as legacy.
>
> Suggest
>
> # @multifd: Perform zero page checking in multifd sender thread if
> # multifd migration is enabled, else in the main migration
> # thread as for @legacy.
>
> Thoughts?
Sounds good. Will change that.
>
> > +#
> > # Since: 9.0
> > +#
> > ##
> > { 'enum': 'ZeroPageDetection',
> > - 'data': [ 'none', 'legacy' ] }
> > + 'data': [ 'none', 'legacy', 'multifd' ] }
> >
> > ##
> > # @BitmapMigrationBitmapAliasTransform:
>
> QAPI schema
> Acked-by: Markus Armbruster <arm...@redhat.com>
>
