qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] bab01e: migration: Make sure that we pass the


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] bab01e: migration: Make sure that we pass the right cache ...
Date: Mon, 30 Oct 2017 06:01:08 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: bab01ed4e87a75ce2e779cf0a79b74ca97fd29ea
      
https://github.com/qemu/qemu/commit/bab01ed4e87a75ce2e779cf0a79b74ca97fd29ea
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M migration/page_cache.c

  Log Message:
  -----------
  migration: Make sure that we pass the right cache size

Instead of passing silently round down the number of pages, make it an
error that the cache size is not a power of 2.

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


  Commit: 2a313e5cf6ed90b932b0abe2b4f2055785397f93
      
https://github.com/qemu/qemu/commit/2a313e5cf6ed90b932b0abe2b4f2055785397f93
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M migration/ram.c

  Log Message:
  -----------
  migration: Don't play games with the requested cache size

Now that we check that the value passed is a power of 2, we don't need
to play games when comparing what is the size that is going to take
the cache.

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


  Commit: c9dede2d482676440cb6e826ebe87450965fe679
      
https://github.com/qemu/qemu/commit/c9dede2d482676440cb6e826ebe87450965fe679
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M migration/migration.c
    M migration/ram.c
    M migration/ram.h

  Log Message:
  -----------
  migration: No need to return the size of the cache

After the previous commits, we make sure that the value passed is
right, or we just drop an error.  So now we return if there is one
error or we have setup correctly the value passed.

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

--

Improve error messasge
Return 0 always for success


  Commit: 73af8dd8d75a3e1f13e6c5d2a509fa56eb406519
      
https://github.com/qemu/qemu/commit/73af8dd8d75a3e1f13e6c5d2a509fa56eb406519
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M hmp.c
    M migration/migration.c
    M migration/migration.h
    M migration/ram.c
    M qapi/migration.json

  Log Message:
  -----------
  migration: Make xbzrle_cache_size a migration parameter

Right now it is a variable in MigrationState instead of a
MigrationParameter.  The change allows to set it as the rest of the
Migration parameters, from the command line, with
query_migration_paramters, set_migrate_parameters, etc.

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


  Commit: 2656bfd9731d430f2767f90906a3f1b1bfbf62d2
      
https://github.com/qemu/qemu/commit/2656bfd9731d430f2767f90906a3f1b1bfbf62d2
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M MAINTAINERS
    M tests/Makefile.include
    A tests/migration-test.c
    R tests/postcopy-test.c

  Log Message:
  -----------
  tests: rename postcopy-test to migration-test

Instead of repeating the code, we are going to bo more tests on this file

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


  Commit: d62fbe603969c54f39eaed2876238c574cbed5cb
      
https://github.com/qemu/qemu/commit/d62fbe603969c54f39eaed2876238c574cbed5cb
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  tests: Refactor setting of parameters/capabilities

So we can use them in future tests

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


  Commit: 7195a87130b2d58b27a0d5b1641a26e48423348b
      
https://github.com/qemu/qemu/commit/7195a87130b2d58b27a0d5b1641a26e48423348b
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  tests: Factorize out migrate_test_start/end

We fix global_test users left and right

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


  Commit: 863e27a8fc85c7d1fb20c5a549d3e45ad835702e
      
https://github.com/qemu/qemu/commit/863e27a8fc85c7d1fb20c5a549d3e45ad835702e
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  tests: Don't abuse global_qtest

As we have two guests running, just pass always who we want to send a
message to.  Once there, refactor return_or_event() into wait_command.

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


  Commit: 56b4a42a2e0ae74cee629abcb82993e79deeb356
      
https://github.com/qemu/qemu/commit/56b4a42a2e0ae74cee629abcb82993e79deeb356
  Author: Juan Quintela <address@hidden>
  Date:   2017-10-29 (Sun, 29 Oct 2017)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  tests: check that migration parameters are really assigned

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


  Commit: 21dab18b53f37664c2046c5bba91b416705ccdf1
      
https://github.com/qemu/qemu/commit/21dab18b53f37664c2046c5bba91b416705ccdf1
  Author: Peter Maydell <address@hidden>
  Date:   2017-10-30 (Mon, 30 Oct 2017)

  Changed paths:
    M MAINTAINERS
    M hmp.c
    M migration/migration.c
    M migration/migration.h
    M migration/page_cache.c
    M migration/ram.c
    M migration/ram.h
    M qapi/migration.json
    M tests/Makefile.include
    A tests/migration-test.c
    R tests/postcopy-test.c

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

migration/next for 20171029

# gpg: Signature made Sun 29 Oct 2017 13:07:43 GMT
# 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/20171029:
  tests: check that migration parameters are really assigned
  tests: Don't abuse global_qtest
  tests: Factorize out migrate_test_start/end
  tests: Refactor setting of parameters/capabilities
  tests: rename postcopy-test to migration-test
  migration: Make xbzrle_cache_size a migration parameter
  migration: No need to return the size of the cache
  migration: Don't play games with the requested cache size
  migration: Make sure that we pass the right cache size

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


Compare: https://github.com/qemu/qemu/compare/ab752f237d75...21dab18b53f3

reply via email to

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