qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: qemu-devel@nongnu.org
Cc: dgilbert@redhat.com, lvivier@redhat.com, peterx@redhat.com
Subject: [Qemu-devel] [PATCH v3 00/12] Migration mini-cleanup
Date: Fri, 12 May 2017 18:00:43 +0200	[thread overview]
Message-ID: <20170512160055.9081-1-quintela@redhat.com> (raw)

Hi

Due to popular demand, and to make reviews easier, I just split the
big cleanup series (41 patches) in small chunks.  This chunk just
include the easy patches:

- create several include files foo.h for functcions exported from foo.c
- split channel operations in channel.c
- move colo headers to migration/
- split vmstate-types.c from vmstate.c (now it only contains the interpreter)
- untangle qemu-file.h, vmstate.h and migration.h.  Now none include the other
  fix all callers.
- Remove uses of migration.h that are not needed

Please, review.

Thanks, Juan.


Juan Quintela (12):
  migration: Create migration/xbzrle.h
  migration: Split migration/channel.c for channel operations
  migration: Export qemu-file-channel.c functions in its own file
  migration: Remove migration.h from colo.h
  migration: Move colo.h to migration/
  migration: Move failover.h to migration/colo-failover.h
  migration: Move page_cache.c to migration/
  migration: Move qjson.h to migration/
  migration: Split vmstate-types.c from vmstate.c
  migration: Remove qemu-file.h from vmstate.h
  migration: Remove vmstate.h from migration.h
  migration: migration.h was not needed

 MAINTAINERS                                        |   4 +-
 Makefile.objs                                      |   1 -
 block/qed.c                                        |   1 -
 hw/i386/pc_q35.c                                   |   1 -
 hw/virtio/vhost-user.c                             |   1 -
 hw/virtio/vhost-vsock.c                            |   1 -
 hw/virtio/virtio.c                                 |   1 -
 include/hw/hw.h                                    |   1 +
 include/migration/migration.h                      |  13 +-
 include/migration/qemu-file.h                      |   4 -
 include/migration/vmstate.h                        |   3 -
 migration/Makefile.objs                            |   4 +-
 migration/block.c                                  |   2 +
 migration/channel.c                                |  70 +++
 migration/channel.h                                |  25 +
 migration/colo-comm.c                              |   4 +-
 migration/colo-failover.c                          |   4 +-
 .../failover.h => migration/colo-failover.h        |   0
 migration/colo.c                                   |   5 +-
 {include/migration => migration}/colo.h            |   1 -
 migration/exec.c                                   |   1 +
 migration/fd.c                                     |   1 +
 migration/migration.c                              |  54 +-
 page_cache.c => migration/page_cache.c             |   0
 {include/migration => migration}/page_cache.h      |   0
 migration/postcopy-ram.c                           |   1 +
 migration/qemu-file-channel.c                      |   1 +
 migration/qemu-file-channel.h                      |  21 +
 migration/qjson.c                                  |   2 +-
 {include/migration => migration}/qjson.h           |   0
 migration/ram.c                                    |   5 +-
 migration/rdma.c                                   |   1 +
 migration/savevm.c                                 |   1 +
 migration/socket.c                                 |   1 +
 migration/tls.c                                    |   1 +
 migration/vmstate-types.c                          | 676 +++++++++++++++++++++
 migration/vmstate.c                                | 671 +-------------------
 migration/xbzrle.c                                 |   2 +-
 migration/xbzrle.h                                 |  21 +
 monitor.c                                          |   1 -
 tests/Makefile.include                             |   4 +-
 tests/test-vmstate.c                               |   2 +
 tests/test-xbzrle.c                                |   2 +-
 43 files changed, 880 insertions(+), 735 deletions(-)
 create mode 100644 migration/channel.c
 create mode 100644 migration/channel.h
 rename include/migration/failover.h => migration/colo-failover.h (100%)
 rename {include/migration => migration}/colo.h (96%)
 rename page_cache.c => migration/page_cache.c (100%)
 rename {include/migration => migration}/page_cache.h (100%)
 create mode 100644 migration/qemu-file-channel.h
 rename {include/migration => migration}/qjson.h (100%)
 create mode 100644 migration/vmstate-types.c
 create mode 100644 migration/xbzrle.h

-- 
2.9.3

             reply	other threads:[~2017-05-12 16:01 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-12 16:00 Juan Quintela [this message]
2017-05-12 16:00 ` [Qemu-devel] [PATCH 01/12] migration: Create migration/xbzrle.h Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 02/12] migration: Split migration/channel.c for channel operations Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 03/12] migration: Export qemu-file-channel.c functions in its own file Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 04/12] migration: Remove migration.h from colo.h Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 05/12] migration: Move colo.h to migration/ Juan Quintela
2017-05-12 17:51   ` Dr. David Alan Gilbert
2017-05-15 11:04     ` Juan Quintela
2017-05-15 12:44       ` Hailiang Zhang
2017-05-12 16:00 ` [Qemu-devel] [PATCH 06/12] migration: Move failover.h to migration/colo-failover.h Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 07/12] migration: Move page_cache.c to migration/ Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 08/12] migration: Move qjson.h " Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 09/12] migration: Split vmstate-types.c from vmstate.c Juan Quintela
2017-05-12 17:54   ` Dr. David Alan Gilbert
2017-05-15 11:05     ` Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 10/12] migration: Remove qemu-file.h from vmstate.h Juan Quintela
2017-05-12 16:00 ` [Qemu-devel] [PATCH 11/12] migration: Remove vmstate.h from migration.h Juan Quintela
2017-05-13 21:09   ` Philippe Mathieu-Daudé
2017-05-12 16:00 ` [Qemu-devel] [PATCH 12/12] migration: migration.h was not needed Juan Quintela
2017-05-12 17:58   ` Dr. David Alan Gilbert
2017-05-15 11:10     ` Juan Quintela

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=20170512160055.9081-1-quintela@redhat.com \
    --to=quintela@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=lvivier@redhat.com \
    --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).