qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 5f55d6: gitlab-ci.yml: Test the TCG interpret

From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 5f55d6: gitlab-ci.yml: Test the TCG interpreter in a CI pi...
Date: Fri, 03 May 2019 02:36:54 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 5f55d64b38cd20c97c31929f1566f38e2c5ae44b
      
https://github.com/qemu/qemu/commit/5f55d64b38cd20c97c31929f1566f38e2c5ae44b
  Author: Thomas Huth <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M .gitlab-ci.yml

  Log Message:
  -----------
  gitlab-ci.yml: Test the TCG interpreter in a CI pipeline

So far we do not have any test coverage for TCI (the TCG interpreter) yet.
Thus let's add a CI pipeline that runs at least some basic TCG tests with
a TCI build, to make sure that there are no further regressions.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>


  Commit: 3fe13fe16e2147ccbab037ace60e2bd3831094fd
      
https://github.com/qemu/qemu/commit/3fe13fe16e2147ccbab037ace60e2bd3831094fd
  Author: Eduardo Habkost <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M MAINTAINERS
    M accel/Makefile.objs
    A accel/qtest.c
    M qtest.c

  Log Message:
  -----------
  qtest: Move accel code to accel/qtest.c

QTest has two parts: the server (-qtest) and the accelerator
(-machine accel=qtest).  The accelerator depends on CONFIG_POSIX
due to its usage of sigwait(), but the server doesn't.

Move the accel code to accel/qtest.c.  Later we will disable
compilation of accel/qtest.c on non-POSIX systems.

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
[thuth: added fixup for MAINTAINERS file]
Signed-off-by: Thomas Huth <address@hidden>


  Commit: a08052bc248632f8c16ef0c5b93e0611543e89cc
      
https://github.com/qemu/qemu/commit/a08052bc248632f8c16ef0c5b93e0611543e89cc
  Author: Eduardo Habkost <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M accel/Makefile.objs
    M accel/qtest.c
    M include/sysemu/qtest.h

  Log Message:
  -----------
  qtest: Don't compile qtest accel on non-POSIX systems

qtest_available() will always return 0 on non-POSIX systems.
It's simpler to just not compile the accelerator code on those
systems instead of relying on the AccelClass::available function.

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden> [on mingw64]
Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 8d006d4bc2ab4f72877d8bd47cba9aa8d24b54d0
      
https://github.com/qemu/qemu/commit/8d006d4bc2ab4f72877d8bd47cba9aa8d24b54d0
  Author: Eduardo Habkost <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M accel/accel.c
    M include/sysemu/accel.h

  Log Message:
  -----------
  accel: Remove unused AccelClass::available field

The field is not used anymore, we can remove it.

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden> [on mingw64]
Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: e6e90feedb706b1b92827a5977b37e1e8defb8ef
      
https://github.com/qemu/qemu/commit/e6e90feedb706b1b92827a5977b37e1e8defb8ef
  Author: Thomas Huth <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Add -Wno-typedef-redefinition to CFLAGS (for Clang)

Without the -Wno-typedef-redefinition option, clang complains if a typedef
gets redefined in gnu99 mode (since this is officially a C11 feature). This
used to also happen with older versions of GCC, but since we've bumped our
minimum GCC version to 4.8, all versions of GCC that we support do not seem
to issue this warning in gnu99 mode anymore. So this has become a common
problem for people who only test their code with GCC - they do not notice
the issue until they submit their patches and suddenly patchew or a
maintainer complains.

Now that we do not urgently need to keep the code clean from typedef
redefintions anymore with recent versions of GCC, we can ease the
situation with clang, too, and simply shut these warnings off for good.

Reviewed-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 9c79024225af6b3ae04ea2dd94a5e5c4132a9e65
      
https://github.com/qemu/qemu/commit/9c79024225af6b3ae04ea2dd94a5e5c4132a9e65
  Author: Thomas Huth <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Remove old *-config-devices.mak.d files when running configure

When running "make" in a build directory from the pre-Kconfig merge time,
the build process currently fails with:

 make: *** No rule to make target `.../default-configs/pci.mak',
  needed by `aarch64-softmmu/config-devices.mak'.  Stop.

To make sure that this problem at least goes away when the user runs
"configure" (or "sh config.status") again, we have to make sure that
we re-generate the .mak.d files. Thus remove the old stale files
while running the configure script.

Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 25ed0ecc0946a50b747fde6c8bce3d0ec99bdeac
      
https://github.com/qemu/qemu/commit/25ed0ecc0946a50b747fde6c8bce3d0ec99bdeac
  Author: Helge Deller <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Relax check for libseccomp

All major distributions do support libseccomp version >= 2.3.0, so there
is no need to special-case on various architectures any longer.

Signed-off-by: Helge Deller <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Acked-by: Eduardo Otubo <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: aff39be0ed9753c9c323f64a14f5533dd5c43525
      
https://github.com/qemu/qemu/commit/aff39be0ed9753c9c323f64a14f5533dd5c43525
  Author: Thomas Huth <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M hw/pci-host/designware.c
    M hw/pci-host/gpex.c
    M hw/pci-host/q35.c
    M hw/pci-host/xilinx-pcie.c

  Log Message:
  -----------
  hw/pci-host: Use object_initialize_child for correct reference counting

Both functions, object_initialize() and object_property_add_child() increase
the reference counter of the new object, so one of the references has to be
dropped afterwards to get the reference counting right. Otherwise the child
object might not be properly cleaned up when the parent gets destroyed.
Some functions of the pci-host devices miss to drop one of the references.
Fix it by using object_initialize_child() instead, which takes care of
calling object_initialize(), object_property_add_child() and object_unref()
in the right order.

Suggested-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: f62d632f4328fab02682335ba1ccfdbd9893d33d
      
https://github.com/qemu/qemu/commit/f62d632f4328fab02682335ba1ccfdbd9893d33d
  Author: Peter Maydell <address@hidden>
  Date:   2019-05-02 (Thu, 02 May 2019)

  Changed paths:
    M .gitlab-ci.yml
    M MAINTAINERS
    M accel/Makefile.objs
    M accel/accel.c
    A accel/qtest.c
    M configure
    M hw/pci-host/designware.c
    M hw/pci-host/gpex.c
    M hw/pci-host/q35.c
    M hw/pci-host/xilinx-pcie.c
    M include/sysemu/accel.h
    M include/sysemu/qtest.h
    M qtest.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/huth-gitlab/tags/pull-request-2019-05-02' into staging

- Move qtest accel code to accel/qtest.c, get rid of AccelClass->available
- Test TCG interpreter in gitlab-ci
- Small improvements to the configure script
- Use object_initialize_child in hw/pci-host

# gpg: Signature made Thu 02 May 2019 17:07:34 BST
# gpg:                using RSA key 2ED9D774FE702DB5
# gpg: Good signature from "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [unknown]
# Primary key fingerprint: 27B8 8847 EEE0 2501 18F3  EAB9 2ED9 D774 FE70 2DB5

* remotes/huth-gitlab/tags/pull-request-2019-05-02:
  hw/pci-host: Use object_initialize_child for correct reference counting
  configure: Relax check for libseccomp
  configure: Remove old *-config-devices.mak.d files when running configure
  configure: Add -Wno-typedef-redefinition to CFLAGS (for Clang)
  accel: Remove unused AccelClass::available field
  qtest: Don't compile qtest accel on non-POSIX systems
  qtest: Move accel code to accel/qtest.c
  gitlab-ci.yml: Test the TCG interpreter in a CI pipeline

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


Compare: https://github.com/qemu/qemu/compare/8482ff2eb3bb...f62d632f4328
reply via email to
[Prev in Thread] Current Thread [Next in Thread]