Pull in a fix to the test harness

Resolves: RHEL-98531
This commit is contained in:
Jeff Moyer 2025-08-27 10:04:01 -04:00
parent 3054953b3c
commit f7fe897aea
2 changed files with 48 additions and 1 deletions

View File

@ -0,0 +1,42 @@
test/dax.sh: adjust trace parsing of fault results
JIRA: https://issues.redhat.com/browse/RHEL-98531
commit 30aca9314e91f385c85dfc1866f0da282dbba1cd
Author: Dan Williams <dan.j.williams@intel.com>
Date: Wed Jun 18 15:21:28 2025 -0700
test/dax.sh: adjust trace parsing of fault results
With current kernel+tracecmd combinations stdout is no longer purely
trace records and column "21" is no longer the vmfault_t result.
Drop, if present, the diagnostic print of how many CPUs are in the trace
and use the more universally compatible assumption that the fault result
is the last column rather than a specific column.
Note that dax.sh is not a unit test on its own, but rather is included in
both dax-xfs.h and dax-ext4.sh.
[ as: only feed trace lines with dax_pmd_fault_done to awk ]
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/20250618222130.672621-4-dan.j.williams@intel.com
Signed-off-by: Alison Schofield <alison.schofield@intel.com>
Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
diff --git a/test/dax.sh b/test/dax.sh
index 3ffbc80..2eec7a6 100755
--- a/test/dax.sh
+++ b/test/dax.sh
@@ -43,7 +43,7 @@ run_test() {
fi
fi
count=$((count + 1))
- done < <(trace-cmd report | awk '{ print $21 }')
+ done < <(trace-cmd report | grep dax_pmd_fault_done | awk '{ print $NF }')
if [ $count -lt 10 ]; then
cleanup "$1"

View File

@ -1,10 +1,11 @@
Name: ndctl
Version: 82
Release: 2%{?dist}
Release: 3%{?dist}
Summary: Manage "libnvdimm" subsystem devices (Non-volatile Memory)
License: GPL-2.0-only and LGPL-2.1-only and CC0-1.0 and MIT
Url: https://github.com/pmem/ndctl
Source0: https://github.com/pmem/%{name}/archive/v%{version}.tar.gz#/%{name}-%{version}.tar.gz
Patch0: 30aca93-test-dax.sh-adjust-trace-parsing-of-fault-results.patch
Requires: ndctl-libs%{?_isa} = %{version}-%{release}
Requires: daxctl-libs%{?_isa} = %{version}-%{release}
@ -243,6 +244,10 @@ fi
%changelog
* Wed Aug 27 2025 Jeff Moyer <jmoyer@redhat.com> - 82-3
- Pull in a fix to the test harness
- Resolves: RHEL-98531
* Tue Aug 26 2025 Jeff Moyer <jmoyer@redhat.com> - 82-2
- bump release for rebuild to re-run tests.
- Related: RHEL-99217