From 46502a3e682dffd5f14bf4ac7ea7cdd0802df655 Mon Sep 17 00:00:00 2001
From: David Valin <dvalin@redhat.com>
Date: Wed, 4 Feb 2026 10:25:14 -0500
Subject: [PATCH 1/4] Add data validation, fix test_tools

---
 passmark/passmark_run      | 29 +++++++++++++++++------------
 passmark/results_schema.py |  7 +++++++
 2 files changed, 24 insertions(+), 12 deletions(-)
 create mode 100644 passmark/results_schema.py

diff --git a/passmark/passmark_run b/passmark/passmark_run
index 5f86729..d8b9518 100755
--- a/passmark/passmark_run
+++ b/passmark/passmark_run
@@ -21,11 +21,13 @@
 # This script automates the execution of coremark.  It will determine the
 # set of default run parameters based on the system configuration.
 #
+script_dir=$(realpath $(dirname $0))
 test_name="passmark"
 passmark_version="v1.0"
 curdir=`pwd`
 script_dir=$(dirname $(realpath $0))
 copy_dirs=""
+rtc=0
 
 if [ ! -f "/tmp/${test_name}.out" ]; then
         command="${0} $@"
@@ -67,8 +69,6 @@ arch=`uname -m`
 
 show_usage=0
 
-# Gather hardware information
-$curdir/test_tools/gather_data ${curdir}
 
 exit_out()
 {
@@ -111,7 +111,7 @@ usage()
 	echo "$1 Usage:"
 	echo "  --cpu_add n: add n cpus each iteration until hit max cpus"
 	echo "  --powers_2: starting from 1 cpu, keep doubling the cpus until max cpus"
-	source ${curdir}/test_tools/general_setup --usage
+	source ${TOOLS_BIN}/general_setup --usage
 	exit 1
 }
 
@@ -143,9 +143,9 @@ produce_report()
 		echo Ran > test_results_report
 
 		located=0
-		[ -f results.csv ] && rm results.csv
+		[ -f results_passmark.csv ] && rm results_passmark.csv
 
-		$TOOLS_BIN/test_header_info --front_matter --results_file results.csv --host $to_configuration --sys_type $to_sys_type --tuned $to_tuned_setting --results_version $passmark_version --test_name $test_name --field_header "Testname,Operations"
+		$TOOLS_BIN/test_header_info --front_matter --results_file results_passmark.csv --host $to_configuration --sys_type $to_sys_type --tuned $to_tuned_setting --results_version $passmark_version --test_name $test_name --field_header "Testname,Operations"
 
 		while IFS= read -r line
 		do
@@ -169,7 +169,7 @@ produce_report()
 			ltest_name=`echo $line | cut -d':' -f1`
 			results=`echo $line | cut -d' ' -f2`
 			data_string=$(build_data_string "${ltest_name}" "${results}" "${start_time}" "${end_time}")
-			echo "${data_string}" >> results.csv
+			echo "${data_string}" >> results_passmark.csv
 		done < "$summary_file"
 	fi
 	echo Checksum: Not applicable for summary file >> $summary_file
@@ -207,12 +207,14 @@ install_tools()
 	# Check to see if the test tools directory exists.  If it does, we do not need to
 	# clone the repo.
 	#
-	if [ ! -d "test_tools" ]; then
-		git clone $tools_git test_tools
+	if [ ! -d "${HOME}/test_tools" ]; then
+		git clone $tools_git ${HOME}/test_tools
 		if [ $? -ne 0 ]; then
 			exit_out "pulling git $tools_git failed." 1
 		fi
 	fi
+	TOOLS_BIN=${HOME}/test_tools
+	export TOOLS_BIN
 
 	if [ $show_usage -eq 1 ]; then
 		usage $1
@@ -344,12 +346,15 @@ run_passmark()
 	if [[ -n "$copy_dirs" ]]; then
 		copy_dirs="--copy_dir $copy_dirs"
 	fi
-
-	${curdir}/test_tools/save_results --curdir $curdir --home_root $to_home_root --results results.csv --test_name $test_name --tuned_setting=$to_tuned_setting --version NONE --user $to_user  --other_files "passmark.summary,results_all_*,${test_name}_${iter}.out,test_results_report" $copy_dirs
+	${TOOLS_BIN}/csv_to_json $to_json_flags --csv_file results_passmark.csv --output_file results_passmark.json
+	${TOOLS_BIN}/verify_results $to_verify_flags --schema_file $script_dir/results_schema.py --class_name Passmark_Results --file results_passmark.json
+	rtc=$?
+	${TOOLS_BIN}/save_results --curdir $curdir --home_root $to_home_root --results results_passmark.csv --test_name $test_name --tuned_setting=$to_tuned_setting --version NONE --user $to_user  --other_files "passmark.summary,results_all_*,${test_name}_${iter}.out,test_results_report" $copy_dirs
 	popd > /dev/null
 }
 
 install_tools "$@"
+$TOOLS_BIN/gather_data ${curdir}
 
 #
 # Variables set by general setup.
@@ -364,7 +369,7 @@ install_tools "$@"
 # to_tuned_setting: tuned setting
 #
 
-source ${curdir}/test_tools/general_setup "$@"
+source ${TOOLS_BIN}/general_setup "$@"
 if [[ $to_use_pcp -eq 1 ]]; then
 	source $TOOLS_BIN/pcp/pcp_commands.inc
 fi
@@ -427,4 +432,4 @@ fi
 $TOOLS_BIN/package_tool --no_packages $to_no_pkg_install --wrapper_config $script_dir/../passmark.json
 
 run_passmark
-exit 0
+exit $rtc
diff --git a/passmark/results_schema.py b/passmark/results_schema.py
new file mode 100644
index 0000000..ac3173f
--- /dev/null
+++ b/passmark/results_schema.py
@@ -0,0 +1,7 @@
+import pydantic
+import datetime
+class Passmark_Results(pydantic.BaseModel):
+    Testname: str
+    Operations: float = pydantic.Field(gt=0, allow_inf_nan=False)
+    Start_Date: datetime.datetime
+    End_Date: datetime.datetime

From 0447e22e1a80b346118024df49ad7182753b124f Mon Sep 17 00:00:00 2001
From: David Valin <dvalin@redhat.com>
Date: Wed, 4 Feb 2026 17:32:37 -0500
Subject: [PATCH 2/4] Use enumeration for tests.

---
 passmark/results_schema.py | 33 ++++++++++++++++++++++++++++++++-
 1 file changed, 32 insertions(+), 1 deletion(-)

diff --git a/passmark/results_schema.py b/passmark/results_schema.py
index ac3173f..35018a5 100644
--- a/passmark/results_schema.py
+++ b/passmark/results_schema.py
@@ -1,7 +1,38 @@
 import pydantic
 import datetime
+
+from enum import Enum
+
+class testname(Enum):
+    CPU_INTEGER_MATH = "CPU_INTEGER_MATH"
+    CPU_FLOATINGPOINT_MATH = "CPU_FLOATINGPOINT_MATH"
+    CPU_PRIME = "CPU_PRIME"
+    CPU_SORTING = "CPU_SORTING"
+    CPU_ENCRYPTION = "CPU_ENCRYPTION"
+    CPU_COMPRESSION = "CPU_COMPRESSION"
+    CPU_SINGLETHREAD = "CPU_SINGLETHREAD"
+    CPU_PHYSICS = "CPU_PHYSICS"
+    CPU_MATRIX_MULT_SSE = "CPU_MATRIX_MULT_SSE"
+    CPU_mm = "CPU_mm"
+    CPU_sse = "CPU_sse"
+    CPU_fma = "CPU_fma"
+    CPU_avx = "CPU_avx"
+    CPU_avx512 = "CPU_avx512"
+    m_CPU_enc_SHA = "m_CPU_enc_SHA"
+    m_CPU_enc_AES = "m_CPU_enc_AES"
+    m_CPU_enc_ECDSA = "m_CPU_enc_ECDSA"
+    ME_ALLOC_S = "ME_ALLOC_S"
+    ME_READ_S = "ME_READ_S"
+    ME_READ_L = "ME_READ_L"
+    ME_WRITE = "ME_WRITE"
+    ME_LARGE = "ME_LARGE"
+    ME_LATENCY = "ME_LATENCY"
+    ME_THREADED = "ME_THREADED"
+    SUMM_CPU = "SUMM_CPU"
+    SUMM_ME = "SUMM_ME"
+
 class Passmark_Results(pydantic.BaseModel):
-    Testname: str
+    Testname: testname
     Operations: float = pydantic.Field(gt=0, allow_inf_nan=False)
     Start_Date: datetime.datetime
     End_Date: datetime.datetime

From 60f81cf648bcb8d38327ee9865cbfbccb3427bc8 Mon Sep 17 00:00:00 2001
From: David Valin <dvalin@redhat.com>
Date: Thu, 5 Feb 2026 08:20:51 -0500
Subject: [PATCH 3/4] Review updates.

---
 passmark/passmark_run | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/passmark/passmark_run b/passmark/passmark_run
index d8b9518..8b5f6ba 100755
--- a/passmark/passmark_run
+++ b/passmark/passmark_run
@@ -207,14 +207,14 @@ install_tools()
 	# Check to see if the test tools directory exists.  If it does, we do not need to
 	# clone the repo.
 	#
-	if [ ! -d "${HOME}/test_tools" ]; then
-		git clone $tools_git ${HOME}/test_tools
+	TOOLS_BIN=${HOME}/test_tools
+	export TOOLS_BIN
+	if [ ! -d "${TOOLS_BIN}" ]; then
+		git clone $tools_git ${TOOLS_BIN}
 		if [ $? -ne 0 ]; then
 			exit_out "pulling git $tools_git failed." 1
 		fi
 	fi
-	TOOLS_BIN=${HOME}/test_tools
-	export TOOLS_BIN
 
 	if [ $show_usage -eq 1 ]; then
 		usage $1

From 3e2e61b6d476cb47711839913d249f98e59f28fb Mon Sep 17 00:00:00 2001
From: David Valin <dvalin@redhat.com>
Date: Thu, 5 Feb 2026 12:10:01 -0500
Subject: [PATCH 4/4] Remove duplicate line

---
 passmark/passmark_run | 1 -
 1 file changed, 1 deletion(-)

diff --git a/passmark/passmark_run b/passmark/passmark_run
index 8b5f6ba..57f50ed 100755
--- a/passmark/passmark_run
+++ b/passmark/passmark_run
@@ -25,7 +25,6 @@ script_dir=$(realpath $(dirname $0))
 test_name="passmark"
 passmark_version="v1.0"
 curdir=`pwd`
-script_dir=$(dirname $(realpath $0))
 copy_dirs=""
 rtc=0