From 7936f06582fa56c400aba86f40b01611782b43b3 Mon Sep 17 00:00:00 2001 From: kthui <18255193+kthui@users.noreply.github.com> Date: Tue, 20 Sep 2022 16:18:56 -0700 Subject: [PATCH] Remove OpenVINO from mem leak whitelist (#4897) * Fix indentation * Remove OpenVINO from mem leak whitelist --- qa/L0_client_memory_growth/test.sh | 4 ++-- qa/L0_client_valgrind/test.sh | 4 ++-- qa/common/check_valgrind_log.py | 6 +----- 3 files changed, 5 insertions(+), 9 deletions(-) diff --git a/qa/L0_client_memory_growth/test.sh b/qa/L0_client_memory_growth/test.sh index 890441fd4a..de384bdb4d 100755 --- a/qa/L0_client_memory_growth/test.sh +++ b/qa/L0_client_memory_growth/test.sh @@ -135,8 +135,8 @@ for PROTOCOL in http grpc; do else python3 ../common/check_valgrind_log.py -f $LEAKCHECK_LOG if [ $? -ne 0 ]; then - echo -e "\n***\n*** Memory leak detected\n***" - RET=1 + echo -e "\n***\n*** Memory leak detected\n***" + RET=1 fi set +e diff --git a/qa/L0_client_valgrind/test.sh b/qa/L0_client_valgrind/test.sh index 062417753c..0870aa883c 100755 --- a/qa/L0_client_valgrind/test.sh +++ b/qa/L0_client_valgrind/test.sh @@ -87,8 +87,8 @@ for PROTOCOL in http grpc; do else python3 ../common/check_valgrind_log.py -f $LEAKCHECK_LOG if [ $? -ne 0 ]; then - echo -e "\n***\n*** Memory leak detected\n***" - RET=1 + echo -e "\n***\n*** Memory leak detected\n***" + RET=1 fi fi done diff --git a/qa/common/check_valgrind_log.py b/qa/common/check_valgrind_log.py index 889d6c73b0..80f919bd42 100755 --- a/qa/common/check_valgrind_log.py +++ b/qa/common/check_valgrind_log.py @@ -35,14 +35,10 @@ # -> tensorflow::LibHDFS::LoadAndBind()::{lambda(char const*, void**)#1}::operator()(char const*, void**) # -> tensorflow::internal::LoadLibrary # -> dlerror -# * InferenceEngine::getAvailableCoresTypes() leak in OpenVINO could be due to -# https://github.com/openvinotoolkit/openvino/issues/6593. Should be fixed -# after version 2021.4(https://github.com/openvinotoolkit/openvino/issues/6593#issuecomment-878093502). -# Remove this after migrating to OpenVINO to 2022.1: DLIS-4055 LEAK_WHITE_LIST = [ 'cnmem', 'tensorflow::NewSession', 'dl-init', 'dl-open', 'dlerror', - 'libtorch', 'InferenceEngine::getAvailableCoresTypes()' + 'libtorch' ]