qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] 58379a: qapi: virtio: Fix the introduced vers


From: Paolo Bonzini
Subject: [Qemu-commits] [qemu/qemu] 58379a: qapi: virtio: Fix the introduced version
Date: Mon, 07 Nov 2022 12:03:34 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 58379af7102616531bf8ca499121e33b034fe9e4
      
https://github.com/qemu/qemu/commit/58379af7102616531bf8ca499121e33b034fe9e4
  Author: Han Han <hhan@redhat.com>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    M qapi/virtio.json

  Log Message:
  -----------
  qapi: virtio: Fix the introduced version

The items of qapi/virtio.json are introduced at a5ebce38576. They will be
in the version 7.2 not 7.1.

Signed-off-by: Han Han <hhan@redhat.com>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20221101014647.3000801-1-hhan@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 1e458f11273c315ee2045f6e632b8dfb5f2b1544
      
https://github.com/qemu/qemu/commit/1e458f11273c315ee2045f6e632b8dfb5f2b1544
  Author: Stefan Weil <sw@weilnetz.de>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    M docs/devel/testing.rst
    M docs/system/arm/cpu-features.rst
    M docs/system/loongarch/loongson3.rst
    M docs/tools/virtiofsd.rst
    M include/exec/memory.h
    M qapi/qom.json
    M qemu-options.hx
    M qga/qapi-schema.json
    M tests/qtest/libqtest.h

  Log Message:
  -----------
  Fix some typos in documentation and comments

Most of them were found and fixed using codespell.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20221030105944.311940-1-sw@weilnetz.de>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: ba24456b93a205b728475d5f0880f3ec495e383a
      
https://github.com/qemu/qemu/commit/ba24456b93a205b728475d5f0880f3ec495e383a
  Author: Chuck Zmudzinski <brchuckz@aol.com>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    M hw/xen/meson.build

  Log Message:
  -----------
  xen/pt: fix syntax error that causes FTBFS in some configurations

When Qemu is built with --enable-xen and --disable-xen-pci-passthrough
and the target os is linux, the build fails with:

meson.build:3477:2: ERROR: File xen_pt_stub.c does not exist.

Fixes: 582ea95f5f93 ("meson: convert hw/xen")

Signed-off-by: Chuck Zmudzinski <brchuckz@aol.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: 
<5f1342a13c09af77b1a7b0aeaba5955bcea89731.1667242033.git.brchuckz@aol.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 0f208a9747a700a4b63afc87d58da0c87f37e214
      
https://github.com/qemu/qemu/commit/0f208a9747a700a4b63afc87d58da0c87f37e214
  Author: Philippe Mathieu-Daudé <philmd@linaro.org>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    R target/m68k/qregs.def
    A target/m68k/qregs.h.inc
    M target/m68k/translate.c

  Log Message:
  -----------
  target/m68k: Rename qregs.def -> qregs.h.inc

We use the .h.inc extension to include C headers. To be consistent
with the rest of the codebase, rename the C headers using the .def
extension.

IDE/tools using our .editorconfig / .gitattributes will leverage
this consistency.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221025235006.7215-2-philmd@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 9cef8d99266cca7189237287dcd18b0a91c9aae7
      
https://github.com/qemu/qemu/commit/9cef8d99266cca7189237287dcd18b0a91c9aae7
  Author: Philippe Mathieu-Daudé <philmd@linaro.org>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    R target/s390x/tcg/insn-data.def
    A target/s390x/tcg/insn-data.h.inc
    R target/s390x/tcg/insn-format.def
    A target/s390x/tcg/insn-format.h.inc
    M target/s390x/tcg/translate.c

  Log Message:
  -----------
  target/s390x: Rename insn-data/format.def -> insn-data/format.h.inc

We use the .h.inc extension to include C headers. To be consistent
with the rest of the codebase, rename the C headers using the .def
extension.

IDE/tools using our .editorconfig / .gitattributes will leverage
this consistency.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221025235006.7215-3-philmd@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 5d756c82c9eb288f9e19bbc711c99e6b6bda9f0c
      
https://github.com/qemu/qemu/commit/5d756c82c9eb288f9e19bbc711c99e6b6bda9f0c
  Author: Philippe Mathieu-Daudé <philmd@linaro.org>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    R target/tricore/csfr.def
    A target/tricore/csfr.h.inc
    M target/tricore/translate.c

  Log Message:
  -----------
  target/tricore: Rename csfr.def -> csfr.h.inc

We use the .h.inc extension to include C headers. To be consistent
with the rest of the codebase, rename the C headers using the .def
extension.

IDE/tools using our .editorconfig / .gitattributes will leverage
this consistency.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20221025235006.7215-4-philmd@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 5a820d5d07cb101752c731799a326648a5aa185a
      
https://github.com/qemu/qemu/commit/5a820d5d07cb101752c731799a326648a5aa185a
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-11-05 (Sat, 05 Nov 2022)

  Changed paths:
    M tests/unit/test-io-channel-command.c

  Log Message:
  -----------
  tests/unit: simpler variable sequence for test-io-channel

This avoids some compilers complaining about a potentially
un-initialised [src|dst]argv. In retrospect using GString was overkill
for what we are constructing.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20221103102329.2581508-1-alex.bennee@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>


  Commit: 4f76b3d9bb1a5b16166217b46a5f50b3a7c2f5a8
      
https://github.com/qemu/qemu/commit/4f76b3d9bb1a5b16166217b46a5f50b3a7c2f5a8
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M util/main-loop.c

  Log Message:
  -----------
  util/main-loop: Fix maximum number of wait objects for win32

The maximum number of wait objects for win32 should be
MAXIMUM_WAIT_OBJECTS, not MAXIMUM_WAIT_OBJECTS + 1.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Message-Id: <20221019102015.2441622-1-bmeng.cn@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: d393b0a176068c41cc08f41c245721ed9ca91d30
      
https://github.com/qemu/qemu/commit/d393b0a176068c41cc08f41c245721ed9ca91d30
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M include/qemu/main-loop.h
    M util/main-loop.c

  Log Message:
  -----------
  util/main-loop: Avoid adding the same HANDLE twice

Fix the logic in qemu_add_wait_object() to avoid adding the same
HANDLE twice, as the behavior is undefined when passing an array
that contains same HANDLEs to WaitForMultipleObjects() API.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Message-Id: <20221019102015.2441622-2-bmeng.cn@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: e0d034bb243cf0bac2416e00af810c4ca7821762
      
https://github.com/qemu/qemu/commit/e0d034bb243cf0bac2416e00af810c4ca7821762
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M util/aio-win32.c

  Log Message:
  -----------
  util/aio-win32: Correct the event array size in aio_poll()

WaitForMultipleObjects() can only wait for MAXIMUM_WAIT_OBJECTS
object handles. Correct the event array size in aio_poll() and
add a assert() to ensure it does not cause out of bound access.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20221019102015.2441622-3-bmeng.cn@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 75ac231c67cdb13f0609943fab5499963858b587
      
https://github.com/qemu/qemu/commit/75ac231c67cdb13f0609943fab5499963858b587
  Author: TaiseiIto <taisei1212@outlook.jp>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M gdb-xml/i386-32bit.xml

  Log Message:
  -----------
  gdb-xml: Fix size of EFER register on i386 architecture when debugged by GDB

Before this commit, there were contradictory descriptions about size of EFER
register.
Line 113 says the size is 8 bytes.
Line 129 says the size is 4 bytes.

As a result, when GDB is debugging an OS running on QEMU, the GDB cannot
read 'g' packets correctly. This 'g' packet transmits values of each
registers of machine emulated by QEMU to GDB. QEMU, the packet sender,
assign 4 bytes for EFER in 'g' packet based on the line 113.
GDB, the packet receiver, extract 8 bytes for EFER in 'g' packet based on
the line 129. Therefore, all registers located behind EFER in 'g' packet
has been shifted 4 bytes in GDB.

After this commit, GDB can read 'g' packets correctly.

Signed-off-by: TaiseiIto <taisei1212@outlook.jp>
Message-Id: 
<TY0PR0101MB4285F637209075C9F65FCDA6A4479@TY0PR0101MB4285.apcprd01.prod.exchangelabs.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 8a0afbb2a4f7da5e6a542787ea90eb240e7ac65d
      
https://github.com/qemu/qemu/commit/8a0afbb2a4f7da5e6a542787ea90eb240e7ac65d
  Author: Stefan Weil <sw@weilnetz.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M configure

  Log Message:
  -----------
  Fix broken configure with -Wunused-parameter

The configure script fails because it tries to compile small C programs
with a main function which is declared with arguments argc and argv
although those arguments are unused.

Running `configure -extra-cflags=-Wunused-parameter` triggers the problem.
configure for a native build does abort but shows the error in config.log.
A cross build configure for Windows with Debian stable aborts with an
error.

Avoiding unused arguments fixes this.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Message-Id: <20221102202258.456359-1-sw@weilnetz.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: b5d3dac17077c4c7d9a90b4c76123ea80ac037da
      
https://github.com/qemu/qemu/commit/b5d3dac17077c4c7d9a90b4c76123ea80ac037da
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M meson.build

  Log Message:
  -----------
  meson: avoid unused arguments of main() in compiler tests

meson.build has one test where "main" is declared unnecessarily
with argc and argv arguments, but does not use them.  Because
the test needs -Werror too, HAVE_BROKEN_SIZE_MAX is defined
incorrectly.

Fix the test and, for consistency, remove argc and argv whenever
they are not needed.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: efa3901e14a6428ef775332a6df958fb5b27af32
      
https://github.com/qemu/qemu/commit/efa3901e14a6428ef775332a6df958fb5b27af32
  Author: Stefan Weil <sw@weilnetz.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M scripts/feature_to_c.sh

  Log Message:
  -----------
  Add missing include statement for global xml_builtin

This fixes some compiler warnings with compiler flag
-Wmissing-variable-declarations (tested with clang):

    aarch64_be-linux-user-gdbstub-xml.c:564:19: warning: no previous extern 
declaration for non-static variable 'xml_builtin' 
[-Wmissing-variable-declarations]
    aarch64-linux-user-gdbstub-xml.c:564:19: warning: no previous extern 
declaration for non-static variable 'xml_builtin' 
[-Wmissing-variable-declarations]
    aarch64-softmmu-gdbstub-xml.c:1763:19: warning: no previous extern 
declaration for non-static variable 'xml_builtin' 
[-Wmissing-variable-declarations]

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 2106106d80489fb9b10cd3ccfaec811988e797cb
      
https://github.com/qemu/qemu/commit/2106106d80489fb9b10cd3ccfaec811988e797cb
  Author: Claudio Fontana <cfontana@suse.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M include/qemu/module.h
    M softmmu/qtest.c
    M util/module.c

  Log Message:
  -----------
  module: removed unused function argument "mayfail"

mayfail is always passed as false for every invocation throughout the program.
It controls whether to printf or not to printf an error on
g_module_open failure.

Remove this unused argument.

Signed-off-by: Claudio Fontana <cfontana@suse.de>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220929093035.4231-2-cfontana@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: dbc0e80553c067f56cb236d9de2cd0d50f3c6131
      
https://github.com/qemu/qemu/commit/dbc0e80553c067f56cb236d9de2cd0d50f3c6131
  Author: Claudio Fontana <cfontana@suse.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M audio/audio.c
    M block.c
    M block/dmg.c
    M hw/core/qdev.c
    M include/qemu/module.h
    M qom/object.c
    M softmmu/qtest.c
    M ui/console.c
    M util/module.c

  Log Message:
  -----------
  module: rename module_load_one to module_load

Signed-off-by: Claudio Fontana <cfontana@suse.de>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929093035.4231-3-cfontana@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: c551fb0b53db5d9a1f0116e8cce12f994605c9ea
      
https://github.com/qemu/qemu/commit/c551fb0b53db5d9a1f0116e8cce12f994605c9ea
  Author: Claudio Fontana <cfontana@suse.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M audio/audio.c
    M block.c
    M block/dmg.c
    M hw/core/qdev.c
    M include/qemu/module.h
    M qom/object.c
    M softmmu/qtest.c
    M ui/console.c
    M util/module.c

  Log Message:
  -----------
  module: add Error arguments to module_load and module_load_qom

improve error handling during module load, by changing:

bool module_load(const char *prefix, const char *lib_name);
void module_load_qom(const char *type);

to:

int module_load(const char *prefix, const char *name, Error **errp);
int module_load_qom(const char *type, Error **errp);

where the return value is:

 -1 on module load error, and errp is set with the error
  0 on module or one of its dependencies are not installed
  1 on module load success
  2 on module load success (module already loaded or built-in)

module_load_qom_one has been introduced in:

commit 28457744c345 ("module: qom module support"), which built on top of
module_load_one, but discarded the bool return value. Restore it.

Adapt all callers to emit errors, or ignore them, or fail hard,
as appropriate in each context.

Replace the previous emission of errors via fprintf in _some_ error
conditions with Error and error_report, so as to emit to the appropriate
target.

A memory leak is also fixed as part of the module_load changes.

audio: when attempting to load an audio module, report module load errors.
Note that still for some callers, a single issue may generate multiple
error reports, and this could be improved further.
Regarding the audio code itself, audio_add() seems to ignore errors,
and this should probably be improved.

block: when attempting to load a block module, report module load errors.
For the code paths that already use the Error API, take advantage of those
to report module load errors into the Error parameter.
For the other code paths, we currently emit the error, but this could be
improved further by adding Error parameters to all possible code paths.

console: when attempting to load a display module, report module load errors.

qdev: when creating a new qdev Device object (DeviceState), report load errors.
      If a module cannot be loaded to create that device, now abort execution
      (if no CONFIG_MODULE) or exit (if CONFIG_MODULE).

qom/object.c: when initializing a QOM object, or looking up class_by_name,
              report module load errors.

qtest: when processing the "module_load" qtest command, report errors
       in the load of the module.

Signed-off-by: Claudio Fontana <cfontana@suse.de>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929093035.4231-4-cfontana@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 971974f0a9745a5163e1c825d38da03118054ae2
      
https://github.com/qemu/qemu/commit/971974f0a9745a5163e1c825d38da03118054ae2
  Author: Kevin Wolf <kwolf@redhat.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M block/dmg.c

  Log Message:
  -----------
  dmg: warn when opening dmg images containing blocks of unknown type

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Claudio Fontana <cfontana@suse.de>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220929093035.4231-5-cfontana@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 5141e9a23fc9a890d66a5700920a5ffd8885121f
      
https://github.com/qemu/qemu/commit/5141e9a23fc9a890d66a5700920a5ffd8885121f
  Author: Claudio Fontana <cfontana@suse.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M accel/accel-softmmu.c

  Log Message:
  -----------
  accel: abort if we fail to load the accelerator plugin

if QEMU is configured with modules enabled, it is possible that the
load of an accelerator module will fail.
Exit in this case, relying on module_object_class_by_name to report
the specific load error if any.

Signed-off-by: Claudio Fontana <cfontana@suse.de>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

[claudio: changed abort() to exit(1)]
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20220929093035.4231-6-cfontana@suse.de>

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 897c0da96f936217e3e2a04c77486ca93c2f1dea
      
https://github.com/qemu/qemu/commit/897c0da96f936217e3e2a04c77486ca93c2f1dea
  Author: Akihiko Odaki <akihiko.odaki@daynix.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/libqos/e1000e.c

  Log Message:
  -----------
  tests/qtest/libqos/e1000e: Refer common PCI ID definitions

This is yet another minor cleanup to ease understanding and
future refactoring of the tests.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20221103015017.19947-1-akihiko.odaki@daynix.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: ff4f45811fb2ca8f17ef78361128b03dbb679185
      
https://github.com/qemu/qemu/commit/ff4f45811fb2ca8f17ef78361128b03dbb679185
  Author: Akihiko Odaki <akihiko.odaki@daynix.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/libqos/e1000e.c

  Log Message:
  -----------
  tests/qtest/libqos/e1000e: Set E1000_CTRL_SLU

The later device status check depends on E1000_STATUS_LU, which is
enabled by E1000_CTRL_SLU. Though E1000_STATUS_LU is not implemented
and E1000_STATUS_LU is always available in the current implementation,
be a bit nicer and set E1000_CTRL_SLU just in case the bit is
implemented in the future.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20221103025451.27446-1-akihiko.odaki@daynix.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: dfa644b231a55e50571b99bc65a2fff530e6913c
      
https://github.com/qemu/qemu/commit/dfa644b231a55e50571b99bc65a2fff530e6913c
  Author: Akihiko Odaki <akihiko.odaki@daynix.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/e1000e-test.c

  Log Message:
  -----------
  tests/qtest/e1000e-test: Use e1000_regs.h

The register definitions in tests/qtest/e1000e-test.c had names
different from hw/net/e1000_regs.h, which made it hard to understand
what test codes corresponds to the implementation. Use
hw/net/e1000_regs.h from tests/qtest/libqos/e1000e.c to remove
these duplications.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20221103095416.110162-1-akihiko.odaki@daynix.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 5ebafa16433e424127738d694c653101cc3133dd
      
https://github.com/qemu/qemu/commit/5ebafa16433e424127738d694c653101cc3133dd
  Author: Akihiko Odaki <akihiko.odaki@daynix.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/libqos/e1000e.c

  Log Message:
  -----------
  tests/qtest/libqos/e1000e: Use E1000_STATUS_ASDV_1000

Nemonics E1000_STATUS_LAN_INIT_DONE and E1000_STATUS_ASDV_1000 have
the same value, and E1000_STATUS_ASDV_1000 should be used here because
E1000_STATUS_ASDV_1000 represents the auto-detected speed tested here
while E1000_STATUS_LAN_INIT_DONE is a value used for a different purpose
with a variant of e1000e family different from the one implemented in
QEMU.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20221103083425.100590-1-akihiko.odaki@daynix.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 624ee20cb9742bb536a778b9585c916b243e78f2
      
https://github.com/qemu/qemu/commit/624ee20cb9742bb536a778b9585c916b243e78f2
  Author: Akihiko Odaki <akihiko.odaki@daynix.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/libqos/e1000e.c

  Log Message:
  -----------
  tests/qtest/libqos/e1000e: Use IVAR shift definitions

There were still some constants defined in e1000_regs.h.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20221105053010.38037-1-akihiko.odaki@daynix.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: d46e6bba55f858b829251e2f4bd7b150cdb5b1d6
      
https://github.com/qemu/qemu/commit/d46e6bba55f858b829251e2f4bd7b150cdb5b1d6
  Author: Stefan Weil <sw@weilnetz.de>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/migration-test.c

  Log Message:
  -----------
  tests/qtest: Fix two format strings

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Message-Id: <20221105115525.623059-1-sw@weilnetz.de>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: d1695f1839769b62e25086187afc58185f49ba2f
      
https://github.com/qemu/qemu/commit/d1695f1839769b62e25086187afc58185f49ba2f
  Author: Bin Meng <bin.meng@windriver.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M tests/qtest/migration-test.c

  Log Message:
  -----------
  tests/qtest: migration-test: Enable TLS PSK tests for win32

Since commit f1018ea0a30f ("tests: avoid DOS line endings in PSK file"),
the bug of the helper test_tls_psk_init_common() that caused TLS PSK
tests to fail on Windows was fixed. Let's enable these tests on win32.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
Message-Id: <20221101035021.729669-1-bin.meng@windriver.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 765de32d87db65abf443d6a5d7097d42fc6e48de
      
https://github.com/qemu/qemu/commit/765de32d87db65abf443d6a5d7097d42fc6e48de
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M .gitlab-ci.d/buildtest.yml

  Log Message:
  -----------
  gitlab-ci: increase clang-user timeout

The clang-user test exceeds the 1 hour timeout occassionally.

Philippe Mathieu-Daudé has pointed out that the number of tcg tests has
increased since QEMU 7.1. The execution time therefore probably reflects
a legitimate increase in tests rather than a performance regression.

Bump the timeout to prevent CI failures.

Suggested-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221104113659.427690-1-stefanha@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: f53b033e4cd2e7706df3cca04f3bf3c5ffc6b08c
      
https://github.com/qemu/qemu/commit/f53b033e4cd2e7706df3cca04f3bf3c5ffc6b08c
  Author: Peter Jin <pjin@linux.ibm.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M hw/s390x/css.c

  Log Message:
  -----------
  s390x/css: revert SCSW ctrl/flag bits on error

Revert the control and flag bits in the subchannel status word in case
the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH).
According to POPS, the control and flag bits are only changed if SSCH,
CSCH, and HSCH return CC 0, and no other action should be taken otherwise.
In order to simulate that after the fact, the bits need to be reverted on
non-zero CC.

While the do_subchannel_work logic for virtual (virtio) devices will
return condition code 0, passthrough (vfio) devices may encounter
errors from either the host kernel or real hardware that need to be
accounted for after this point. This includes restoring the state of
the Subchannel Status Word to reflect the subchannel, as these bits
would not be set in the event of a non-zero condition code from the
affected instructions.

Experimentation has shown that a failure on a START SUBCHANNEL (SSCH)
to a passthrough device would leave the subchannel with the START
PENDING activity control bit set, thus blocking subsequent SSCH
operations in css_do_ssch() until some form of error recovery was
undertaken since no interrupt would be expected.

Signed-off-by: Peter Jin <pjin@linux.ibm.com>
Message-Id: <20221027212341.2904795-1-pjin@linux.ibm.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Reviewed-by: Matthew Rosato <mjrosato@linux.ibm.com>
[thuth: Updated the commit description to Eric's suggestion]
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 4a8d21ba50fc8625c3bd51dab903872952f95718
      
https://github.com/qemu/qemu/commit/4a8d21ba50fc8625c3bd51dab903872952f95718
  Author: Matthew Rosato <mjrosato@linux.ibm.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M hw/s390x/s390-pci-inst.c

  Log Message:
  -----------
  s390x/pci: RPCIT second pass when mappings exhausted

If we encounter a new mapping while the number of available DMA entries
in vfio is 0, we are currently skipping that mapping which is a problem
if we manage to free up DMA space after that within the same RPCIT --
we will return to the guest with CC0 and have not mapped everything
within the specified range.  This issue was uncovered while testing
changes to the s390 linux kernel iommu/dma code, where a different
usage pattern was employed (new mappings start at the end of the
aperture and work back towards the front, making us far more likely
to encounter new mappings before invalidated mappings during a
global refresh).

Fix this by tracking whether any mappings were skipped due to vfio
DMA limit hitting 0; when this occurs, we still continue the range
and unmap/map anything we can - then we must re-run the range again
to pickup anything that was missed.  This must occur in a loop until
all requests are satisfied (success) or we detect that we are still
unable to complete all mappings (return ZPCI_RPCIT_ST_INSUFF_RES).

Link: 
https://lore.kernel.org/linux-s390/20221019144435.369902-1-schnelle@linux.ibm.com/
Fixes: 37fa32de70 ("s390x/pci: Honor DMA limits set by vfio")
Reported-by: Niklas Schnelle <schnelle@linux.ibm.com>
Signed-off-by: Matthew Rosato <mjrosato@linux.ibm.com>
Message-Id: <20221028194758.204007-2-mjrosato@linux.ibm.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 1fd396e32288bbf536483c74b68cb3ee86005a9f
      
https://github.com/qemu/qemu/commit/1fd396e32288bbf536483c74b68cb3ee86005a9f
  Author: Pierre Morel <pmorel@linux.ibm.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M hw/s390x/s390-virtio-ccw.c

  Log Message:
  -----------
  s390x: Register TYPE_S390_CCW_MACHINE properties as class properties

Currently, when running 'qemu-system-s390x -M s390-ccw-virtio,help'
the s390x-specific properties are not listed anymore. This happens
because since commit d8fb7d0969 ("vl: switch -M parsing to keyval")
the properties have to be defined at the class level and not at the
instance level anymore. Fix it on s390x now, too, by moving the
registration of the properties to the class level"

Fixes: d8fb7d0969 ("vl: switch -M parsing to keyval")
Signed-off-by: Pierre Morel <pmorel@linux.ibm.com>
Message-Id: <20221103170150.20789-2-pmorel@linux.ibm.com>
[thuth: Add patch description]
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 6393b29966fce3c0e47746a9646ae439e7fd0728
      
https://github.com/qemu/qemu/commit/6393b29966fce3c0e47746a9646ae439e7fd0728
  Author: Pierre Morel <pmorel@linux.ibm.com>
  Date:   2022-11-06 (Sun, 06 Nov 2022)

  Changed paths:
    M hw/s390x/s390-virtio-ccw.c
    M include/hw/s390x/s390-virtio-ccw.h

  Log Message:
  -----------
  s390x/cpu topology: add max_threads machine class attribute

The S390 CPU topology accepts the smp.threads argument while
in reality it does not effectively allow multthreading.

Let's keep this behavior for machines older than 7.2 and
refuse to use threads in newer machines until multithreading
is really exposed to the guest by the machine.

Signed-off-by: Pierre Morel <pmorel@linux.ibm.com>
Message-Id: <20221103170150.20789-3-pmorel@linux.ibm.com>
[thuth: Small fixes to the commit description]
Signed-off-by: Thomas Huth <thuth@redhat.com>


  Commit: 64be791d3c251a09eb2dbbf556478f976211b30c
      
https://github.com/qemu/qemu/commit/64be791d3c251a09eb2dbbf556478f976211b30c
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-07 (Mon, 07 Nov 2022)

  Changed paths:
    M docs/devel/testing.rst
    M docs/system/arm/cpu-features.rst
    M docs/system/loongarch/loongson3.rst
    M docs/tools/virtiofsd.rst
    M hw/xen/meson.build
    M include/exec/memory.h
    M qapi/qom.json
    M qapi/virtio.json
    M qemu-options.hx
    M qga/qapi-schema.json
    R target/m68k/qregs.def
    A target/m68k/qregs.h.inc
    M target/m68k/translate.c
    R target/s390x/tcg/insn-data.def
    A target/s390x/tcg/insn-data.h.inc
    R target/s390x/tcg/insn-format.def
    A target/s390x/tcg/insn-format.h.inc
    M target/s390x/tcg/translate.c
    R target/tricore/csfr.def
    A target/tricore/csfr.h.inc
    M target/tricore/translate.c
    M tests/qtest/libqtest.h
    M tests/unit/test-io-channel-command.c

  Log Message:
  -----------
  Merge tag 'trivial-branch-for-7.2-pull-request' of 
https://gitlab.com/laurent_vivier/qemu into staging

Pull request trivial branch 20221103-v2

# -----BEGIN PGP SIGNATURE-----
#
# iQJGBAABCAAwFiEEzS913cjjpNwuT1Fz8ww4vT8vvjwFAmNmvt8SHGxhdXJlbnRA
# dml2aWVyLmV1AAoJEPMMOL0/L748HOMP/35HpUyXmjtpHegSgIQ43KLFNG+lBbj8
# 83W6HmOnd1ogSxiTmAx15a0jbcdfOPlloelG91GqptJsDBuwVBJ3wt2rDN4AseZe
# GTt4pXJPBITp+UGlz6BkGB1WrDTAB8xieijx6sXhyAfLYt1EwNx6qkWnTQh0sQjT
# jqjCTpF6V2odaxHJ7zmhhtlOea84q017ookhguDpOHbzzVEG5ma9YJwW2B5SPMUy
# IWrBEgV48S9epYX1o7HWqRbWN0gP0tuPaTIf+QSEwopqFkaArfBgwgGbPmU2njg6
# NY8TnEwmgEbcK8nxWUSpSWlFJ2YPhKTV1BqpcNSmZkVtG0bLxvLaMJu8TC+p4B90
# vepn7rysQqENCbcSaWUuVDtlhvi8LtRyaGtmeEzu6CrvMhsjzEIsRriVbFmHhDVh
# cmoePVPu4Ypn2QSoT2a8ZXT2KzgB92/9jKCoHpC8K5eR69NH1QAb+YrlmTjum+DE
# XZ5OhuI/nSY6xTR1nU4oRjq0MRHoS4OFR8c82wbA/XhMFju7Ej2Fj2tCeOdKK/IH
# /jFXwkx3GJbwuq1RetCmJdYStJ3K7F9gNDkgCYYSZoW6FsVUheWRakH8Ujyhk7Xb
# xnuOxrEGOw9A2cDjmRiDHqd3ErRJskzFGPtd2XIDr1loOwyWbYKRLd8A6W7HcQiU
# y3W6uHedI1ZL
# =PYU9
# -----END PGP SIGNATURE-----
# gpg: Signature made Sat 05 Nov 2022 15:51:59 EDT
# gpg:                using RSA key CD2F75DDC8E3A4DC2E4F5173F30C38BD3F2FBE3C
# gpg:                issuer "laurent@vivier.eu"
# gpg: Good signature from "Laurent Vivier <lvivier@redhat.com>" [full]
# gpg:                 aka "Laurent Vivier <laurent@vivier.eu>" [full]
# gpg:                 aka "Laurent Vivier (Red Hat) <lvivier@redhat.com>" 
[full]
# Primary key fingerprint: CD2F 75DD C8E3 A4DC 2E4F  5173 F30C 38BD 3F2F BE3C

* tag 'trivial-branch-for-7.2-pull-request' of 
https://gitlab.com/laurent_vivier/qemu:
  tests/unit: simpler variable sequence for test-io-channel
  target/tricore: Rename csfr.def -> csfr.h.inc
  target/s390x: Rename insn-data/format.def -> insn-data/format.h.inc
  target/m68k: Rename qregs.def -> qregs.h.inc
  xen/pt: fix syntax error that causes FTBFS in some configurations
  Fix some typos in documentation and comments
  qapi: virtio: Fix the introduced version

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: 20a885a87d13fe3456b21716fdeca29519bdd9b8
      
https://github.com/qemu/qemu/commit/20a885a87d13fe3456b21716fdeca29519bdd9b8
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-07 (Mon, 07 Nov 2022)

  Changed paths:
    M accel/accel-softmmu.c
    M audio/audio.c
    M block.c
    M block/dmg.c
    M configure
    M gdb-xml/i386-32bit.xml
    M hw/core/qdev.c
    M include/qemu/main-loop.h
    M include/qemu/module.h
    M meson.build
    M qom/object.c
    M scripts/feature_to_c.sh
    M softmmu/qtest.c
    M ui/console.c
    M util/aio-win32.c
    M util/main-loop.c
    M util/module.c

  Log Message:
  -----------
  Merge tag 'for-upstream' of https://gitlab.com/bonzini/qemu into staging

* bug fixes for Win32 event loop
* bug fixes for -Wextra
* fix gdb XML for 32-bit x86
* improve error handling for module load

# -----BEGIN PGP SIGNATURE-----
#
# iQFIBAABCAAyFiEE8TM4V0tmI4mGbHaCv/vSX3jHroMFAmNndPcUHHBib256aW5p
# QHJlZGhhdC5jb20ACgkQv/vSX3jHroOVVwf+Nfvx9aoDTo6vU4kHmCbh7+BTtcUA
# qEkaccL4pfujwNfRl5gHHKtRhgT6py9OjfIX8pAck3jtc5r+5/niFn7CLcOP2G/C
# xuyVKPx3ONCMKCLjWwg63I8/t4JZDEnYEzyddAfV7Xb5600aUJlrxScW751K5eYQ
# CzHFAASdc00eSiWE2eFL4rV7dcj0NadHnJxfjUcguHX4Qlr1TmK/ihEGv6MMRo5S
# 9Ak/vLmRs+LsQJm2cXUu5QVQiDwp31xbv3AP9knB+irYxrsp4LSDxV8eKZwLehpW
# I4A3gz2xKAprfzVJKZI3Dc/hJSRpoKoCjz+QoI1NeVbWjsnYZAwD+VMSew==
# =Nroy
# -----END PGP SIGNATURE-----
# gpg: Signature made Sun 06 Nov 2022 03:48:55 EST
# gpg:                using RSA key F13338574B662389866C7682BFFBD25F78C7AE83
# gpg:                issuer "pbonzini@redhat.com"
# gpg: Good signature from "Paolo Bonzini <bonzini@gnu.org>" [full]
# gpg:                 aka "Paolo Bonzini <pbonzini@redhat.com>" [full]
# Primary key fingerprint: 46F5 9FBD 57D6 12E7 BFD4  E2F7 7E15 100C CD36 69B1
#      Subkey fingerprint: F133 3857 4B66 2389 866C  7682 BFFB D25F 78C7 AE83

* tag 'for-upstream' of https://gitlab.com/bonzini/qemu:
  accel: abort if we fail to load the accelerator plugin
  dmg: warn when opening dmg images containing blocks of unknown type
  module: add Error arguments to module_load and module_load_qom
  module: rename module_load_one to module_load
  module: removed unused function argument "mayfail"
  Add missing include statement for global xml_builtin
  meson: avoid unused arguments of main() in compiler tests
  Fix broken configure with -Wunused-parameter
  gdb-xml: Fix size of EFER register on i386 architecture when debugged by GDB
  util/aio-win32: Correct the event array size in aio_poll()
  util/main-loop: Avoid adding the same HANDLE twice
  util/main-loop: Fix maximum number of wait objects for win32

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


  Commit: cd706454c6cd239a477cb227caf3e3dfbb742d1a
      
https://github.com/qemu/qemu/commit/cd706454c6cd239a477cb227caf3e3dfbb742d1a
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-11-07 (Mon, 07 Nov 2022)

  Changed paths:
    M .gitlab-ci.d/buildtest.yml
    M hw/s390x/css.c
    M hw/s390x/s390-pci-inst.c
    M hw/s390x/s390-virtio-ccw.c
    M include/hw/s390x/s390-virtio-ccw.h
    M tests/qtest/e1000e-test.c
    M tests/qtest/libqos/e1000e.c
    M tests/qtest/migration-test.c

  Log Message:
  -----------
  Merge tag 'pull-request-2022-11-06' of https://gitlab.com/thuth/qemu into 
staging

* e1000e qtest improvements
* Allow TLS PSK tests on win32
* Increase the timeout of the clang-user CI job
* Some s390x fixes for QEMU 7.2

# -----BEGIN PGP SIGNATURE-----
#
# iQJEBAABCAAvFiEEJ7iIR+7gJQEY8+q5LtnXdP5wLbUFAmNn0vMRHHRodXRoQHJl
# ZGhhdC5jb20ACgkQLtnXdP5wLbXCoQ/1HXhfnPv/sIjdfJqC0M5/DjpOz2VM/fTb
# g/GGf3EiAlbnK09rjvZA939PAar+MQzBC5YJkn77jX6KFAyeQrQR3xCHIsjgeR+S
# c9v5Ix+OD4WAZHMwQ4XFtK5gxIqYHXY9kkBXqJspPC4BbQRRF4wfLXS5z7Mny7pH
# goz2F2FBCaT+hMK7Z8ibh2trdGEAevCoG6rRRUCkgrURy7mGy4fMiSyXHMOR8OD2
# quxUj+1aq57jdqPGix45pqzRNot6SRkqRuJXruSHJ3nV1ZV1192LiHPA1tUZIPpt
# pKNt2HJwcWJY0letAA9zB/xwwX8GppGrnJq65RKprSZXWLFHevM/jXx44KxPCWAz
# X7ASSpyX2ZW6H6MNyWpt+Bs66x0x6j6XYL0nPU4rmVeGteimnKyzYTcWlOcVA3Zp
# LDov8wIBlxTaQPK9RfrgvdiAR9RnH9OHUiSZVIhl9lJycQr2URLRM6pVXGDhXL/O
# YMe6gxoui8es4blXuMeEJfo1PWrZGsvY+sb0Ixz2+AFO/CT8HQYWVkK6lH2j9ymK
# NjDKmfFmNfzv/JA25CqDhY3/eUjLn3Nej2up9tzJBtddHjaqvaN1EzBpLipX42M1
# +QnfzP6WTimMgP+QznT0Y1OE1irJyAi/jNW7lUWcrtMHqSQSgexuLjljVBgyVZls
# 2AyGTz/g+Q==
# =w9Q7
# -----END PGP SIGNATURE-----
# gpg: Signature made Sun 06 Nov 2022 10:29:55 EST
# gpg:                using RSA key 27B88847EEE0250118F3EAB92ED9D774FE702DB5
# gpg:                issuer "thuth@redhat.com"
# gpg: Good signature from "Thomas Huth <th.huth@gmx.de>" [full]
# gpg:                 aka "Thomas Huth <thuth@redhat.com>" [full]
# gpg:                 aka "Thomas Huth <huth@tuxfamily.org>" [full]
# gpg:                 aka "Thomas Huth <th.huth@posteo.de>" [unknown]
# Primary key fingerprint: 27B8 8847 EEE0 2501 18F3  EAB9 2ED9 D774 FE70 2DB5

* tag 'pull-request-2022-11-06' of https://gitlab.com/thuth/qemu:
  s390x/cpu topology: add max_threads machine class attribute
  s390x: Register TYPE_S390_CCW_MACHINE properties as class properties
  s390x/pci: RPCIT second pass when mappings exhausted
  s390x/css: revert SCSW ctrl/flag bits on error
  gitlab-ci: increase clang-user timeout
  tests/qtest: migration-test: Enable TLS PSK tests for win32
  tests/qtest: Fix two format strings
  tests/qtest/libqos/e1000e: Use IVAR shift definitions
  tests/qtest/libqos/e1000e: Use E1000_STATUS_ASDV_1000
  tests/qtest/e1000e-test: Use e1000_regs.h
  tests/qtest/libqos/e1000e: Set E1000_CTRL_SLU
  tests/qtest/libqos/e1000e: Refer common PCI ID definitions

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>


Compare: https://github.com/qemu/qemu/compare/466e81ff1201...cd706454c6cd



reply via email to

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