Skip to content

Commit 2fd7684

Browse files
divyanshkmeta-codesync[bot]
authored andcommitted
Rename variable in disablePtiActivities (#1209)
Summary: In `disablePtiActivities`, we should have `activityDisabled` instead of `activityEnabled` to track activity status. Pull Request resolved: #1209 Reviewed By: aaronenyeshi Differential Revision: D89735444 Pulled By: divyanshk fbshipit-source-id: 27ec6383455f7417d4128b84eb65ed9fdcff9d1b
1 parent d4ea1f9 commit 2fd7684

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

libkineto/src/plugin/aiupti/AiuptiActivityApi.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -271,33 +271,33 @@ void AiuptiActivityApi::enableAiuptiActivities(
271271
void AiuptiActivityApi::disablePtiActivities(
272272
const std::set<ActivityType>& selected_activities) {
273273
#ifdef HAS_AIUPTI
274-
bool activityEnabled = false;
274+
bool activityDisabled = false;
275275
for (const auto& activity : selected_activities) {
276276
if (activity == ActivityType::GPU_MEMCPY) {
277277
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_MEMCPY));
278278
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_MEMCPY2));
279279
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_SYNCHRONIZATION));
280-
activityEnabled = true;
280+
activityDisabled = true;
281281
}
282282
if (activity == ActivityType::GPU_MEMSET) {
283283
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_MEMORY));
284-
activityEnabled = true;
284+
activityDisabled = true;
285285
}
286286
if (activity == ActivityType::CONCURRENT_KERNEL) {
287287
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_CMPT));
288-
activityEnabled = true;
288+
activityDisabled = true;
289289
}
290290
if (activity == ActivityType::PRIVATEUSE1_RUNTIME) {
291291
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_RUNTIME));
292-
activityEnabled = true;
292+
activityDisabled = true;
293293
}
294294
if (activity == ActivityType::PRIVATEUSE1_DRIVER) {
295295
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_DRIVER));
296-
activityEnabled = true;
296+
activityDisabled = true;
297297
}
298298
}
299299

300-
if (activityEnabled == false) {
300+
if (activityDisabled == false) {
301301
const char* env_value = std::getenv("ProfilerActivity");
302302
if (env_value != nullptr && std::string(env_value) == "PrivateUse1") {
303303
AIUPTI_CALL(aiuptiActivityDisable(AIUPTI_ACTIVITY_KIND_MEMCPY));

0 commit comments

Comments
 (0)