mbox series

[0/2] *** Properly apply migration compression level

Message ID 20240301035901.4006936-1-bryan.zhang@bytedance.com (mailing list archive)
Headers show
Series *** Properly apply migration compression level | expand

Message

Bryan Zhang March 1, 2024, 3:58 a.m. UTC
From: Bryan Zhang <bryan.zhang@BYTEDANCE.COM>

There is some glue code missing, such that the
`qmp_migrate_set_parameters` function does not properly update the
`multifd_zstd_level` and `multifd_zlib_level` parameters. This patch
adds the glue code and also adds some function calls to the existing
migration tests to make sure the set_parameters function gets tested.

Bryan Zhang (2):
  migration: Properly apply migration compression level parameters
  tests/migration: Set compression level in migration tests

 migration/options.c          | 12 ++++++++++++
 tests/qtest/migration-test.c | 10 ++++++++++
 2 files changed, 22 insertions(+)

Comments

Peter Xu March 1, 2024, 4:12 a.m. UTC | #1
On Fri, Mar 01, 2024 at 03:58:59AM +0000, Bryan Zhang wrote:
> From: Bryan Zhang <bryan.zhang@BYTEDANCE.COM>
> 
> There is some glue code missing, such that the
> `qmp_migrate_set_parameters` function does not properly update the
> `multifd_zstd_level` and `multifd_zlib_level` parameters. This patch
> adds the glue code and also adds some function calls to the existing
> migration tests to make sure the set_parameters function gets tested.
> 
> Bryan Zhang (2):
>   migration: Properly apply migration compression level parameters
>   tests/migration: Set compression level in migration tests

queued, thanks!