[Qemu-commits] [qemu/qemu] 2b8419: tests: avoid running duplicate qom-tests

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 2b8419cb4911731db6c883fa7b0428ad4a355d9d
      
https://github.com/qemu/qemu/commit/2b8419cb4911731db6c883fa7b0428ad4a355d9d
  Author: Michael Roth <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M tests/Makefile

  Log Message:
  -----------
  tests: avoid running duplicate qom-tests

Since 3687d532 we've been unconditionally adding qom-test to our qtests
for every arch. However, some archs inherit their tests from Makefile
variables for other archs, such as i386/x86_64,
microblaze/microblazeel, and xtensa/xtensaeb. Since these are evaluated
in a lazy manner, we ultimately end up adding qom-test twice.

In the case x86_64, where we have a large number of machine types that
we rerun qom-test for, this has lead to a fairly noticeable increase
in the overall run-time of `make check` (78s vs. 42s on my machine).
Similar speed-ups are visible for other such archs, but not nearly as
significant.

Fix this by only adding qom-test to an arch's test list if it's not
already present.

Signed-off-by: Michael Roth <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Cc: address@hidden
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 10905cb26a10848921ba5a2b77e4efddad7c0ec1
      
https://github.com/qemu/qemu/commit/10905cb26a10848921ba5a2b77e4efddad7c0ec1
  Author: Alex Bennée <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  .travis.yml: add more linux-user to the build matrix

At the same time I've grouped the $ARCH-linux-user and $ARCH-softmmu
builds together (hoping FS cache helps) and grouped all $ARCH-softmmu
only builds into one target. This reduces the build matrix slightly
which will hopefully help with build times.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Alexander Graf <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: eebf29401abebbe8dba3e8fa8764c05fe4e6c1f1
      
https://github.com/qemu/qemu/commit/eebf29401abebbe8dba3e8fa8764c05fe4e6c1f1
  Author: Alex Bennée <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  .travis.yml: make the make slightly more parallel

The Travis VMs have 1.5 cores so we might as well make some use of the
paralellism.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Alexander Graf <address@hidden>
Reviewed-by: Brian Jackson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: cb021cfee772c916bf73f43010c6c299c3d91800
      
https://github.com/qemu/qemu/commit/cb021cfee772c916bf73f43010c6c299c3d91800
  Author: Alex Bennée <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  .travis.yml: pre-seed sub-modules for speed

A significant portion of the build time is spent initialising all the
sub-modules we use in the source tree. Often this is almost as long as
the build itself. By pre-seeding the .git/modules tree this will
hopefully improve things.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Alexander Graf <address@hidden>
Reviewed-by: Brian Jackson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: ed173cb704f01a62143a3ef0dcf8b493bc795c23
      
https://github.com/qemu/qemu/commit/ed173cb704f01a62143a3ef0dcf8b493bc795c23
  Author: Alex Bennée <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  .travis.yml: remove "make check" from main matrix

There are problems with unreliability in "make check" which still need
to be tracked down. As the tests are broadly the same for all targets if
added one explicit target to the matrix to run it. However this does
build all softmmu targets to ensure they at least "run"

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Alexander Graf <address@hidden>
Reviewed-by: Brian Jackson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: c5d1e2cce3a994484a809b599e6b38325c4e4c0e
      
https://github.com/qemu/qemu/commit/c5d1e2cce3a994484a809b599e6b38325c4e4c0e
  Author: Adrian-Ken Rueegsegger <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M scripts/kvm/vmxcap

  Log Message:
  -----------
  vmxcap: Update according to SDM of September 2014

This adds reporting of RDSEED exiting and XSAVES/XRSTORS #UD and fixes
the range of VMCS revision as well as some typos.

Signed-off-by: Adrian-Ken Rueegsegger <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 04404c21cc973e37ca33c4a8443eaf7578cc2246
      
https://github.com/qemu/qemu/commit/04404c21cc973e37ca33c4a8443eaf7578cc2246
  Author: Eric Blake <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M tests/Makefile

  Log Message:
  -----------
  qapi: Consistent whitespace in tests/Makefile

tests/Makefile had a mix of TAB vs. 8-space indentation; given
that it is a Makefile, TAB is more idiomatic even though in these
particular cases the choice of whitespace didn't matter.

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 597db727cc9dfeace19faed49014dbcda8a1a8fd
      
https://github.com/qemu/qemu/commit/597db727cc9dfeace19faed49014dbcda8a1a8fd
  Author: Eric Blake <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M tests/.gitignore

  Log Message:
  -----------
  qapi: Ignore files created during make check

After an in-tree build and run of 'make check-{qapi-schema,unit}',
I noticed some leftover files.

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Wenchao Xia <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 59a2c4ce2be04e00c6831e8a4410e5977813c604
      
https://github.com/qemu/qemu/commit/59a2c4ce2be04e00c6831e8a4410e5977813c604
  Author: Eric Blake <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M docs/qapi-code-gen.txt

  Log Message:
  -----------
  qapi: Update docs given recent event, spacing fixes

Commit 21cd70d added event support but didn't document what the
generated code looks like.  Commit 05dfb26 removed some unwanted
spaces in the generated code, but didn't reflect those changes
into the documentation.  Finally, the docs start with a big
disclaimer about QMP not using QAPI yet, which feels rather stale.

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 63ce8e150c2c3178c56b3969ec16ec702ced7abd
      
https://github.com/qemu/qemu/commit/63ce8e150c2c3178c56b3969ec16ec702ced7abd
  Author: Gonglei <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M os-posix.c

  Log Message:
  -----------
  os-posix: change tab to space avoid violating coding style

Cc: Eric Blake <address@hidden>
Signed-off-by: Gonglei <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 97699eff3afc9c0dda865882415f81584b926727
      
https://github.com/qemu/qemu/commit/97699eff3afc9c0dda865882415f81584b926727
  Author: Gonglei <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M os-posix.c

  Log Message:
  -----------
  os-posix: remove confused errno

If we get inside the 'else if (status == 1)' conditional,
then we know that read() succeeded, and therefore errno is
unspecified. Printing strerror(errno) on a random value
is not helpful.

Cc: Eric Blake <address@hidden>
Signed-off-by: Gonglei <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e5048d15ce6addae869f23514b2a1f0d4466418a
      
https://github.com/qemu/qemu/commit/e5048d15ce6addae869f23514b2a1f0d4466418a
  Author: Gonglei <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M os-posix.c

  Log Message:
  -----------
  os-posix: report error message when lock file failed

It will cause that create vm failed When manager
tool is killed forcibly (kill -9 libvirtd_pid),
the file not was unlink, and unlock. It's better
that report the error message for users.

Signed-off-by: Huangweidong <address@hidden>
Signed-off-by: Gonglei <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e80084d352be64913f28d7b706fec644b85face4
      
https://github.com/qemu/qemu/commit/e80084d352be64913f28d7b706fec644b85face4
  Author: Peter Maydell <address@hidden>
  Date:   2014-09-26 (Fri, 26 Sep 2014)

  Changed paths:
    M .travis.yml
    M docs/qapi-code-gen.txt
    M os-posix.c
    M scripts/kvm/vmxcap
    M tests/.gitignore
    M tests/Makefile

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/mjt/tags/trivial-patches-2014-09-26' 
into staging

trivial patches for 2014-09-26

# gpg: Signature made Fri 26 Sep 2014 18:33:53 BST using RSA key ID A4C3D7DB
# gpg: Good signature from "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 8044 65C5
#      Subkey fingerprint: 6F67 E18E 7C91 C5B1 5514  66A7 BEE5 9D74 A4C3 D7DB

* remotes/mjt/tags/trivial-patches-2014-09-26:
  os-posix: report error message when lock file failed
  os-posix: remove confused errno
  os-posix: change tab to space avoid violating coding style
  qapi: Update docs given recent event, spacing fixes
  qapi: Ignore files created during make check
  qapi: Consistent whitespace in tests/Makefile
  vmxcap: Update according to SDM of September 2014
  .travis.yml: remove "make check" from main matrix
  .travis.yml: pre-seed sub-modules for speed
  .travis.yml: make the make slightly more parallel
  .travis.yml: add more linux-user to the build matrix
  tests: avoid running duplicate qom-tests

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


Compare: https://github.com/qemu/qemu/compare/81ab11a7a524...e80084d352be