On 14/01/2026 11.01, Daniel P. Berrangé wrote:
On Wed, Jan 14, 2026 at 10:59:04AM +0100, Thomas Huth wrote:
From: Thomas Huth <[email protected]>
The memlock test analyzes /proc/*/status files and expects the layout
from Linux in there. However, these files also exist on NetBSD hosts
with a completely different layout, causing this test to fail. Thus
limit the test to Linux hosts now.
Signed-off-by: Thomas Huth <[email protected]>
---
tests/functional/x86_64/meson.build | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tests/functional/x86_64/meson.build
b/tests/functional/x86_64/meson.build
index f78eec5e6cf..97286d78b8f 100644
--- a/tests/functional/x86_64/meson.build
+++ b/tests/functional/x86_64/meson.build
@@ -9,12 +9,13 @@ test_x86_64_timeouts = {
'virtio_balloon': 120,
}
-tests_x86_64_system_quick = [
+tests_x86_64_system_quick = \
+ (host_os == 'linux' ? ['memlock'] : []) + \
IMHO this should be done with a decorator in the test program, so
we keep all conditions in the source, not meson.
If we already know here that the test will only SKIP during runtime, I think
there is no need to add it to the test plan. Otherwise this will just always
waste some cycles when running the tests.
Thomas
