From 1e3243ff59e4ef8fa3a476fa7dfdc9bedd830401 Mon Sep 17 00:00:00 2001 From: rahjain-amd Date: Fri, 25 Jul 2025 21:15:50 +0530 Subject: [PATCH] Fixing 0ms and inf GB/s issue in img2col (#2565) issue : ==== ``` sh $ bin/tile_example_img2col Perf: 0 ms, inf GB/s ``` solution : ====== Problem occured because config.time_kernel is false by default. if false, then no need to calculate perf, just print proper message `image_to_coloumn: pass, No Perf generated due to config.time_kernel=0` [ROCm/composable_kernel commit: 78082855d8195467c36eabb5cd2428b94777231d] --- example/ck_tile/04_img2col/image_to_column.cpp | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/example/ck_tile/04_img2col/image_to_column.cpp b/example/ck_tile/04_img2col/image_to_column.cpp index 6380cd2994..388fcf5137 100644 --- a/example/ck_tile/04_img2col/image_to_column.cpp +++ b/example/ck_tile/04_img2col/image_to_column.cpp @@ -149,9 +149,16 @@ int main(int argc, char* argv[]) float ave_time = image_to_column(traits, args, ck_tile::stream_config{nullptr, config.time_kernel}); - std::size_t num_btype = G * NHoWo * CYX * (sizeof(OutDataType) + sizeof(InDataType)); - float gb_per_sec = num_btype / 1.E6 / ave_time; - std::cout << "Perf: " << ave_time << " ms, " << gb_per_sec << " GB/s" << std::endl; + if (config.time_kernel) + { + std::size_t num_btype = G * NHoWo * CYX * (sizeof(OutDataType) + sizeof(InDataType)); + float gb_per_sec = num_btype / 1.E6 / ave_time; + std::cout << "Perf: " << ave_time << " ms, " << gb_per_sec << " GB/s" << std::endl; + } + else + { + std::cout << "image_to_column: pass, No Perf generated due to config.time_kernel=0" << std::endl; + } bool pass = true;