All of lore.kernel.org
 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 v2 00/10] Another cleanup of includes
Date: Wed, 31 May 2017 12:34:59 +0200	[thread overview]
Message-ID: <20170531103509.22021-1-quintela@redhat.com> (raw)

Hi

Changes from v1:
- patch 1 got reviewed on the pull request
- drop one change from patch2 as lvivier comments
- fix include of snapshot.h

Please review.

Thanks, Juan.


{v1}
- make all migration methods export its functions in its own header
- move block.h into migration/
- create migration/savevm.h for functions exported in savevm.c
- create migration/ram.h for functions from ram.c
- split qemu-file.h into it an qemu-file-types.h
  only the last one is used outside of migration/
- move migration.h to be private
- create include/migration/misc.h for misc exported functions that
  don't belong anywhere




Juan Quintela (10):
  migration: Remove unneeded includes of migration/vmstate.h
  migration: Split qemu-file.h
  migration: Export exec.c functions in its own file
  migration: Export fd.c functions in its own file
  migration: Export socket.c functions in its own file
  migration: Export tls.c functions in its own file
  migration: Export rdma.c functions in its own file
  migration: Create include for migration snapshots
  migration: Export ram.c functions in its own file
  migration: Move include/migration/block.h into migration/

 hmp.c                                        |   5 +-
 hw/core/qdev.c                               |   1 -
 hw/i2c/i2c-ddc.c                             |   1 +
 hw/intc/s390_flic.c                          |   1 -
 hw/intc/s390_flic_kvm.c                      |   1 -
 hw/s390x/s390-skeys.c                        |   1 -
 include/block/block_int.h                    |   4 +-
 include/hw/acpi/memory_hotplug.h             |   1 -
 include/hw/acpi/pcihp.h                      |   1 -
 include/hw/hw.h                              |   2 +-
 include/hw/pci/shpc.h                        |   1 -
 include/migration/migration.h                |  68 -----------
 include/migration/misc.h                     |  29 +++++
 include/migration/qemu-file-types.h          | 164 +++++++++++++++++++++++++++
 include/migration/snapshot.h                 |  21 ++++
 include/sysemu/sysemu.h                      |   3 -
 migration/block.c                            |   5 +-
 {include/migration => migration}/block.h     |   2 -
 migration/channel.c                          |   1 +
 migration/colo.c                             |   4 +-
 migration/exec.c                             |   1 +
 migration/exec.h                             |  26 +++++
 migration/fd.c                               |   1 +
 migration/fd.h                               |  23 ++++
 migration/migration.c                        |  10 +-
 migration/postcopy-ram.c                     |   3 +-
 migration/qemu-file-channel.c                |   3 +-
 migration/qemu-file.c                        |   2 +-
 {include/migration => migration}/qemu-file.h | 157 +------------------------
 migration/ram.c                              |   4 +-
 migration/ram.h                              |  70 ++++++++++++
 migration/rdma.c                             |   5 +-
 migration/rdma.h                             |  25 ++++
 migration/savevm.c                           |   7 +-
 migration/socket.c                           |   3 +-
 migration/socket.h                           |  28 +++++
 migration/tls.c                              |   1 +
 migration/tls.h                              |  34 ++++++
 migration/vmstate-types.c                    |   3 +-
 migration/vmstate.c                          |   2 +-
 replay/replay-snapshot.c                     |   5 +-
 target/alpha/cpu.c                           |   1 -
 target/hppa/cpu.c                            |   1 -
 target/s390x/cpu.c                           |   1 -
 target/tilegx/cpu.c                          |   1 -
 tests/test-vmstate.c                         |   3 +-
 vl.c                                         |   5 +-
 47 files changed, 476 insertions(+), 265 deletions(-)
 create mode 100644 include/migration/misc.h
 create mode 100644 include/migration/qemu-file-types.h
 create mode 100644 include/migration/snapshot.h
 rename {include/migration => migration}/block.h (93%)
 create mode 100644 migration/exec.h
 create mode 100644 migration/fd.h
 rename {include/migration => migration}/qemu-file.h (62%)
 create mode 100644 migration/ram.h
 create mode 100644 migration/rdma.h
 create mode 100644 migration/socket.h
 create mode 100644 migration/tls.h

-- 
2.9.4

             reply	other threads:[~2017-05-31 10:35 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-31 10:34 Juan Quintela [this message]
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 01/10] migration: Remove unneeded includes of migration/vmstate.h Juan Quintela
2017-06-01 12:00   ` Dr. David Alan Gilbert
2017-06-01 12:03   ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 02/10] migration: Split qemu-file.h Juan Quintela
2017-06-01 12:21   ` Dr. David Alan Gilbert
2017-06-01 15:19     ` Juan Quintela
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 03/10] migration: Export exec.c functions in its own file Juan Quintela
2017-06-01 12:28   ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 04/10] migration: Export fd.c " Juan Quintela
2017-06-01 12:29   ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 05/10] migration: Export socket.c " Juan Quintela
2017-06-01 12:37   ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 06/10] migration: Export tls.c " Juan Quintela
2017-06-01 12:41   ` Dr. David Alan Gilbert
2017-06-01 15:20     ` Juan Quintela
2017-06-01 15:56       ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 07/10] migration: Export rdma.c " Juan Quintela
2017-06-01 12:46   ` Dr. David Alan Gilbert
2017-06-01 15:22     ` Juan Quintela
2017-06-01 15:51       ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 08/10] migration: Create include for migration snapshots Juan Quintela
2017-06-01 12:43   ` Dr. David Alan Gilbert
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 09/10] migration: Export ram.c functions in its own file Juan Quintela
2017-05-31 10:35 ` [Qemu-devel] [PATCH v2 10/10] migration: Move include/migration/block.h into migration/ Juan Quintela
2017-06-01 12:44   ` Dr. David Alan Gilbert

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=20170531103509.22021-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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.