Skip to content

Commit

Permalink
Merge pull request #6130 from osalyk/pmem2_mem
Browse files Browse the repository at this point in the history
test: disable VARIANT_MOVDIR64B under Valgrind
  • Loading branch information
janekmi authored Jan 8, 2025
2 parents 518b742 + 584650e commit 058c17b
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/test/pmem2_mem_ext/TESTS.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!../env.py
# SPDX-License-Identifier: BSD-3-Clause
# Copyright 2020-2023, Intel Corporation
# Copyright 2020-2024, Intel Corporation
#

import testframework as t
Expand Down Expand Up @@ -113,11 +113,15 @@ def setup(self, ctx):
ret = tools.Tools(ctx.env, ctx.build).cpufd()
self.check_arch(ctx.variant(), ret.returncode)

# XXX all tests with VARIANT_AVX512F are disabled under Valgrind
# until the issue https://github.com/pmem/pmdk/issues/5640 is fixed.
# XXX all tests with VARIANT_AVX512F or VARIANT_MOVDIR64B
# are disabled under Valgrind until the issue
# https://github.com/pmem/pmdk/issues/5640 and
# https://github.com/pmem/pmdk/issues/6129 are fixed.
if ctx.valgrind is not None and ctx.valgrind.tool.name != "NONE":
if ctx.variant() == VARIANT_AVX512F:
raise futils.Skip("AVX512F unavailable under Valigrind")
if ctx.variant() == VARIANT_MOVDIR64B:
raise futils.Skip("MOVDIR64B unavailable under Valigrind")

def check_arch(self, variant, available_arch):
if variant == VARIANT_MOVDIR64B:
Expand Down

0 comments on commit 058c17b

Please sign in to comment.