Re: [PATCH v9 05/16] qapi: net: add stream and dgram netdevs

2022-09-28 Thread David Gibson
On Mon, Sep 26, 2022 at 09:50:37PM +0200, Laurent Vivier wrote: > Copied from socket netdev file and modified to use SocketAddress > to be able to introduce new features like unix socket. > > "udp" and "mcast" are squashed into dgram netdev, multicast is detected > according to the IP address type

Re: Should we maybe move Cirrus-CI jobs away from Gitlab again?

2022-09-28 Thread Thomas Huth
On 27/09/2022 21.10, Stefan Hajnoczi wrote: On Tue, 27 Sept 2022 at 15:04, Thomas Huth wrote: On 27/09/2022 20.47, Stefan Hajnoczi wrote: On Tue, 27 Sept 2022 at 14:40, Thomas Huth wrote: On 27/09/2022 19.57, Daniel P. Berrangé wrote: On Tue, Sep 27, 2022 at 01:36:20PM -0400, Stefan Hajno

[PULL 00/37] qtests fixes

2022-09-28 Thread Thomas Huth
The following changes since commit dbc4f48b5ab3e6d85f78aa4df6bd6ad561c3d152: Merge tag 'net-pull-request' of https://github.com/jasowang/qemu into staging (2022-09-27 11:08:36 -0400) are available in the Git repository at: https://gitlab.com/thuth/qemu.git tags/pull-request-2022-09-28 for

Re: [PATCH v2 35/39] tests/qtest: migration-test: Skip running some TLS cases for win32

2022-09-28 Thread Daniel P . Berrangé
On Wed, Sep 28, 2022 at 02:03:28PM +0800, Bin Meng wrote: > Hi Daniel, > > On Tue, Sep 27, 2022 at 11:40 PM Daniel P. Berrangé > wrote: > > > > On Thu, Sep 22, 2022 at 07:54:05PM +0800, Bin Meng wrote: > > > On Thu, Sep 22, 2022 at 6:39 PM Daniel P. Berrangé > > > wrote: > > > > > > > > On Thu

[PULL 06/37] tests/qtest: fdc-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-10-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 01/37] tests/qtest: i440fx-test: Rewrite create_blob_file() to be portable

2022-09-28 Thread Thomas Huth
From: Bin Meng Previously request_{bios, pflash} cases were skipped on win32, mainly due to create_blob_file() calling mmap() which does not exist on win32. This rewirtes create_blob_file() to be portable, so that we can enable these cases on Windows. Suggested-by: Marc-André Lureau Signed-off-

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Ani Sinha
On Wed, Sep 28, 2022 at 12:28 PM Daniel P. Berrangé wrote: > > On Tue, Sep 27, 2022 at 05:18:10PM -0400, Michael S. Tsirkin wrote: > > On Tue, Sep 27, 2022 at 09:33:27AM +0100, Daniel P. Berrangé wrote: > > > On Tue, Sep 27, 2022 at 01:43:15PM +0530, Ani Sinha wrote: > > > > On Sun, Sep 18, 2022 a

[PULL 19/37] tests/qtest: qmp-test: Skip running test_qmp_oob for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng The test_qmp_oob test case calls mkfifo() which does not exist on win32. Exclude it. Signed-off-by: Bin Meng Reviewed-by: Thomas Huth Message-Id: <20220925113032.1949844-31-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- tests/qtest/qmp-test.c | 7 +++ 1 file changed, 7

[PULL 04/37] tests/qtest: boot-serial-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Thomas Huth Message-Id: <20220925113032.1949844-8-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- t

[PULL 15/37] tests/unit: test-qga: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_get_tmp_dir() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-24-bmeng...@gmail.com> Signed-off-by: Thomas Huth -

[PULL 29/37] tests/qtest: migration-test: Skip running some TLS cases for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng Some migration test cases use TLS to communicate, but they fail on Windows with the following error messages: qemu-system-x86_64: TLS handshake failed: Insufficient credentials for that request. qemu-system-x86_64: TLS handshake failed: Error in the pull function. query-mig

[PULL 07/37] tests/qtest: generic_fuzz: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_dir_make_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-11-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 14/37] tests/unit: test-image-locking: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-23-bmeng...@gmail.com> Signed-off-by: Thomas Huth

Re: [PATCH] target/arm: Use the max page size in a 2-stage ptw

2022-09-28 Thread Marc Zyngier
On Wed, 28 Sep 2022 05:34:53 +0100, Zenghui Yu wrote: > > [ Fix Marc's email address ] Ah, many thanks Zenghui! I was wondering whether my discussion with Richard had any result. As it turns out, it had an almost immediate result! > > On 2022/9/13 21:56, Richard Henderson wrote: > > We had onl

[PULL 10/37] tests/qtest: vhost-user-blk-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_get_tmp_dir() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-18-bmeng...@gmail.com> Signed-off-by: Thomas Huth -

[PULL 27/37] tests/qtest: microbit-test: Fix socket access for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng Sockets on Windows do not use *nix-style file descriptors, so write()/read()/close() do not work on Windows. Switch over to use send()/recv()/closesocket() which work with sockets on all platforms. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <202209251130

Re: [PATCH v4] block: Refactor get_tmp_filename()

2022-09-28 Thread Markus Armbruster
Bin Meng writes: > From: Bin Meng > > At present there are two callers of get_tmp_filename() and they are > inconsistent. > > One does: > > /* TODO: extra byte is a hack to ensure MAX_PATH space on Windows. */ > char *tmp_filename = g_malloc0(PATH_MAX + 1); > ... > ret = get_tmp_

[PULL 21/37] tests/qtest: libqtest: Exclude the *_fds APIs for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng libqmp.c::qmp_fd_vsend_fds() is not available on Windows, hence any APIs in libqtest that call libqmp.c::qmp_fd_vsend_fds() should be excluded for win32 too. This includes the following: * qtest_qmp_vsend_fds() * qtest_vqmp_fds() * qtest_qmp_fds() * qtest_qmp_add_client()

[PULL 30/37] .gitlab-ci.d/windows.yml: Display meson test logs

2022-09-28 Thread Thomas Huth
From: Bin Meng When CI fails we don't know what causes the failure. Displaying the meson test logs can be helpful. Signed-off-by: Bin Meng Reviewed-by: Thomas Huth Message-Id: <20220925113032.1949844-53-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- .gitlab-ci.d/windows.yml | 4 ++-- 1 f

[PULL 22/37] tests/qtest: {ahci, ide}-test: Use relative path for temporary files for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng These test cases uses "blkdebug:path/to/config:path/to/image" for testing. On Windows, absolute file paths contain the delimiter ':' which causes the blkdebug filename parser fail to parse filenames. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Reviewed-by: Thomas Huth

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Daniel P . Berrangé
On Tue, Sep 27, 2022 at 05:18:10PM -0400, Michael S. Tsirkin wrote: > On Tue, Sep 27, 2022 at 09:33:27AM +0100, Daniel P. Berrangé wrote: > > On Tue, Sep 27, 2022 at 01:43:15PM +0530, Ani Sinha wrote: > > > On Sun, Sep 18, 2022 at 1:58 AM Michael S. Tsirkin > > > wrote: > > > > > > > > On Fri, Se

[PULL 11/37] tests/qtest: virtio-blk-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-20-bmeng...@gmail.com> Signed-off-by: Thomas Huth

Re: Should we maybe move Cirrus-CI jobs away from Gitlab again?

2022-09-28 Thread Daniel P . Berrangé
On Tue, Sep 27, 2022 at 08:40:44PM +0200, Thomas Huth wrote: > On 27/09/2022 19.57, Daniel P. Berrangé wrote: > > On Tue, Sep 27, 2022 at 01:36:20PM -0400, Stefan Hajnoczi wrote: > > > On Tue, 27 Sept 2022 at 11:54, Daniel P. Berrangé > > > wrote: > > > > > > > > On Tue, Sep 27, 2022 at 11:44:45

[PULL 08/37] tests/qtest: virtio_blk_fuzz: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-12-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 24/37] tests/qtest: migration-test: Disable IO redirection for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng On Windows the QEMU executable is created via CreateProcess() and IO redirection does not work, so don't bother adding IO redirection to the command line. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-40-bmeng...@gmail.com> Signed-off

Re: [PATCH v2 10/11] pytest: add pytest to the meson build system

2022-09-28 Thread Thomas Huth
On 27/09/2022 23.26, Michael S. Tsirkin wrote: On Tue, Sep 06, 2022 at 02:10:56PM +0100, Daniel P. Berrangé wrote: On Tue, Jul 12, 2022 at 12:22:10PM +0530, Ani Sinha wrote: On Mon, 11 Jul 2022, John Snow wrote: On Sun, Jul 10, 2022 at 1:01 PM Ani Sinha wrote: Integrate the pytest framew

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Daniel P . Berrangé
On Tue, Sep 27, 2022 at 06:09:22PM -0400, Michael S. Tsirkin wrote: > On Tue, Sep 27, 2022 at 11:44:56PM +0200, Paolo Bonzini wrote: > > I also second the idea of using avocado instead of pytest, by the way. snip > Problem is I don't think avocado is yet at the level where I can > ask random deve

Re: Should we maybe move Cirrus-CI jobs away from Gitlab again?

2022-09-28 Thread Daniel P . Berrangé
On Tue, Sep 27, 2022 at 03:10:54PM -0400, Stefan Hajnoczi wrote: > On Tue, 27 Sept 2022 at 15:04, Thomas Huth wrote: > > > > On 27/09/2022 20.47, Stefan Hajnoczi wrote: > > > On Tue, 27 Sept 2022 at 14:40, Thomas Huth wrote: > > >> > > >> On 27/09/2022 19.57, Daniel P. Berrangé wrote: > > >>> On

[PULL 09/37] tests/qtest: ide-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-14-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 23/37] tests/qtest: bios-tables-test: Adapt the case for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng Single quotes in the arguments (oem_id='CRASH ') are not removed in the Windows environment before it is passed to the QEMU executable. The space in the argument causes the "-acpitable" option parser to think that all of its parameters are done, hence it complains: '-acpitable'

[PULL 18/37] tests/qtest: Build test-filter-{mirror, redirector} cases for posix only

2022-09-28 Thread Thomas Huth
From: Bin Meng The test-filter-{mirror,redirector} cases use socketpair() API that is only available on POSIX and should only be built for POSIX. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-30-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- t

[PULL 26/37] tests/qtest: virtio-net-failover: Disable migration tests for win32

2022-09-28 Thread Thomas Huth
From: Xuzhou Cheng These tests use the exec migration protocol, which is unsupported on Windows as of today. Disable these tests for now. Signed-off-by: Xuzhou Cheng Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-42-bmeng...@gmail.com> Signed-off-by

[PULL 09/38] linux-user: Fix strace of chmod() if mode == 0

2022-09-28 Thread Laurent Vivier
From: Helge Deller If the mode parameter of chmod() is zero, this value isn't shown when stracing a program: chmod("filename",) This patch fixes it up to show the zero-value as well: chmod("filename",000) Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Reviewed-by: Philippe Mat

[PULL 08/38] linux-user/hppa: Dump IIR on register dump

2022-09-28 Thread Laurent Vivier
From: Helge Deller Include the IIR register (which holds the opcode of the failing instruction) when dumping the hppa registers. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20220918194555.83535-7-del...@gmx.de> Signed-off-by: Lauren

[PULL 25/37] tests/qtest: ide-test: Open file in binary mode

2022-09-28 Thread Thomas Huth
From: Xuzhou Cheng By default Windows opens file in text mode, while a POSIX compliant implementation treats text files and binary files the same. The fopen() 'mode' string can include the letter 'b' to indicate binary mode shall be used. POSIX spec says the character 'b' shall have no effect, b

[PULL 17/37] tests/qtest: Skip running virtio-net-test cases that require socketpair() for win32

2022-09-28 Thread Thomas Huth
From: Bin Meng Some of the virtio-net-test test cases require socketpair() to do the test setup. Skip them for win32. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-29-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- tests/qtest/virtio-net-test.c

[PULL 02/37] tests/qtest: ahci-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Thomas Huth Message-Id: <20220925113032.1949844-6-bmeng...@gmail.com> Signed-off-by: Thomas Huth --- t

[PULL 06/38] linux-user: Log failing executable in EXCP_DUMP()

2022-09-28 Thread Laurent Vivier
From: Helge Deller Enhance the EXCP_DUMP() macro to print out the failing program too. During debugging it's sometimes hard to track down the actual failing program if you are e.g. building a whole debian package. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Reviewed-by: Philippe Ma

[PULL 20/37] tests/qtest: libqtest: Adapt global_qtest declaration for win32

2022-09-28 Thread Thomas Huth
From: Xuzhou Cheng Commit dd2107497275 ("tests/libqtest: Use libqtest-single.h in tests that require global_qtest") moved global_qtest to libqtest-single.h, by declaring global_qtest attribute to be common and weak. This trick unfortunately does not work on Windows, and building qtest test case

[PULL 28/37] tests/qtest: libqtest: Replace the call to close a socket with closesocket()

2022-09-28 Thread Thomas Huth
From: Bin Meng close() is a *nix function. It works on any file descriptor, and sockets in *nix are an example of a file descriptor. closesocket() is a Windows-specific function, which works only specifically with sockets. Sockets on Windows do not use *nix-style file descriptors, and socket() r

[PULL 19/38] linux-user: Add proper strace format strings for getdents()/getdents64()

2022-09-28 Thread Laurent Vivier
From: Helge Deller Signed-off-by: Helge Deller Message-Id: <20220924114501.21767-3-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/strace.list | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/strace.list b/linux-user/strace.list index c6f748505a9e.

[PULL 03/37] tests/qtest: aspeed_smc-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Thomas Huth Reviewed-by: Cédric Le Goater Message-Id: <20220925113032.1949844-7-bmeng...@gmail.com> Sig

[PULL 32/37] tests/qtest: hd-geo-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Message-Id: <20220927110632.1973965-13-bmeng...@gmail.com> Reviewed-by: Thomas Huth Signed-off-by: Thomas Huth ---

[PULL 31/37] tests/x86: Move common code to function in device-plug-test

2022-09-28 Thread Thomas Huth
From: Michael Labiuk Move common code for device removing to function. Signed-off-by: Michael Labiuk Message-Id: <20220920104842.605530-2-michael.lab...@virtuozzo.com> Reviewed-by: Thomas Huth Signed-off-by: Thomas Huth --- tests/qtest/device-plug-test.c | 42 ++--

[PULL 05/37] tests/qtest: cxl-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_dir_make_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-9-bmeng...@gmail.com> Signed-off-by: Thomas Huth -

[PULL 34/37] tests/qtest: qmp-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_dir_make_tmp() for a portable implementation. Signed-off-by: Bin Meng Message-Id: <20220927110632.1973965-17-bmeng...@gmail.com> Reviewed-by: Marc-André Lureau Signed-off-by: Thomas Huth

[PULL 23/38] linux-user/hppa: Allow PROT_GROWSUP and PROT_GROWSDOWN in mprotect()

2022-09-28 Thread Laurent Vivier
From: Helge Deller The hppa platform uses an upwards-growing stack and required in Linux kernels < 5.18 an executable stack for signal processing. For that some executables and libraries are marked to have an executable stack, for which glibc uses the mprotect() syscall to mark the stack like th

[PULL 13/37] tests/qtest: libqtest: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng The qtest library was written to use hardcoded /tmp directory for temporary files. Update to use g_get_tmp_dir() and g_dir_make_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-22-bmeng...@gmail.com>

[PULL 17/38] tests/tcg/linux-test: Add linux-madvise test

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich Add a test that checks madvise(MADV_DONTNEED) behavior with anonymous and file mappings in order to prevent regressions. Signed-off-by: Ilya Leoshkevich Reviewed-by: Richard Henderson Message-Id: <20220906000839.1672934-6-...@linux.ibm.com> Signed-off-by: Laurent Vivier

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Thomas Huth
On 28/09/2022 09.06, Daniel P. Berrangé wrote: On Tue, Sep 27, 2022 at 06:09:22PM -0400, Michael S. Tsirkin wrote: On Tue, Sep 27, 2022 at 11:44:56PM +0200, Paolo Bonzini wrote: I also second the idea of using avocado instead of pytest, by the way. snip Problem is I don't think avocado is y

[PULL 33/37] tests/qtest: pflash-cfi02-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Message-Id: <20220927110632.1973965-16-bmeng...@gmail.com> Reviewed-by: Marc-André Lureau Signed-off-by: Thomas Huth

[PULL 12/37] tests/qtest: virtio-scsi-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-21-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 30/38] linux-user: Combine do_futex and do_futex_time64

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Pass a boolean to select between time32 and time64. Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-2-richard.hender...@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/syscall.c | 67 -

[PULL 37/37] docs/devel: testing: Document writing portable test cases

2022-09-28 Thread Thomas Huth
From: Bin Meng Update the best practices of how to write portable test cases that can be built and run successfully on both Linux and Windows hosts. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220927110632.1973965-55-bmeng...@gmail.com> Reviewed-by: Thomas Huth Signe

[PULL 00/38] Linux user for 7.2 patches

2022-09-28 Thread Laurent Vivier
ll request 20220928 use 'max' instead of 'qemu32' / 'qemu64' add pidfd_open(), pidfd_send_signal() and pidfd_getfd() Improve madvise(MADV_DONTNEED) futex syscal rework strace improvement HP/PA fixes and improvement Misc fixes

[PULL 16/37] tests: vhost-user-bridge: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_file_open_tmp() for a portable implementation. Signed-off-by: Bin Meng Reviewed-by: Marc-André Lureau Message-Id: <20220925113032.1949844-25-bmeng...@gmail.com> Signed-off-by: Thomas Huth

[PULL 27/38] linux-user/s390x: Save/restore fpc when handling a signal

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich Linux kernel does this in fpregs_store() and fpregs_load(), so qemu-user should do this as well. Found by running valgrind's none/tests/s390x/test_sig. Signed-off-by: Ilya Leoshkevich Reviewed-by: Richard Henderson Message-Id: <20220817123902.585623-1-...@linux.ibm.com>

[PULL 04/38] linux-user: Add missing clock_gettime64() syscall strace

2022-09-28 Thread Laurent Vivier
From: Helge Deller Allow linux-user to strace the clock_gettime64() syscall. This syscall is used a lot on 32-bit guest architectures which use newer glibc versions. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220918194555.83535-3-del...@gmx.de> Signed-off-by: Lauren

[PULL 36/37] tests/qtest: boot-serial-test: Close the serial file before starting QEMU

2022-09-28 Thread Thomas Huth
From: Bin Meng This qtest executable created a serial chardev file to be passed to the QEMU executable. The serial file was created by g_file_open_tmp(), which internally opens the file with FILE_SHARE_WRITE security attribute on Windows. Based on [1], there is only one case that allows the first

[PULL 34/38] linux-user: Implement PI futexes

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Define the missing FUTEX_* constants in syscall_defs.h Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-6-richard.hender...@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/syscall.c | 10 ++ linux-use

[PULL 10/38] linux-user/hppa: Set TASK_UNMAPPED_BASE to 0xfa000000 for hppa arch

2022-09-28 Thread Laurent Vivier
From: Helge Deller On the parisc architecture the stack grows upwards. Move the TASK_UNMAPPED_BASE to high memory area as it's done by the kernel on physical machines. Signed-off-by: Helge Deller Message-Id: <20220918194555.83535-9-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/m

[PULL 35/37] tests/qtest: vhost-user-test: Avoid using hardcoded /tmp

2022-09-28 Thread Thomas Huth
From: Bin Meng This case was written to use hardcoded /tmp directory for temporary files. Update to use g_dir_make_tmp() for a portable implementation. Signed-off-by: Bin Meng Message-Id: <20220927110632.1973965-19-bmeng...@gmail.com> Reviewed-by: Marc-André Lureau Signed-off-by: Thomas Huth

[PULL 38/38] linux-user: Add parameters of getrandom() syscall for strace

2022-09-28 Thread Laurent Vivier
From: Helge Deller Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220927093538.8954-2-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/strace.list | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/strace.list b/linux-user/strace.

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Daniel P . Berrangé
On Wed, Sep 28, 2022 at 12:45:46PM +0530, Ani Sinha wrote: > On Wed, Sep 28, 2022 at 12:28 PM Daniel P. Berrangé > wrote: > > > > On Tue, Sep 27, 2022 at 05:18:10PM -0400, Michael S. Tsirkin wrote: > > > On Tue, Sep 27, 2022 at 09:33:27AM +0100, Daniel P. Berrangé wrote: > > > > On Tue, Sep 27, 2

[PULL 01/38] linux-user: use 'max' instead of 'qemu32' / 'qemu64' by default

2022-09-28 Thread Laurent Vivier
From: Daniel P. Berrangé The 'qemu64' CPU model implements the least featureful x86_64 CPU that's possible. Historically this hasn't been an issue since it was rare for OS distros to build with a higher mandatory CPU baseline. With RHEL-9, however, the entire distro is built for the x86_64-v2 AB

[PULL 18/38] linux-user: Fix TARGET_PROT_SEM for XTENSA

2022-09-28 Thread Laurent Vivier
From: Helge Deller The xtensa platform has a value of 0x10 for PROT_SEM. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220924114501.21767-2-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/syscall_defs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

If your networking is failing after updating to the latest git version of QEMU...

2022-09-28 Thread Thomas Huth
... it might have happened due to the removal of the "slirp" submodule from the git repository. For example if you see an error message like this: Parameter 'type' expects a netdev backend type this likely means that the "user" mode backend type is not available in your binary anymore. To

[PULL 15/38] linux-user: Implement stracing madvise()

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich The default implementation has several problems: the first argument is not displayed as a pointer, making it harder to grep; the third argument is not symbolized; and there are several extra unused arguments. Signed-off-by: Ilya Leoshkevich Reviewed-by: Richard Henderson

[PULL 20/38] linux-user/hppa: Add signal trampoline for hppa target

2022-09-28 Thread Laurent Vivier
From: Helge Deller In Linux kernel v5.18 the vDSO for signal trampoline was added. This code mimiks the bare minimum of this vDSO and thus avoids that the parisc emulation needs executable stacks. Signed-off-by: Helge Deller Message-Id: <20220924114501.21767-4-del...@gmx.de> Signed-off-by: Laur

Re: [PATCH] i386: Add new CPU model SapphireRapids

2022-09-28 Thread Yang Zhong
On Mon, Sep 26, 2022 at 09:51:13AM +0100, Dr. David Alan Gilbert wrote: > * Yang Zhong (yang.zh...@linux.intel.com) wrote: > > On Sat, Sep 24, 2022 at 12:01:16AM +0800, Xiaoyao Li wrote: > > > On 9/23/2022 9:30 PM, Yang Zhong wrote: > > > > On Wed, Sep 21, 2022 at 03:51:42PM +0100, Dr. David Alan G

Re: [RFC PATCH] tests/qtest: bump up QOS_PATH_MAX_ELEMENT_SIZE

2022-09-28 Thread Alex Bennée
Paolo Bonzini writes: > What is an example of one such huge path? This would mean that LTO is > changing the set of tests that are run, which is > unexpected. It does look like the LTO build runs more tests although its hard to diff as the order if different: ➜ grep "list" without-lto | wc -

[PULL 05/38] linux-user: Add pidfd_open(), pidfd_send_signal() and pidfd_getfd() syscalls

2022-09-28 Thread Laurent Vivier
From: Helge Deller I noticed those were missing when running the glib2.0 testsuite. Add the syscalls including the strace output. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220918194555.83535-4-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/strace.c

[PULL 21/38] linux-user/hppa: Drop stack guard page on hppa target

2022-09-28 Thread Laurent Vivier
From: Helge Deller The stack-overflow check when building the "grep" debian package fails on the debian hppa target. Reason is, that the guard page at the top of the stack (which is added by qemu) prevents the fault handler in the grep program to properly detect the stack overflow. The Linux ker

Re: [PATCH v9 05/10] s390x/cpu: reporting drawers and books topology to the guest

2022-09-28 Thread Pierre Morel
On 9/7/22 12:36, Janis Schoetterl-Glausch wrote: On Fri, 2022-09-02 at 09:55 +0200, Pierre Morel wrote: The guest can ask for a topology report on drawer's or book's level. Let's implement the STSI instruction's handling for the corresponding selector values. Signed-off-by: Pierre Morel ---

[PULL 03/38] linux-user: Add missing signals in strace output

2022-09-28 Thread Laurent Vivier
From: Helge Deller Some of the guest signal numbers are currently not converted to their representative names in the strace output, e.g. SIGVTALRM. This patch introduces a smart way to generate and keep in sync the host-to-guest and guest-to-host signal conversion tables for usage in the qemu si

[PULL 07/38] linux-user/hppa: Use EXCP_DUMP() to show enhanced debug info

2022-09-28 Thread Laurent Vivier
From: Helge Deller Enhance the hppa linux-user cpu_loop() to show more debugging info on hard errors. Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220918194555.83535-6-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/hppa/cpu_loop.c | 6 +- 1 file cha

[PULL 02/38] linux-user: fix readlinkat handling with magic exe symlink

2022-09-28 Thread Laurent Vivier
From: Jameson Nash Exactly the same as f17f4989fa193fa8279474c5462289a3cfe69aea before was for readlink. I suppose this was simply missed at the time. Signed-off-by: Jameson Nash Reviewed-by: Laurent Vivier Message-Id: <20220808190727.875155-1-vtjn...@gmail.com> Signed-off-by: Laurent Vivier

[PULL 26/38] linux-user: Don't assume 0 is not a valid host timer_t value

2022-09-28 Thread Laurent Vivier
From: Peter Maydell For handling guest POSIX timers, we currently use an array g_posix_timers[], whose entries are a host timer_t value, or 0 for "this slot is unused". When the guest calls the timer_create syscall we look through the array for a slot containing 0, and use that for the new timer

[PULL 32/38] linux-user: Implement FUTEX_WAKE_BITSET

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-4-richard.hender...@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/syscall.c | 1 + 1 file changed, 1 insertion(+) diff --git a/linux-user/syscall.c b/linux-user/s

[PULL 11/38] linux-user: Add strace for clock_nanosleep()

2022-09-28 Thread Laurent Vivier
From: Helge Deller Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220918194555.83535-10-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/strace.c| 15 +++ linux-user/strace.list | 3 ++- 2 files changed, 17 insertions(+), 1 deletion(-) diff

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Michael S. Tsirkin
On Wed, Sep 28, 2022 at 07:58:46AM +0100, Daniel P. Berrangé wrote: > biosbits isn't tied to QEMU versions, it is an entirely separate 3rd > party project. This binary is just providing the test env, and IIUC, > control over what executes in this env is still done by the QEMU side > test scripts. I

[PULL 12/38] linux-user: Show timespec on strace for futex()

2022-09-28 Thread Laurent Vivier
From: Helge Deller Signed-off-by: Helge Deller Reviewed-by: Laurent Vivier Message-Id: <20220918194555.83535-11-del...@gmx.de> Signed-off-by: Laurent Vivier --- linux-user/strace.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/linux-user/strace.c b/linux-user

[PULL 13/38] linux-user: Provide MADV_* definitions

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich Provide MADV_* definitions using target_mman.h header, similar to what kernel does. Most architectures use the same values, with the exception of alpha and hppa. Signed-off-by: Ilya Leoshkevich Reviewed-by: Richard Henderson Message-Id: <20220906000839.1672934-2-...@linu

[PULL 31/38] linux-user: Sink call to do_safe_futex

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Leave only the argument adjustments within the shift, and sink the actual syscall to the end. Sink the timespec conversion as well, as there will be more users. Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-3-richard.hen

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Thomas Huth
On 28/09/2022 11.35, Michael S. Tsirkin wrote: On Wed, Sep 28, 2022 at 10:31:39AM +0200, Thomas Huth wrote: On 27/09/2022 23.21, Michael S. Tsirkin wrote: On Tue, Sep 27, 2022 at 04:45:09PM +0100, Daniel P. Berrangé wrote: On Tue, Sep 27, 2022 at 07:35:13PM +0530, Ani Sinha wrote: ... Alrigh

[PULL 16/38] linux-user: Passthrough MADV_DONTNEED for certain file mappings

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich This is a follow-up for commit 892a4f6a750a ("linux-user: Add partial support for MADV_DONTNEED"), which added passthrough for anonymous mappings. File mappings can be handled in a similar manner. In order to do that, mark pages, for which mmap() was passed through, with P

[PULL 14/38] linux-user: Fix madvise(MADV_DONTNEED) on alpha

2022-09-28 Thread Laurent Vivier
From: Ilya Leoshkevich MADV_DONTNEED has a different value on alpha, compared to all the other architectures. Fix by using TARGET_MADV_DONTNEED instead of MADV_DONTNEED. Fixes: 892a4f6a750a ("linux-user: Add partial support for MADV_DONTNEED") Signed-off-by: Ilya Leoshkevich Reviewed-by: Richar

[PULL 25/38] linux-user: fix bug about missing signum convert of sigqueue

2022-09-28 Thread Laurent Vivier
From: fanwenjie Fixes: 66fb9763af ("basic signal handling") Fixes: cf8b8bfc50 ("linux-user: add support for rt_tgsigqueueinfo() system call") Signed-off-by: fanwenjie Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Laurent Vivier --- linux-user/syscall.c | 4 ++-- 1 file changed, 2 insert

[PULL 35/38] linux-user: Update print_futex_op

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Use a table for the names; print unknown values in hex, since the value contains flags. Signed-off-by: Richard Henderson Message-Id: <20220829021006.67305-7-richard.hender...@linaro.org> [lv: update print_futex() according to "linux-user: Show timespec on strace for fute

[PULL 24/38] linux-user/hppa: Fix setup_sigcontext()

2022-09-28 Thread Laurent Vivier
From: Helge Deller We don't emulate a preemptive kernel on this level, and the hppa architecture doesn't allow context switches on the gateway page. So we always have to return to sc_iaoq[] and not to gr[31]. This fixes the remaining random segfaults which still occured. Signed-off-by: Helge Del

[PULL 28/38] linux-user: Introduce stubs for ELF AT_BASE_PLATFORM

2022-09-28 Thread Laurent Vivier
From: Jiaxun Yang AT_BASE_PLATFORM is a elf auxiliary vector pointing to a string to pass some architecture information. See getauxval(3) man-page. Signed-off-by: Jiaxun Yang Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20220803103009.95972-2-jiaxun.y...@flygoat.com> Signed-off-by: Laurent

[PULL 22/38] linux-user/hppa: Increase guest stack size to 80MB for hppa target

2022-09-28 Thread Laurent Vivier
From: Helge Deller The hppa target requires a much bigger stack than many other targets, and the Linux kernel allocates 80 MB by default for it. This patch increases the guest stack for hppa to 80MB, and prevents that this default stack size gets reduced by a lower stack limit on the host. Sinc

[PULL 37/38] linux-user: Log tid for strace

2022-09-28 Thread Laurent Vivier
From: Richard Henderson Printing the same pid for all threads isn't helpful. Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-9-richard.hender...@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/strace.c | 2 +- 1 file changed, 1 insertio

Re: [PATCH v2 07/11] acpi/tests/bits: add python test that exercizes QEMU bios tables using biosbits

2022-09-28 Thread Michael S. Tsirkin
On Wed, Sep 28, 2022 at 09:35:59AM +0100, Daniel P. Berrangé wrote: > On Wed, Sep 28, 2022 at 10:31:39AM +0200, Thomas Huth wrote: > > On 27/09/2022 23.21, Michael S. Tsirkin wrote: > > > On Tue, Sep 27, 2022 at 04:45:09PM +0100, Daniel P. Berrangé wrote: > > > > On Tue, Sep 27, 2022 at 07:35:13PM

Re: [PATCH v9 09/10] s390x/cpu_topology: activating CPU topology

2022-09-28 Thread Pierre Morel
Hi Cedric, On 9/27/22 16:41, Cédric Le Goater wrote: On 9/2/22 09:55, Pierre Morel wrote: Starting with a new machine, s390-virtio-ccw-7.2, the machine property topology-disable is set to false while it is kept to true for older machine. We probably need a machine class option also because we

[PULL 33/38] linux-user: Convert signal number for FUTEX_FD

2022-09-28 Thread Laurent Vivier
From: Richard Henderson The val argument to FUTEX_FD is a signal number. Convert to match the host, as it will be converted back when the signal is delivered. Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier Message-Id: <20220829021006.67305-5-richard.hender...@linaro.org> Signed-

Re: [PATCH v7 4/5] dmg: warn when opening dmg images containing blocks of unknown type

2022-09-28 Thread Markus Armbruster
The patch looks like a useful improvement on its own. But I wonder whether users would appreciate a configuration knob to fail open right away instead of risking I/O errors later.

Re: [PATCH v9 10/10] docs/s390x: document s390x cpu topology

2022-09-28 Thread Pierre Morel
On 9/12/22 15:41, Janis Schoetterl-Glausch wrote: On Fri, 2022-09-02 at 09:55 +0200, Pierre Morel wrote: Add some basic examples for the definition of cpu topology in s390x. Signed-off-by: Pierre Morel --- docs/system/s390x/cpu_topology.rst | 88 ++ 1 file cha

Re: [PATCH v4 35/54] tests/qtest: libqtest: Install signal handler via signal()

2022-09-28 Thread Bin Meng
On Wed, Sep 28, 2022 at 5:43 PM Thomas Huth wrote: > > On 27/09/2022 13.06, Bin Meng wrote: > > From: Bin Meng > > > > At present the codes uses sigaction() to install signal handler with > > a flag SA_RESETHAND. Such usage can be covered by the signal() API > > that is a simplified interface to

[PULL 29/38] linux-user: Set ELF_BASE_PLATFORM for MIPS

2022-09-28 Thread Laurent Vivier
From: Jiaxun Yang Match most appropriate base platform string based on insn_flags. Logic is aligned with aligned with set_isa() from arch/mips/kernel/cpu-probe.c in Linux kernel. Signed-off-by: Jiaxun Yang Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20220803103009.95972-3-jiaxun.y...@flyg

  1   2   3   4   >