From 659cad2b0f64f72de3176243f284b613346934be Mon Sep 17 00:00:00 2001 From: Brian Kelley Date: Wed, 6 Dec 2023 12:42:42 -0700 Subject: [PATCH 1/3] Add header/legend line to kp_reader describing what each column means --- profiling/simple-kernel-timer/kp_reader.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/profiling/simple-kernel-timer/kp_reader.cpp b/profiling/simple-kernel-timer/kp_reader.cpp index 094c0a8f1..a2d8916df 100644 --- a/profiling/simple-kernel-timer/kp_reader.cpp +++ b/profiling/simple-kernel-timer/kp_reader.cpp @@ -102,6 +102,9 @@ int main(int argc, char* argv[]) { } } + printf("(Type) Total Time, Call Count, Avg. Time per Call, %%Total Time in Kernels, %%Total Program Time\n"); + printf("-------------------------------------------------------------------------\n\n"); + printf("Regions: \n\n"); for (unsigned int i = 0; i < kernelInfo.size(); i++) { From b16ed731e1242516c0676b84d3901bfea7d38803 Mon Sep 17 00:00:00 2001 From: Brian Kelley Date: Wed, 6 Dec 2023 12:50:51 -0700 Subject: [PATCH 2/3] formatting --- profiling/simple-kernel-timer/kp_reader.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/profiling/simple-kernel-timer/kp_reader.cpp b/profiling/simple-kernel-timer/kp_reader.cpp index a2d8916df..d6e23ac3b 100644 --- a/profiling/simple-kernel-timer/kp_reader.cpp +++ b/profiling/simple-kernel-timer/kp_reader.cpp @@ -102,8 +102,12 @@ int main(int argc, char* argv[]) { } } - printf("(Type) Total Time, Call Count, Avg. Time per Call, %%Total Time in Kernels, %%Total Program Time\n"); - printf("-------------------------------------------------------------------------\n\n"); + printf( + "(Type) Total Time, Call Count, Avg. Time per Call, %%Total Time in " + "Kernels, %%Total Program Time\n"); + printf( + "------------------------------------------------------------------------" + "-\n\n"); printf("Regions: \n\n"); From 19b41e3fe4628ab6cff36d8bd05b6cd28a9b4833 Mon Sep 17 00:00:00 2001 From: Brian Kelley Date: Wed, 6 Dec 2023 13:01:05 -0700 Subject: [PATCH 3/3] Add a space to header, matching region/kernel lines --- profiling/simple-kernel-timer/kp_reader.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/profiling/simple-kernel-timer/kp_reader.cpp b/profiling/simple-kernel-timer/kp_reader.cpp index d6e23ac3b..12bd76709 100644 --- a/profiling/simple-kernel-timer/kp_reader.cpp +++ b/profiling/simple-kernel-timer/kp_reader.cpp @@ -103,7 +103,7 @@ int main(int argc, char* argv[]) { } printf( - "(Type) Total Time, Call Count, Avg. Time per Call, %%Total Time in " + " (Type) Total Time, Call Count, Avg. Time per Call, %%Total Time in " "Kernels, %%Total Program Time\n"); printf( "------------------------------------------------------------------------"