From 14b76671ec493fa0be3cd3aedc3c0299142c707b Mon Sep 17 00:00:00 2001 From: Gisle Dankel Date: Mon, 10 May 2021 15:02:09 -0700 Subject: [PATCH] Rename theoretical occupancy to est. achieved occupancy Summary: Theoretical occupancy is used in Nvidia docs to mean the occupancy a kernel could achieve given sufficient input parallelism (grid size). It's more interesting in a trace to know what occupancy is actually achieved. Then if it's low, theoretical occupancy can be studied for each kernel to estimate improvement from increasing grid size. But that's something we can do as part of a recommendation or tutorial. Reviewed By: ilia-cher Differential Revision: D28327605 fbshipit-source-id: 5f80205af31b63ce49629f63b945d1287ac9522f --- libkineto/src/output_json.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libkineto/src/output_json.cpp b/libkineto/src/output_json.cpp index 50577329e..0bb25ae0e 100644 --- a/libkineto/src/output_json.cpp +++ b/libkineto/src/output_json.cpp @@ -372,7 +372,7 @@ void ChromeTraceLogger::handleGpuActivity( "warps per SM": {}, "grid": [{}, {}, {}], "block": [{}, {}, {}], - "theoretical occupancy %": {} + "est. achieved occupancy %": {} }} }},)JSON", traceActivityJson(activity, "stream "),