[Qemu-commits] [qemu/qemu] f44b55: tests/boot_linux_console: Fetch assets from Debian...

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: f44b5549efe97975805ed3e7be1af49361b60ce7
      
https://github.com/qemu/qemu/commit/f44b5549efe97975805ed3e7be1af49361b60ce7
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/acceptance/boot_linux_console.py

  Log Message:
  -----------
  tests/boot_linux_console: Fetch assets from Debian snapshot archives

The kernel packaged was fetched from an unstable repository.
Use the stable snapshot archive instead.

Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Tested-by: Willian Rampazzo <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: b67d22aa7003310185f9571c27a8090d9314a6a1
      
https://github.com/qemu/qemu/commit/b67d22aa7003310185f9571c27a8090d9314a6a1
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/acceptance/boot_linux_console.py

  Log Message:
  -----------
  Revert "Acceptance test: cancel test if m68k kernel packages goes missing"

Now than we use the stable snapshot archive, we can remove this check.

This reverts commit d2499aca4bac613d837e2720e7bbe3378bc91245.

Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Tested-by: Willian Rampazzo <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 13ae8cdb9ba2c5d7a0bec155d9415dff8d0324c6
      
https://github.com/qemu/qemu/commit/13ae8cdb9ba2c5d7a0bec155d9415dff8d0324c6
  Author: Marc-André Lureau <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M scripts/analyze-migration.py

  Log Message:
  -----------
  analyze-migration.py: fix find() type error

Traceback (most recent call last):
  File "../scripts/analyze-migration.py", line 611, in <module>
    dump.read(desc_only = True)
  File "../scripts/analyze-migration.py", line 513, in read
    self.load_vmsd_json(file)
  File "../scripts/analyze-migration.py", line 556, in load_vmsd_json
    vmsd_json = file.read_migration_debug_json()
  File "../scripts/analyze-migration.py", line 89, in read_migration_debug_json
    nulpos = data.rfind("\0")
TypeError: argument should be integer or bytes-like object, not 'str'

Signed-off-by: Marc-André Lureau <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 83de0ea4ad0e3bbc20c6f0911a71966b2ec7773c
      
https://github.com/qemu/qemu/commit/83de0ea4ad0e3bbc20c6f0911a71966b2ec7773c
  Author: Marc-André Lureau <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M scripts/analyze-migration.py

  Log Message:
  -----------
  analyze-migration.py: replace numpy with python 3.2

Use int.from_bytes() from python 3.2 instead.

Signed-off-by: Marc-André Lureau <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 74b56bb5539553d93908d6acb2bc2c5e2f019f7d
      
https://github.com/qemu/qemu/commit/74b56bb5539553d93908d6acb2bc2c5e2f019f7d
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/machine.py

  Log Message:
  -----------
  python/qemu: Add set_qmp_monitor() to QEMUMachine

The QEMUMachine VM has a monitor setup on which an QMP
connection is always attempted on _post_launch() (executed
by launch()). In case the QEMU process immediatly exits
then the qmp.accept() (used to establish the connection) stalls
until it reaches timeout and consequently an exception raises.

That behavior is undesirable when, for instance, it needs to
gather information from the QEMU binary ($ qemu -cpu list) or a
test which launches the VM expecting its failure.

This patch adds the set_qmp_monitor() method to QEMUMachine that
allows turn off the creation of the monitor machinery on VM launch.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
[Cleber: trivial indentation fix]
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: c80c6beb171c3175a14ddb99217dc7b89354a9a3
      
https://github.com/qemu/qemu/commit/c80c6beb171c3175a14ddb99217dc7b89354a9a3
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/acceptance/empty_cpu_model.py
    M tests/acceptance/linux_initrd.py

  Log Message:
  -----------
  tests/acceptance: Makes linux_initrd and empty_cpu_model use QEMUMachine

On linux_initrd and empty_cpu_model tests the same effect of
calling QEMU through run() to inspect the terminated process is
achieved with a sequence of set_qmp_monitor() / launch() / wait()
commands on an QEMUMachine object. This patch changes those
tests to use QEMUMachine instead, so they follow the same pattern
to launch QEMU found on other acceptance tests.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 3e9c1810c65f8338a863d1e59fa0efedc884e915
      
https://github.com/qemu/qemu/commit/3e9c1810c65f8338a863d1e59fa0efedc884e915
  Author: Cleber Rosa <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/acceptance/x86_cpu_model_versions.py

  Log Message:
  -----------
  Acceptance test x86_cpu_model_versions: use default vm

The default vm provided by the test, available as self.vm, serves the
same purpose of the one obtained by self.get_vm(), but saves a line
and matches the style of other tests.

Signed-off-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: f108934fca0a5d6acfc93be6d3684574d18794e2
      
https://github.com/qemu/qemu/commit/f108934fca0a5d6acfc93be6d3684574d18794e2
  Author: Cleber Rosa <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/acceptance/avocado_qemu/__init__.py

  Log Message:
  -----------
  Acceptance tests: introduce utility method for tags unique vals

Currently a test can describe the target architecture binary that it
should primarily be run with, be setting a single tag value.

The same approach is expected to be done with other QEMU aspects to be
tested, for instance, the machine type and accelerator, so let's
generalize the logic into a utility method.

Signed-off-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: ba21bde930f50360b135268e4e69a0ad561ddca2
      
https://github.com/qemu/qemu/commit/ba21bde930f50360b135268e4e69a0ad561ddca2
  Author: Cleber Rosa <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M docs/devel/testing.rst
    M tests/acceptance/avocado_qemu/__init__.py
    M tests/acceptance/boot_linux_console.py
    M tests/acceptance/cpu_queries.py
    M tests/acceptance/linux_initrd.py
    M tests/acceptance/linux_ssh_mips_malta.py
    M tests/acceptance/machine_m68k_nextcube.py
    M tests/acceptance/machine_sparc_leon3.py
    M tests/acceptance/ppc_prep_40p.py
    M tests/acceptance/x86_cpu_model_versions.py

  Log Message:
  -----------
  Acceptance tests: use avocado tags for machine type

The same way the arch tag is being used as a fallback for the arch
parameter, let's do the same for QEMU's machine and avoid some boiler
plate code.

This is now possible because, since Avocado 72.0, it's possible to use
tags with names that match the machine types on QEMU.

Signed-off-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 2d320ad18876ac263049ee7c5425052f2661f706
      
https://github.com/qemu/qemu/commit/2d320ad18876ac263049ee7c5425052f2661f706
  Author: Cleber Rosa <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M tests/Makefile.include

  Log Message:
  -----------
  Acceptance tests: use relative location for tests

An Avocado Test ID[1] is composed by a number of components, but it
starts with the Test Name, usually a file system location that was
given to the loader.

Because the source directory is being given as a prefix to the
"tests/acceptance" directory containing the acceptance tests, the test
names will needlessly include the directory the user is using to host
the QEMU sources (and/or build tree).

Let's remove the source dir (or a build dir) from the path given to
the test loader.  This should give more constant names, and when using
result servers and databases, it should give the same test names
across executions from different people or from different directories.

[1] - 
https://avocado-framework.readthedocs.io/en/69.0/ReferenceGuide.html#test-id

Signed-off-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Wainer dos Santos Moschetta <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 8b272e001868581736144dbedca41aa8bbe82019
      
https://github.com/qemu/qemu/commit/8b272e001868581736144dbedca41aa8bbe82019
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/__init__.py
    A python/qemu/accel.py
    M tests/vm/basevm.py

  Log Message:
  -----------
  python/qemu: Move kvm_available() to its own module

This creates the 'accel' Python module to be the home for
utilities that deal with accelerators. Also moved kvm_available()
from __init__.py to this new module.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 1650c3e3043ac7e8f16bdfb746430a106e49c18f
      
https://github.com/qemu/qemu/commit/1650c3e3043ac7e8f16bdfb746430a106e49c18f
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/accel.py

  Log Message:
  -----------
  python/qemu: accel: Add list_accel() method

Since commit cbe6d6365a48 the command `qemu -accel help` returns
the list of accelerators enabled in the QEMU binary. This adds
the list_accel() method which return that same list.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: 53a049d7d78e5ccf6d4c0d7b0edb18f8b999df94
      
https://github.com/qemu/qemu/commit/53a049d7d78e5ccf6d4c0d7b0edb18f8b999df94
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/accel.py

  Log Message:
  -----------
  python/qemu: accel: Strengthen kvm_available() checks

Currently kvm_available() checks for the presence of kvm module
and, if target and host arches don't mismatch. This patch adds
an 3rd checking: if QEMU binary was compiled with kvm
support.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: d3ca7bb599b777271316c17a5221c5ee28815833
      
https://github.com/qemu/qemu/commit/d3ca7bb599b777271316c17a5221c5ee28815833
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/accel.py

  Log Message:
  -----------
  python/qemu: accel: Add tcg_available() method

This adds a method to check if the tcg accelerator is enabled
in the QEMU binary.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: e42cb9678cee7ff7c28afe7917c1002d6d8fdc0d
      
https://github.com/qemu/qemu/commit/e42cb9678cee7ff7c28afe7917c1002d6d8fdc0d
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2019-12-16 (Mon, 16 Dec 2019)

  Changed paths:
    M python/qemu/__init__.py

  Log Message:
  -----------
  python/qemu: Remove unneeded imports in __init__

__init_.py import some sub-modules unnecessarily. So let's
clean it up.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Suggested-by: Cleber Rosa <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cleber Rosa <address@hidden>


  Commit: f6e7a97acbe913b8a027e695e9c8793739914c7c
      
https://github.com/qemu/qemu/commit/f6e7a97acbe913b8a027e695e9c8793739914c7c
  Author: Peter Maydell <address@hidden>
  Date:   2019-12-17 (Tue, 17 Dec 2019)

  Changed paths:
    M docs/devel/testing.rst
    M python/qemu/__init__.py
    A python/qemu/accel.py
    M python/qemu/machine.py
    M scripts/analyze-migration.py
    M tests/Makefile.include
    M tests/acceptance/avocado_qemu/__init__.py
    M tests/acceptance/boot_linux_console.py
    M tests/acceptance/cpu_queries.py
    M tests/acceptance/empty_cpu_model.py
    M tests/acceptance/linux_initrd.py
    M tests/acceptance/linux_ssh_mips_malta.py
    M tests/acceptance/machine_m68k_nextcube.py
    M tests/acceptance/machine_sparc_leon3.py
    M tests/acceptance/ppc_prep_40p.py
    M tests/acceptance/x86_cpu_model_versions.py
    M tests/vm/basevm.py

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/cleber/tags/python-next-pull-request' 
into staging

Python queue 2019-12-17

# gpg: Signature made Tue 17 Dec 2019 05:12:43 GMT
# gpg:                using RSA key 7ABB96EB8B46B94D5E0FE9BB657E8D33A5F209F3
# gpg: Good signature from "Cleber Rosa <address@hidden>" [marginal]
# gpg: WARNING: This key is not certified with sufficiently trusted signatures!
# gpg:          It is not certain that the signature belongs to the owner.
# Primary key fingerprint: 7ABB 96EB 8B46 B94D 5E0F  E9BB 657E 8D33 A5F2 09F3

* remotes/cleber/tags/python-next-pull-request:
  python/qemu: Remove unneeded imports in __init__
  python/qemu: accel: Add tcg_available() method
  python/qemu: accel: Strengthen kvm_available() checks
  python/qemu: accel: Add list_accel() method
  python/qemu: Move kvm_available() to its own module
  Acceptance tests: use relative location for tests
  Acceptance tests: use avocado tags for machine type
  Acceptance tests: introduce utility method for tags unique vals
  Acceptance test x86_cpu_model_versions: use default vm
  tests/acceptance: Makes linux_initrd and empty_cpu_model use QEMUMachine
  python/qemu: Add set_qmp_monitor() to QEMUMachine
  analyze-migration.py: replace numpy with python 3.2
  analyze-migration.py: fix find() type error
  Revert "Acceptance test: cancel test if m68k kernel packages goes missing"
  tests/boot_linux_console: Fetch assets from Debian snapshot archives

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


Compare: https://github.com/qemu/qemu/compare/98ac38cd5ad5...f6e7a97acbe9