Branch: refs/heads/staging
Home: https://github.com/qemu/qemu
Commit: 4ff969ce8e20586f1cc340c673e346a7d18e80d7
https://github.com/qemu/qemu/commit/4ff969ce8e20586f1cc340c673e346a7d18e80d7
Author: Shameer Kolothum <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/migration/compression-tests.c
Log Message:
-----------
tests/qtest/migration: Fix compile errors when CONFIG_UADK is set
Removes accidental inclusion of unrelated functions within CONFIG_UADK
as this causes compile errors like:
error: redefinition of ‘migrate_hook_start_xbzrle’
Fixes: 932f74f3fe6e ("tests/qtest/migration: Split compression tests from
migration-test.c")
Signed-off-by: Shameer Kolothum <[email protected]>
Reviewed-by: Fabiano Rosas <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: cd196679f413c7905143e5eb9b7d63b7a7eea158
https://github.com/qemu/qemu/commit/cd196679f413c7905143e5eb9b7d63b7a7eea158
Author: Fabiano Rosas <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/migration/precopy-tests.c
Log Message:
-----------
tests/qtest/migration: Do proper cleanup in the dirty_limit test
The dirty_limit test does two migrations in a row and is leaking the
first 'to' instance. Do proper cleanup.
Reviewed-by: Peter Xu <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: 117221ad999a3d027d38ae9ddbaa94d4e1d6a303
https://github.com/qemu/qemu/commit/117221ad999a3d027d38ae9ddbaa94d4e1d6a303
Author: Fabiano Rosas <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/migration/migration-util.c
Log Message:
-----------
tests/qtest/migration: Initialize buffer in probe_o_direct_support
Valgrind complains about the probe_o_direct_support() function reading
from an uninitialized buffer. For probing O_DIRECT support we don't
actually need to write to the file, just make sure the pwrite call
doesn't reject the write. Still, write zeroes to the buffer to
suppress the warning.
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Peter Xu <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: a475e5de01f3d0938d73a1062d1ec7a3df37d621
https://github.com/qemu/qemu/commit/a475e5de01f3d0938d73a1062d1ec7a3df37d621
Author: Fabiano Rosas <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/bios-tables-test.c
Log Message:
-----------
tests/qtest/bios-tables-test: Free tables at dump_aml_files
The dump_aml_files() function calls load_expected_aml() to allocate
the tables but never frees it. Add the missing call to
free_test_data().
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: 9a9320a543704087e3f27f243de6f58af948ce56
https://github.com/qemu/qemu/commit/9a9320a543704087e3f27f243de6f58af948ce56
Author: Fabiano Rosas <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/virtio-iommu-test.c
Log Message:
-----------
tests/qtest/virtio-iommu-test: Don't pass uninitialized data into
qtest_memwrite
Valgrind complains about:
Use of uninitialised value of size 8
&
Conditional jump or move depends on uninitialised value(s)
both at:
at 0x5265931: _itoa_word (_itoa.c:180)
by 0x527EEC7: __vfprintf_internal (vfprintf-internal.c:1687)
by 0x528C8B0: __vsprintf_internal (iovsprintf.c:96)
by 0x526B920: sprintf (sprintf.c:30)
by 0x1296C7: qtest_memwrite (libqtest.c:1273)
by 0x193C04: send_map (virtio-iommu-test.c:125)
by 0x194392: test_attach_detach (virtio-iommu-test.c:214)
by 0x17BDE7: run_one_test (qos-test.c:181)
by 0x4B0699D: test_case_run (gtestutils.c:2900)
by 0x4B0699D: g_test_run_suite_internal (gtestutils.c:2988)
by 0x4B068B2: g_test_run_suite_internal (gtestutils.c:3005)
by 0x4B068B2: g_test_run_suite_internal (gtestutils.c:3005)
by 0x4B068B2: g_test_run_suite_internal (gtestutils.c:3005)
Uninitialised value was created by a stack allocation
at 0x193AFD: send_map (virtio-iommu-test.c:103)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: bc3ace691492a5aefa6e9ad1006879a30502d842
https://github.com/qemu/qemu/commit/bc3ace691492a5aefa6e9ad1006879a30502d842
Author: Peter Xu <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/migration/framework.c
Log Message:
-----------
tests/migration: Drop arch_[source|target]
Coverity complained about them. These two variables are never used now
after commit 832c732c5d ("migration-test: Create arch_opts"), and/or commit
34cc54fb35 ("tests/qtest/migration-test: Use custom asm bios for ppc64").
Resolves: Coverity CID 1568379
Resolves: Coverity CID 1568380
Signed-off-by: Peter Xu <[email protected]>
Reviewed-by: Alex Bennée <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: 06056ef1ff1e457787d277796bff927f51c092c9
https://github.com/qemu/qemu/commit/06056ef1ff1e457787d277796bff927f51c092c9
Author: Fabiano Rosas <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/migration/migration-util.c
Log Message:
-----------
tests/qtest/migration: Re-enable postcopy tests
Postcopy tests have been inadvertently disabled since commit
124a3c58b8 ("tests/qtest/migration: Move ufd_version_check to
utils"). That commit moved the ufd_version_check() function to another
file but failed to make sense of the ifdefs and includes:
The <sys/syscall> include was incorrectly dropped. It is needed to
pull in <asm/unistd.h> for __NR_userfaultfd.
The <sys/ioctl.h> was moved under the wrong ifdef.
Fixes: 124a3c58b8 ("tests/qtest/migration: Move ufd_version_check to utils")
Reviewed-by: Peter Xu <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Message-Id: <[email protected]>
Commit: d9f2b09dd6c3030cda9af9c29e26a0bdfc02bec6
https://github.com/qemu/qemu/commit/d9f2b09dd6c3030cda9af9c29e26a0bdfc02bec6
Author: Ani Sinha <[email protected]>
Date: 2025-01-02 (Thu, 02 Jan 2025)
Changed paths:
M tests/qtest/fw_cfg-test.c
Log Message:
-----------
qtest/fw-cfg: remove compiled out code
Remove code that is already compiled out. This prevents confusion.
CC: [email protected]
Signed-off-by: Ani Sinha <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Fabiano Rosas <[email protected]>
Commit: 9ee90cfc25747ab25c7da31a50f167fc5122e20e
https://github.com/qemu/qemu/commit/9ee90cfc25747ab25c7da31a50f167fc5122e20e
Author: Stefan Hajnoczi <[email protected]>
Date: 2025-01-03 (Fri, 03 Jan 2025)
Changed paths:
M tests/qtest/bios-tables-test.c
M tests/qtest/fw_cfg-test.c
M tests/qtest/migration/compression-tests.c
M tests/qtest/migration/framework.c
M tests/qtest/migration/migration-util.c
M tests/qtest/migration/precopy-tests.c
M tests/qtest/virtio-iommu-test.c
Log Message:
-----------
Merge tag 'qtest-20250102-pull-request' of https://gitlab.com/farosas/qemu
into staging
Qtest pull request
- migration-test: fix UADK build, reinstate postcopy tests and other cleanups
- a couple of memory leak fixes for bios-tables-test and virtio-iommu-test
- drop dead code from fw_cfg-test
# -----BEGIN PGP SIGNATURE-----
#
# iQJEBAABCAAuFiEEqhtIsKIjJqWkw2TPx5jcdBvsMZ0FAmd3DSMQHGZhcm9zYXNA
# c3VzZS5kZQAKCRDHmNx0G+wxnUy5EACPnvy3U2jen2Cc5Hke69byCQtMkDTAJHH5
# xlwTLoTEF25AXMjLyee9vM4MoDHl+EUaGd3zmmAu2/P5/zOHWrBh9VbkFKDDvJJS
# XTVSZ0Iv9a0LDGKsv74Pk7LUZCEanr9a0qpfYTFPuwUh9yu7pd4QeYpbEdwzKzX6
# TMpJTYOE5gUTgP6X8Ir6u3po8DChfE9Eb+Z3LhRU7S+Xi3FoUWRFgRzuvXhXLkf3
# SPNEA27XCaqWQjG8SfQ9yhafrEz9OIGq39Q2unSJ8Df8riYurrdhxYrWNZ/6URQi
# /sYoBJZ2IMBzLg4cVOWXXQwdFkmU+0LUYPOANy6MrYRXl1QJtWway3eHOj+ZlkIH
# r2ulAJWAXqIQ3Ki6Dv3TYCg0/snF8arFzmdsaAbn0M8YYQB7XAlcqHus5axEMIVO
# 0VG7rXctRyH2h7DFfKeW/G1aaK5ox+/0aJoVhjmFJSlYZ2DUx+htfsPlIJmwhRRF
# i61VaOw7NsrHsW0fEgTRurQBVVuZ2/nJT8saJwvXpFiJu0X1GQ6lmKQNlWqm5h6C
# JyLc096+vOG0Cqih7p+M3NvKTHNQZQzUPSLJU2UXItPDPv1oAmFrHpVlBQiUD37I
# h/2R6vxZIY5ei2ECFITnl+LaV4MNjGExpeOEi020KSLEz9Y8nk+i5E+4oDUct93z
# wAsWLadmxg==
# =sLkH
# -----END PGP SIGNATURE-----
# gpg: Signature made Thu 02 Jan 2025 17:03:15 EST
# gpg: using RSA key AA1B48B0A22326A5A4C364CFC798DC741BEC319D
# gpg: issuer "[email protected]"
# gpg: Good signature from "Fabiano Rosas <[email protected]>" [unknown]
# gpg: aka "Fabiano Almeida Rosas <[email protected]>"
[unknown]
# gpg: WARNING: The key's User ID is not certified with a trusted signature!
# gpg: There is no indication that the signature belongs to the owner.
# Primary key fingerprint: AA1B 48B0 A223 26A5 A4C3 64CF C798 DC74 1BEC 319D
* tag 'qtest-20250102-pull-request' of https://gitlab.com/farosas/qemu:
qtest/fw-cfg: remove compiled out code
tests/qtest/migration: Re-enable postcopy tests
tests/migration: Drop arch_[source|target]
tests/qtest/virtio-iommu-test: Don't pass uninitialized data into
qtest_memwrite
tests/qtest/bios-tables-test: Free tables at dump_aml_files
tests/qtest/migration: Initialize buffer in probe_o_direct_support
tests/qtest/migration: Do proper cleanup in the dirty_limit test
tests/qtest/migration: Fix compile errors when CONFIG_UADK is set
Signed-off-by: Stefan Hajnoczi <[email protected]>
Compare: https://github.com/qemu/qemu/compare/1ada452efc7d...9ee90cfc2574
To unsubscribe from these emails, change your notification settings at
https://github.com/qemu/qemu/settings/notifications
