qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 0f42f6: migration: Use savevm_handlers instea


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 0f42f6: migration: Use savevm_handlers instead of loadvm c...
Date: Fri, 02 Jun 2017 07:18:55 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0f42f65781fd784d91730a1366bffcbb9ff47d98
      
https://github.com/qemu/qemu/commit/0f42f65781fd784d91730a1366bffcbb9ff47d98
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M migration/savevm.c

  Log Message:
  -----------
  migration: Use savevm_handlers instead of loadvm copy

There is no reason for having the loadvm_handlers at all.  There is
only one use, and we can use the savevm handlers.

We will remove the loadvm handlers on a following patch.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>

--

- Added load_version_id: version_id read from the stream (laurent)
- Added load_section_id: section_id read from the stream (dave)


  Commit: c2355ad47d60d04efb640de57cd719cb24ebe563
      
https://github.com/qemu/qemu/commit/c2355ad47d60d04efb640de57cd719cb24ebe563
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M include/migration/vmstate.h
    M include/qemu/typedefs.h
    M migration/migration.c
    M migration/savevm.c

  Log Message:
  -----------
  migration: loadvm handlers are not used

So we remove all traces of them.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 3a011c26bc74ab0fb4b5289a10fff13e46a02e8c
      
https://github.com/qemu/qemu/commit/3a011c26bc74ab0fb4b5289a10fff13e46a02e8c
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M migration/savevm.c

  Log Message:
  -----------
  migration: Remove section_id parameter from vmstate_load

Everything else assumes that we always load a device from its own
savevm handler.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 3482655bbc21d158ed0daaa294c890997238cc23
      
https://github.com/qemu/qemu/commit/3482655bbc21d158ed0daaa294c890997238cc23
  Author: Peter Xu <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: fix leak of src file on dst

The return path channel is possibly leaked. Fix it.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 660819b1df9f79cb8d3bb99c590476f131858e75
      
https://github.com/qemu/qemu/commit/660819b1df9f79cb8d3bb99c590476f131858e75
  Author: Peter Xu <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M migration/migration.c
    M migration/postcopy-ram.c

  Log Message:
  -----------
  migration: shut src return path unconditionally

We were do the shutting off only for postcopy. Now we do this as long as
the source return path is there.

Moving the cleanup of from_src_file there too.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 107da9acb5f2ec99dc5cb70041245a53e16bfcbd
      
https://github.com/qemu/qemu/commit/107da9acb5f2ec99dc5cb70041245a53e16bfcbd
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M hw/core/qdev.c
    M include/hw/acpi/memory_hotplug.h
    M include/hw/acpi/pcihp.h
    M include/hw/pci/shpc.h
    M target/alpha/cpu.c
    M target/hppa/cpu.c
    M target/s390x/cpu.c
    M target/tilegx/cpu.c

  Log Message:
  -----------
  migration: Remove unneeded includes of migration/vmstate.h

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 08a0aee15cba50bd082770b7596e75023163fe87
      
https://github.com/qemu/qemu/commit/08a0aee15cba50bd082770b7596e75023163fe87
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M hw/i2c/i2c-ddc.c
    M hw/intc/s390_flic.c
    M hw/intc/s390_flic_kvm.c
    M hw/s390x/s390-skeys.c
    M include/hw/hw.h
    A include/migration/qemu-file-types.h
    R include/migration/qemu-file.h
    M migration/block.c
    M migration/colo.c
    M migration/migration.c
    M migration/postcopy-ram.c
    M migration/qemu-file-channel.c
    M migration/qemu-file.c
    A migration/qemu-file.h
    M migration/ram.c
    M migration/rdma.c
    M migration/savevm.c
    M migration/socket.c
    M migration/vmstate-types.c
    M migration/vmstate.c
    M tests/test-vmstate.c

  Log Message:
  -----------
  migration: Split qemu-file.h

Split the file into public and internal interfaces.  I have to rename
the external one because we can't have two include files with the same
name in the same directory.  Build system gets confused.  The only
exported functions are the ones that handle basic types.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: f4dbe1bf346b199b63920992cd8f2ab97ef161a1
      
https://github.com/qemu/qemu/commit/f4dbe1bf346b199b63920992cd8f2ab97ef161a1
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M migration/exec.c
    A migration/exec.h
    M migration/migration.c

  Log Message:
  -----------
  migration: Export exec.c functions in its own file

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 7fcac4a2cc87cc124615b0341053e3ddb8398e47
      
https://github.com/qemu/qemu/commit/7fcac4a2cc87cc124615b0341053e3ddb8398e47
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M migration/fd.c
    A migration/fd.h
    M migration/migration.c

  Log Message:
  -----------
  migration: Export fd.c functions in its own file

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 61e8b14880858e25ae9b8591500f5440bd152b8c
      
https://github.com/qemu/qemu/commit/61e8b14880858e25ae9b8591500f5440bd152b8c
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M migration/migration.c
    M migration/socket.c
    A migration/socket.h

  Log Message:
  -----------
  migration: Export socket.c functions in its own file

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 41d64227ed0a8f843b022dbf3058aa51c240c8d9
      
https://github.com/qemu/qemu/commit/41d64227ed0a8f843b022dbf3058aa51c240c8d9
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M migration/channel.c
    M migration/migration.c
    M migration/tls.c
    A migration/tls.h

  Log Message:
  -----------
  migration: Export tls.c functions in its own file

Just for the functions exported from tls.c.  Notice that we can't
remove the migration/migration.h include from tls.c because it access
directly MigrationState for the tls params.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: e1a3ecee3b85864932d093c3a29966779d038485
      
https://github.com/qemu/qemu/commit/e1a3ecee3b85864932d093c3a29966779d038485
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    M migration/migration.c
    M migration/rdma.c
    A migration/rdma.h

  Log Message:
  -----------
  migration: Export rdma.c functions in its own file

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 5e22479ae241965575318ec081c26c907bfc3210
      
https://github.com/qemu/qemu/commit/5e22479ae241965575318ec081c26c907bfc3210
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M hmp.c
    M include/block/block_int.h
    A include/migration/snapshot.h
    M include/sysemu/sysemu.h
    M migration/savevm.c
    M replay/replay-snapshot.c
    M vl.c

  Log Message:
  -----------
  migration: Create include for migration snapshots

Start removing migration code from sysemu/sysemu.h.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 7b1e1a2202a492d492d4fb8ef25a5e648daed774
      
https://github.com/qemu/qemu/commit/7b1e1a2202a492d492d4fb8ef25a5e648daed774
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    M include/migration/migration.h
    A include/migration/misc.h
    M migration/migration.c
    M migration/postcopy-ram.c
    M migration/ram.c
    A migration/ram.h
    M migration/rdma.c
    M migration/savevm.c
    M vl.c

  Log Message:
  -----------
  migration: Export ram.c functions in its own file

All functions are internal except for ram_mig_init().  Create
migration/misc.h for this kind of functions.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 2c9e6fec89ff032a3f75a5a1caccc31901fb4056
      
https://github.com/qemu/qemu/commit/2c9e6fec89ff032a3f75a5a1caccc31901fb4056
  Author: Juan Quintela <address@hidden>
  Date:   2017-06-01 (Thu, 01 Jun 2017)

  Changed paths:
    R include/migration/block.h
    M include/migration/misc.h
    M migration/block.c
    A migration/block.h
    M migration/colo.c
    M migration/migration.c
    M vl.c

  Log Message:
  -----------
  migration: Move include/migration/block.h into migration/

All functions were internal, except blk_mig_init() that is exported in
misc.h now.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>


  Commit: d47a851caeda96d5979bf48d4bae6a87784ad91d
      
https://github.com/qemu/qemu/commit/d47a851caeda96d5979bf48d4bae6a87784ad91d
  Author: Peter Maydell <address@hidden>
  Date:   2017-06-02 (Fri, 02 Jun 2017)

  Changed paths:
    M hmp.c
    M hw/core/qdev.c
    M hw/i2c/i2c-ddc.c
    M hw/intc/s390_flic.c
    M hw/intc/s390_flic_kvm.c
    M hw/s390x/s390-skeys.c
    M include/block/block_int.h
    M include/hw/acpi/memory_hotplug.h
    M include/hw/acpi/pcihp.h
    M include/hw/hw.h
    M include/hw/pci/shpc.h
    R include/migration/block.h
    M include/migration/migration.h
    A include/migration/misc.h
    A include/migration/qemu-file-types.h
    R include/migration/qemu-file.h
    A include/migration/snapshot.h
    M include/migration/vmstate.h
    M include/qemu/typedefs.h
    M include/sysemu/sysemu.h
    M migration/block.c
    A migration/block.h
    M migration/channel.c
    M migration/colo.c
    M migration/exec.c
    A migration/exec.h
    M migration/fd.c
    A migration/fd.h
    M migration/migration.c
    M migration/postcopy-ram.c
    M migration/qemu-file-channel.c
    M migration/qemu-file.c
    A migration/qemu-file.h
    M migration/ram.c
    A migration/ram.h
    M migration/rdma.c
    A migration/rdma.h
    M migration/savevm.c
    M migration/socket.c
    A migration/socket.h
    M migration/tls.c
    A migration/tls.h
    M migration/vmstate-types.c
    M migration/vmstate.c
    M replay/replay-snapshot.c
    M target/alpha/cpu.c
    M target/hppa/cpu.c
    M target/s390x/cpu.c
    M target/tilegx/cpu.c
    M tests/test-vmstate.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/juanquintela/tags/migration/20170601' 
into staging

migration/next for 20170601

# gpg: Signature made Thu 01 Jun 2017 17:51:04 BST
# gpg:                using RSA key 0xF487EF185872D723
# gpg: Good signature from "Juan Quintela <address@hidden>"
# gpg:                 aka "Juan Quintela <address@hidden>"
# Primary key fingerprint: 1899 FF8E DEBF 58CC EE03  4B82 F487 EF18 5872 D723

* remotes/juanquintela/tags/migration/20170601:
  migration: Move include/migration/block.h into migration/
  migration: Export ram.c functions in its own file
  migration: Create include for migration snapshots
  migration: Export rdma.c functions in its own file
  migration: Export tls.c functions in its own file
  migration: Export socket.c functions in its own file
  migration: Export fd.c functions in its own file
  migration: Export exec.c functions in its own file
  migration: Split qemu-file.h
  migration: Remove unneeded includes of migration/vmstate.h
  migration: shut src return path unconditionally
  migration: fix leak of src file on dst
  migration: Remove section_id parameter from vmstate_load
  migration: loadvm handlers are not used
  migration: Use savevm_handlers instead of loadvm copy

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/7693cd7cb622...d47a851caeda

reply via email to

[Prev in Thread] Current Thread [Next in Thread]