[PULL 00/13] Compression code patches

Juan Quintela posted 13 patches 11 months, 3 weeks ago
Failed in applying to current master (apply log)
Maintainers: Juan Quintela <quintela@redhat.com>, Peter Xu <peterx@redhat.com>, Leonardo Bras <leobras@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
migration/meson.build        |   6 +-
migration/migration.c        |   9 +
migration/qemu-file.c        |  11 +
migration/qemu-file.h        |   1 +
migration/ram-compress.c     | 485 +++++++++++++++++++++++++++++++++
migration/ram-compress.h     |  70 +++++
migration/ram.c              | 502 +++--------------------------------
tests/qtest/migration-test.c | 134 ++++++++++
8 files changed, 758 insertions(+), 460 deletions(-)
create mode 100644 migration/ram-compress.c
create mode 100644 migration/ram-compress.h
[PULL 00/13] Compression code patches
Posted by Juan Quintela 11 months, 3 weeks ago
The following changes since commit 792f77f376adef944f9a03e601f6ad90c2f891b2:

  Merge tag 'pull-loongarch-20230506' of https://gitlab.com/gaosong/qemu into staging (2023-05-06 08:11:52 +0100)

are available in the Git repository at:

  https://gitlab.com/juan.quintela/qemu.git tags/compression-code-pull-request

for you to fetch changes up to c323518a7aab1c01740a468671b7f2b517d3bca6:

  migration: Initialize and cleanup decompression in migration.c (2023-05-08 15:25:27 +0200)

----------------------------------------------------------------
Migration PULL request (20230508 edition, take 2)

Hi

This is just the compression bits of the Migration PULL request for
20230428.  Only change is that we don't run the compression tests by
default.

The problem already exist with compression code.  The test just show
that it don't work.

- Add migration tests for (old) compress migration code (lukas)
- Make compression code independent of ram.c (lukas)
- Move compression code into ram-compress.c (lukas)

Please apply, Juan.

----------------------------------------------------------------

Lukas Straub (13):
  qtest/migration-test.c: Add tests with compress enabled
  qtest/migration-test.c: Add postcopy tests with compress enabled
  ram.c: Let the compress threads return a CompressResult enum
  ram.c: Dont change param->block in the compress thread
  ram.c: Reset result after sending queued data
  ram.c: Do not call save_page_header() from compress threads
  ram.c: Call update_compress_thread_counts from
    compress_send_queued_data
  ram.c: Remove last ram.c dependency from the core compress code
  ram.c: Move core compression code into its own file
  ram.c: Move core decompression code into its own file
  ram compress: Assert that the file buffer matches the result
  ram-compress.c: Make target independent
  migration: Initialize and cleanup decompression in migration.c

 migration/meson.build        |   6 +-
 migration/migration.c        |   9 +
 migration/qemu-file.c        |  11 +
 migration/qemu-file.h        |   1 +
 migration/ram-compress.c     | 485 +++++++++++++++++++++++++++++++++
 migration/ram-compress.h     |  70 +++++
 migration/ram.c              | 502 +++--------------------------------
 tests/qtest/migration-test.c | 134 ++++++++++
 8 files changed, 758 insertions(+), 460 deletions(-)
 create mode 100644 migration/ram-compress.c
 create mode 100644 migration/ram-compress.h

-- 
2.40.0
Re: [PULL 00/13] Compression code patches
Posted by Richard Henderson 11 months, 3 weeks ago
On 5/8/23 19:51, Juan Quintela wrote:
> The following changes since commit 792f77f376adef944f9a03e601f6ad90c2f891b2:
> 
>    Merge tag 'pull-loongarch-20230506' ofhttps://gitlab.com/gaosong/qemu  into staging (2023-05-06 08:11:52 +0100)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/juan.quintela/qemu.git  tags/compression-code-pull-request
> 
> for you to fetch changes up to c323518a7aab1c01740a468671b7f2b517d3bca6:
> 
>    migration: Initialize and cleanup decompression in migration.c (2023-05-08 15:25:27 +0200)
> 
> ----------------------------------------------------------------
> Migration PULL request (20230508 edition, take 2)
> 
> Hi
> 
> This is just the compression bits of the Migration PULL request for
> 20230428.  Only change is that we don't run the compression tests by
> default.
> 
> The problem already exist with compression code.  The test just show
> that it don't work.
> 
> - Add migration tests for (old) compress migration code (lukas)
> - Make compression code independent of ram.c (lukas)
> - Move compression code into ram-compress.c (lukas)
> 
> Please apply, Juan.

Applied, thanks.  Please update https://wiki.qemu.org/ChangeLog/8.1 as appropriate.


r~