SWDEV-406099 - fix zero-length gnu_printf format warning in linux PSD…

…B logs (#3262) Change-Id: I71ca50a11a83e57d4b8d41c225a55e5138e55731
ROCm · Jun 16, 2023 · e8437f6 · e8437f6
1 parent d21183c
commit e8437f6
Show file tree

Hide file tree

Showing 11 changed files with 24 additions and 24 deletions.
diff --git a/tests/src/runtimeApi/cooperativeGrps/api_failure_tests.cpp b/tests/src/runtimeApi/cooperativeGrps/api_failure_tests.cpp
@@ -60,7 +60,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << "hip error: " << hipGetErrorString(errval);
     std::cerr << std::endl;
     std::cerr << "    Location: " << file << ":" << line << std::endl;
-    failed("");
+    failed("\n");
   }
   if (last_err != errval) {
     std::cerr << "Error: the return value of a function was not the same ";
@@ -70,7 +70,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << " (" << errval << ")" << std::endl;
     std::cerr << "hipGetLastError() returned: " << hipGetErrorString(last_err);
     std::cerr << " (" << last_err << ")" << std::endl;
-    failed("");
+    failed("\n");
   }
 }
 #define hipCheckErr(errval) \
@@ -180,7 +180,7 @@ int main(int argc, char** argv) {
         std::cerr << "to work on the device." << std::endl;
         std::cerr << "This is incorrect, and could possibly lead users ";
         std::cerr << "to try to launch kernels that will fail." << std::endl;
-        //failed("");
+        //failed("\n");
         FailFlag = 1;
         break;
       }
@@ -279,7 +279,7 @@ int main(int argc, char** argv) {
       for (int m = 0; m < 2; ++m) {
         HIPCHECK(hipFree(dev_array[m]));
       }
-      failed("");
+      failed("\n");
     }
   }
   passed();

diff --git a/tests/src/runtimeApi/cooperativeGrps/grid_group_data_sharing.cpp b/tests/src/runtimeApi/cooperativeGrps/grid_group_data_sharing.cpp
@@ -67,7 +67,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << " (" << errval << ")" << std::endl;
     std::cerr << "hipGetLastError() returned: " << hipGetErrorString(last_err);
     std::cerr << " (" << last_err << ")" << std::endl;
-    failed("");
+    failed("\n");
   }
 }
 #define hipCheckErr(errval)\
@@ -298,6 +298,6 @@ int main(int argc, char** argv) {
   if (!flag) {
     passed();
   } else {
-    failed("");
+    failed("\n");
   }
 }
diff --git a/tests/src/runtimeApi/cooperativeGrps/multi_gpu_api_failure_tests.cpp b/tests/src/runtimeApi/cooperativeGrps/multi_gpu_api_failure_tests.cpp
@@ -54,7 +54,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << "hip error: " << hipGetErrorString(errval);
     std::cerr << std::endl;
     std::cerr << "    Location: " << file << ":" << line << std::endl;
-    failed("");
+    failed("\n");
   }
   if (last_err != errval) {
     std::cerr << "Error: the return value of a function was not the same ";
@@ -64,7 +64,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << " (" << errval << ")" << std::endl;
     std::cerr << "hipGetLastError() returned: " << hipGetErrorString(last_err);
     std::cerr << " (" << last_err << ")" << std::endl;
-    failed("");
+    failed("\n");
   }
 }
 #define hipCheckErr(errval) \
@@ -580,7 +580,7 @@ int main(int argc, char** argv) {
     }
   }
   if (FailFlag == 1) {
-    failed("");
+    failed("\n");
   } else {
   passed();
   }

diff --git a/tests/src/runtimeApi/cooperativeGrps/multi_gpu_streams.cpp b/tests/src/runtimeApi/cooperativeGrps/multi_gpu_streams.cpp
@@ -97,7 +97,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << "hip error: " << hipGetErrorString(errval);
     std::cerr << std::endl;
     std::cerr << "    Location: " << file << ":" << line << std::endl;
-    failed("");
+    failed("\n");
   }
   if (last_err != errval) {
     std::cerr << "Error: the return value of a function was not the same ";
@@ -107,7 +107,7 @@ static inline void hipCheckAndFail(hipError_t errval,
     std::cerr << " (" << errval << ")" << std::endl;
     std::cerr << "hipGetLastError() returned: " << hipGetErrorString(last_err);
     std::cerr << " (" << last_err << ")" << std::endl;
-    failed("");
+    failed("\n");
   }
 }
 #define hipCheckErr(errval) \
@@ -312,7 +312,7 @@ int main(int argc, char** argv) {
       std::cerr << std::endl;
       std::cerr << "You requested " << desired_blocks << " but we can only ";
       std::cerr << "fit " << (max_blocks_per_sm * num_sm) << std::endl;
-      failed("");
+      failed("\n");
     }
 
     /*************************************************************************/
@@ -653,7 +653,7 @@ int main(int argc, char** argv) {
     }
   }
   if (FailFlag == 1) {
-    failed("");
+    failed("\n");
   } else {
     passed();
   }

diff --git a/tests/src/runtimeApi/cooperativeGrps/multi_grid_group_all_gpus.cpp b/tests/src/runtimeApi/cooperativeGrps/multi_grid_group_all_gpus.cpp
@@ -353,7 +353,7 @@ int main(int argc, char** argv) {
     std::cerr << "Requesting to run " << requested_blocks << " blocks, ";
     std::cerr << "but we can only guarantee to simultaneously run ";
     std::cerr << (max_blocks_per_sm * num_sm) << std::endl;
-    failed("");
+    failed("\n");
   }
 
   /*************************************************************************/
@@ -439,7 +439,7 @@ int main(int argc, char** argv) {
     int local_ret_val = verify_barrier_buffer(loops, requested_blocks,
                                               host_buffer[dev], num_devices);
     if (local_ret_val) {
-      failed("");
+      failed("\n");
     }
   }
 
@@ -463,7 +463,7 @@ int main(int argc, char** argv) {
     free(host_buffer[k]);
   }
   if (flag == 1) {
-    failed("");
+    failed("\n");
   } else {
     passed();
   }

diff --git a/tests/src/runtimeApi/cooperativeGrps/simple_grid_group_barrier.cpp b/tests/src/runtimeApi/cooperativeGrps/simple_grid_group_barrier.cpp
@@ -219,7 +219,7 @@ int main(int argc, char** argv) {
       std::cerr << "Requesting to run " << requested_blocks << " blocks, ";
       std::cerr << "but we can only guarantee to simultaneously run ";
       std::cerr << (max_blocks_per_sm * num_sms) << std::endl;
-      failed("");
+      failed("\n");
     }
 
     /*************************************************************************/
@@ -278,7 +278,7 @@ int main(int argc, char** argv) {
     HIPCHECK(hipFree(kernel_buffer));
     HIPCHECK(hipFree(kernel_atomic));
     if (ret_val == -1) {
-      failed("");
+      failed("\n");
     } else {
       passed();
     }

diff --git a/tests/src/runtimeApi/cooperativeGrps/simple_multi_grid_group_barrier.cpp b/tests/src/runtimeApi/cooperativeGrps/simple_multi_grid_group_barrier.cpp
@@ -352,7 +352,7 @@ int main(int argc, char** argv) {
       std::cerr << "Requesting to run " << requested_blocks << " blocks, ";
       std::cerr << "but we can only guarantee to simultaneously run ";
       std::cerr << (max_blocks_per_sm * num_sm) << std::endl;
-      failed("");
+      failed("\n");
     }
 
     /*************************************************************************/
@@ -463,7 +463,7 @@ int main(int argc, char** argv) {
     }
   }
   if (flag == 1) {
-    failed("");
+    failed("\n");
   } else {
     passed();
   }

diff --git a/tests/src/runtimeApi/memory/hipIpcMemAccessTest.cpp b/tests/src/runtimeApi/memory/hipIpcMemAccessTest.cpp
@@ -236,7 +236,7 @@ int main() {
   IfTestPassed = true;
 #endif
   if (IfTestPassed == false) {
-    failed("");
+    failed("\n");
   }
   passed();
 }
diff --git a/tests/src/runtimeApi/memory/hipMallocManaged_MultiScenario.cpp b/tests/src/runtimeApi/memory/hipMallocManaged_MultiScenario.cpp
@@ -426,7 +426,7 @@ int main(int argc, char* argv[]) {
       OverAllStatus = false;
     }
     if (!OverAllStatus) {
-      failed("");
+      failed("\n");
     }
   }
   if (p_tests == 2) {

diff --git a/tests/src/runtimeApi/memory/hipManagedKeyword.cpp b/tests/src/runtimeApi/memory/hipManagedKeyword.cpp
@@ -78,7 +78,7 @@ int main(int argc, char *argv[]) {
       OverAllStatus = false;
    }
    if (!OverAllStatus) {
-      failed("");
+      failed("\n");
    }
    passed();
 }
diff --git a/tests/src/runtimeApi/module/hipFuncSetSharedMemConfig.cpp b/tests/src/runtimeApi/module/hipFuncSetSharedMemConfig.cpp
@@ -111,6 +111,6 @@ int main() {
   if (IfTestPassed) {
     passed();
   } else {
-    failed("");
+    failed("\n");
   }
 }