qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fabiano Rosas <farosas@suse.de>
To: qemu-devel@nongnu.org
Cc: "Daniel P . Berrangé" <berrange@redhat.com>,
	"Peter Xu" <peterx@redhat.com>,
	devel@lists.libvirt.org
Subject: [PATCH v3 0/6] migration removals & deprecations
Date: Tue, 30 Apr 2024 11:27:31 -0300	[thread overview]
Message-ID: <20240430142737.29066-1-farosas@suse.de> (raw)

since v2:

- removed some more stuff which I missed:
   blk/inc options from hmp-commands.hx
   the entire ram-compress.h
   unused declarations from options.h
   unused compression functions from qemu-file.c

- removed must_remove_block_options earlier in the 'blk' patch

- added a deprecation warning to outgoing/incoming fd

CI run: https://gitlab.com/farosas/qemu/-/pipelines/1272385260

v2:
https://lore.kernel.org/r/20240426131408.25410-1-farosas@suse.de
v1:
https://lore.kernel.org/r/20240425150939.19268-1-farosas@suse.de

Hi everyone,

Here's some cleaning up of deprecated code. It removes the old block
migration and compression code. Both have suitable replacements in the
form of the blockdev-mirror driver and multifd compression,
respectively.

There's also a deprecation for fd: + file to cope with the fact that
the new MigrationAddress API defines transports instead of protocols
(loose terms) like the old string API did. So we cannot map 1:1 from
fd: to any transport because fd: allows *both* file migration and
socket migration.

Fabiano Rosas (6):
  migration: Remove 'skipped' field from MigrationStats
  migration: Remove 'inc' option from migrate command
  migration: Remove 'blk/-b' option from migrate commands
  migration: Remove block migration
  migration: Remove non-multifd compression
  migration: Deprecate fd: for file migration

 .gitlab-ci.d/buildtest.yml       |    2 +-
 MAINTAINERS                      |    1 -
 docs/about/deprecated.rst        |   51 +-
 docs/about/removed-features.rst  |  103 +++
 docs/devel/migration/main.rst    |    2 +-
 hmp-commands.hx                  |   17 +-
 hw/core/machine.c                |    1 -
 include/migration/misc.h         |    6 -
 meson.build                      |    2 -
 meson_options.txt                |    2 -
 migration/block.c                | 1019 ------------------------------
 migration/block.h                |   52 --
 migration/colo.c                 |    1 -
 migration/fd.c                   |   12 +
 migration/meson.build            |    4 -
 migration/migration-hmp-cmds.c   |   97 +--
 migration/migration.c            |   70 +-
 migration/migration.h            |   11 -
 migration/options.c              |  229 -------
 migration/options.h              |   13 -
 migration/qemu-file.c            |   78 ---
 migration/qemu-file.h            |    4 -
 migration/ram-compress.c         |  564 -----------------
 migration/ram-compress.h         |   77 ---
 migration/ram.c                  |  169 +----
 migration/savevm.c               |    5 -
 qapi/migration.json              |  205 +-----
 scripts/meson-buildoptions.sh    |    4 -
 tests/qemu-iotests/183           |  147 -----
 tests/qemu-iotests/183.out       |   66 --
 tests/qemu-iotests/common.filter |    7 -
 tests/qtest/migration-test.c     |  139 ----
 32 files changed, 147 insertions(+), 3013 deletions(-)
 delete mode 100644 migration/block.c
 delete mode 100644 migration/block.h
 delete mode 100644 migration/ram-compress.c
 delete mode 100644 migration/ram-compress.h
 delete mode 100755 tests/qemu-iotests/183
 delete mode 100644 tests/qemu-iotests/183.out


base-commit: fd87be1dada5672f877e03c2ca8504458292c479
-- 
2.35.3



             reply	other threads:[~2024-04-30 14:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-30 14:27 Fabiano Rosas [this message]
2024-04-30 14:27 ` [PATCH v3 1/6] migration: Remove 'skipped' field from MigrationStats Fabiano Rosas
2024-04-30 14:27 ` [PATCH v3 2/6] migration: Remove 'inc' option from migrate command Fabiano Rosas
2024-04-30 14:27 ` [PATCH v3 3/6] migration: Remove 'blk/-b' option from migrate commands Fabiano Rosas
2024-05-02 11:53   ` Markus Armbruster
2024-05-02 13:35     ` Dr. David Alan Gilbert
2024-05-02 14:49       ` Peter Xu
2024-04-30 14:27 ` [PATCH v3 4/6] migration: Remove block migration Fabiano Rosas
2024-04-30 14:27 ` [PATCH v3 5/6] migration: Remove non-multifd compression Fabiano Rosas
2024-04-30 14:27 ` [PATCH v3 6/6] migration: Deprecate fd: for file migration Fabiano Rosas
2024-05-01 15:53 ` [PATCH v3 0/6] migration removals & deprecations Peter Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240430142737.29066-1-farosas@suse.de \
    --to=farosas@suse.de \
    --cc=berrange@redhat.com \
    --cc=devel@lists.libvirt.org \
    --cc=peterx@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).