gpu: nvgpu: unit: fix pass calculation for L1 tests

Fix the pass/fail verdict caldulation for the main() return value and
the nvtest print out. They were not considering the skipped tests.

JIRA NVGPU-3200

Change-Id: I4383cff4ef864c49f072299ad2f38a94ca1f5230
Signed-off-by: Philip Elcan <pelcan@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/2103514
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: svc-mobile-misra <svc-mobile-misra@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
This commit is contained in:
Philip Elcan
2019-04-22 13:58:12 -04:00
committed by mobile promotions
parent 766a3902e8
commit de1edcc1ee
2 changed files with 5 additions and 3 deletions

View File

@@ -222,7 +222,8 @@ int core_exec(struct unit_fw *fw)
if (args(fw)->nvtest) {
/* special prints for NVTEST fw in GVS */
printf("[%s: %s]\n",
fw->results->nr_tests == fw->results->nr_passing ?
((fw->results->nr_tests - fw->results->nr_passing -
fw->results->nr_skipped) == 0) ?
"pass" : "fail",
args(fw)->binary_name);
}

View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 2018, NVIDIA CORPORATION. All rights reserved.
* Copyright (c) 2018-2019, NVIDIA CORPORATION. All rights reserved.
*
* Permission is hereby granted, free of charge, to any person obtaining a
* copy of this software and associated documentation files (the "Software"),
@@ -77,7 +77,8 @@ int main(int argc, char **argv)
core_print_test_status(fw);
if ((fw->results->nr_tests - fw->results->nr_passing) != 0) {
if ((fw->results->nr_tests - fw->results->nr_passing -
fw->results->nr_skipped) != 0) {
/* Some tests failed */
return -1;
}