Move device name validation into meta plugin

riverlijunjie · Mar 11, 2024 · 765e3e8 · 765e3e8
1 parent 78aa44a
commit 765e3e8
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 15 deletions.
diff --git a/src/inference/src/dev/core_impl.cpp b/src/inference/src/dev/core_impl.cpp
@@ -234,16 +234,6 @@ bool ov::is_config_applicable(const std::string& user_device_name, const std::st
 ov::Parsed ov::parseDeviceNameIntoConfig(const std::string& deviceName,
                                          const AnyMap& config,
                                          const bool keep_core_property) {
-    // check to the validity of device name
-    auto bracket_pos = deviceName.find(")");
-    while (bracket_pos != std::string::npos) {
-        if (bracket_pos < deviceName.length() - 1 &&
-            (deviceName[bracket_pos + 1] != ',' || bracket_pos + 1 == deviceName.length() - 1)) {
-            OPENVINO_THROW("Device with \"", deviceName, "\" name is illegal in the OpenVINO Runtime");
-        }
-        bracket_pos = deviceName.find(")", bracket_pos + 1);
-    }
-
     auto updated_config = config;
     auto updated_device_name = deviceName;
 

diff --git a/src/inference/tests/unit/core.cpp b/src/inference/tests/unit/core.cpp
@@ -382,11 +382,6 @@ TEST(CoreTests_parse_device_config, get_device_config) {
         "HETERO",
         ov::AnyMap{ov::device::priorities("MULTI,DEVICE"),
                    ov::device::properties(ov::AnyMap{{"MULTI", ov::AnyMap{ov::device::priorities("DEVICE")}}})});
-
-    // invalid device name with characters after parenthesis except comma
-    EXPECT_THROW(ov::parseDeviceNameIntoConfig("DEVICE(0)ov", ov::AnyMap{}), ov::Exception);
-    EXPECT_THROW(ov::parseDeviceNameIntoConfig("MULTI:DEVICE(0)ov,DEVICE(1)", ov::AnyMap{}), ov::Exception);
-    EXPECT_THROW(ov::parseDeviceNameIntoConfig("MULTI:DEVICE(0),DEVICE(1),", ov::AnyMap{}), ov::Exception);
 }
 
 class ApplyAutoBatchThreading : public testing::Test {