v4.11.0 (#4)

Co-authored-by: HailoRT-Automation <[email protected]>
hailo-ai · Nov 7, 2022 · 2af498c · 2af498c
1 parent 905f26d
commit 2af498c
Show file tree

Hide file tree

Showing 179 changed files with 4,636 additions and 2,826 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -1,19 +1,5 @@
 cmake_minimum_required(VERSION 3.0.0)
 
-option(HAILO_BUILD_PYBIND "Build Python binding" OFF)
-option(HAILO_BUILD_PYHAILORT_VENV "Build pyhailort in venv. Only used if HAILO_BUILD_PYBIND is on" ON)
-option(HAILO_BUILD_EMULATOR "Build hailort for emulator" OFF)
-option(HAILO_BUILD_UT "Build Unit Tests" OFF)
-option(HAILO_BUILD_GSTREAMER "Compile gstreamer plugins" OFF)
-option(HAILO_BUILD_EXAMPLES "Build examples" OFF)
-option(HAILO_OFFLINE_COMPILATION "Don't download external dependencies" OFF)
-option(HAILO_MICROPROFILE "Microprofile code" OFF)
-option(HAILO_BUILD_SERVICE "Build hailort service" OFF)
-
-if(WIN32 AND ${HAILO_BUILD_SERVICE})
-    message(FATAL_ERROR "HailoRT service is not supported on Windows")
-endif()
-
 find_program(CCACHE_PROGRAM ccache)
 if(CCACHE_PROGRAM)
     set_property(GLOBAL PROPERTY RULE_LAUNCH_COMPILE "${CCACHE_PROGRAM}")
@@ -67,14 +53,6 @@ else()
     message(FATAL_ERROR "Unexpeced host, stopping build")
 endif()
 
-enable_testing()
-
-# Flag for emulator (FPGA/Veloce)
-if(HAILO_BUILD_EMULATOR)
-    message(WARNING "HailoRT is building with Emulator flag on")
-    set(HAILORT_COMPILE_OPTIONS ${HAILORT_COMPILE_OPTIONS} -DHAILO_EMULATOR)
-endif()
-
 # Enable output of compile commands during generation
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 

diff --git a/README.md b/README.md
@@ -23,6 +23,8 @@ HailoRT supports Linux and Windows, and can be compiled from sources to be integ
 
 See [**hailo.ai developer zone documentation**](https://hailo.ai/developer-zone/documentation/hailort/latest/) (registration is required for  full documentation access).
 
+For compilation instructions, see  [**Compiling HailoRT from Sources**](https://hailo.ai/developer-zone/documentation/hailort/latest/?sp_referrer=install/install.html#compiling-from-sources).
+
 For HailoRT API examples - see [**HailoRT examples**](https://github.com/hailo-ai/hailort/tree/master/hailort/libhailort/examples).
 
 ## Changelog

diff --git a/common/include/context_switch_defs.h b/common/include/context_switch_defs.h
@@ -101,6 +101,8 @@ typedef enum __attribute__((packed)) {
     CONTEXT_SWITCH_DEFS__ACTION_TYPE_VALIDATE_VDMA_CHANNEL,
     CONTEXT_SWITCH_DEFS__ACTION_TYPE_BURST_CREDITS_TASK_START,
     CONTEXT_SWITCH_DEFS__ACTION_TYPE_DDR_BUFFERING_RESET,
+    CONTEXT_SWITCH_DEFS__ACTION_TYPE_OPEN_BOUNDARY_INPUT_CHANNEL,
+    CONTEXT_SWITCH_DEFS__ACTION_TYPE_OPEN_BOUNDARY_OUTPUT_CHANNEL,
 
     /* Must be last */
     CONTEXT_SWITCH_DEFS__ACTION_TYPE_COUNT
@@ -270,7 +272,6 @@ typedef struct {
     CONTEXT_SWITCH_DEFS__stream_reg_info_t stream_reg_info;
     CONTROL_PROTOCOL__host_buffer_info_t host_buffer_info;
     uint32_t initial_credit_size;
-    bool is_single_context_app;
 } CONTEXT_SWITCH_DEFS__activate_boundary_input_data_t;
 
 typedef struct {
@@ -314,13 +315,25 @@ typedef struct {
     uint32_t buffered_rows_count;
 } CONTEXT_SWITCH_DEFS__activate_ddr_buffer_output_data_t;
 
+typedef struct {
+    uint8_t packed_vdma_channel_id;
+    CONTROL_PROTOCOL__host_buffer_info_t host_buffer_info;
+} CONTEXT_SWITCH_DEFS__open_boundary_input_channel_data_t;
+
+typedef struct {
+    uint8_t packed_vdma_channel_id;
+    CONTROL_PROTOCOL__host_buffer_info_t host_buffer_info;
+} CONTEXT_SWITCH_DEFS__open_boundary_output_channel_data_t;
+
 typedef union {
     CONTEXT_SWITCH_DEFS__activate_boundary_input_data_t activate_boundary_input_data;
     CONTEXT_SWITCH_DEFS__activate_inter_context_input_data_t activate_inter_context_input_data;
     CONTEXT_SWITCH_DEFS__activate_ddr_buffer_input_data_t activate_ddr_buffer_input_data;
     CONTEXT_SWITCH_DEFS__activate_boundary_output_data_t activate_boundary_output_data;
     CONTEXT_SWITCH_DEFS__activate_inter_context_output_data_t activate_inter_context_output_data;
     CONTEXT_SWITCH_DEFS__activate_ddr_buffer_output_data_t activate_ddr_buffer_output_data;
+    CONTEXT_SWITCH_DEFS__open_boundary_input_channel_data_t open_boundary_input_channel_data;
+    CONTEXT_SWITCH_DEFS__open_boundary_output_channel_data_t open_boundary_output_channel_data;
 } CONTEXT_SWITCH_COMMON__activate_edge_layer_action_t;
 
 typedef struct {

diff --git a/common/include/control_protocol.h b/common/include/control_protocol.h
@@ -38,7 +38,6 @@ extern "C" {
 #define CONTROL_PROTOCOL__MAX_CONTEXT_SWITCH_APPLICATIONS (32)
 #define CONTROL_PROTOCOL__MAX_NUMBER_OF_CLUSTERS (8)
 #define CONTROL_PROTOCOL__MAX_CONTROL_LENGTH (1500)
-#define CONTROL_PROTOCOL__MAX_TOTAL_CONTEXTS (128)
 #define CONTROL_PROTOCOL__SOC_ID_LENGTH (32)
 #define CONTROL_PROTOCOL__MAX_CFG_CHANNELS (4)
 #define CONTROL_PROTOCOL__MAX_NETWORKS_PER_NETWORK_GROUP (8)
@@ -1041,11 +1040,9 @@ typedef enum {
 typedef enum {
     CONTROL_PROTOCOL__HOST_BUFFER_TYPE_EXTERNAL_DESC = 0,
     CONTROL_PROTOCOL__HOST_BUFFER_TYPE_CCB,
+    CONTROL_PROTOCOL__HOST_BUFFER_TYPE_HOST_MANAGED_EXTERNAL_DESC, /* DEPRECATED */
 
-    // The buffer uses external descriptors that is host managed - the firmware don't need to config this buffer
-    CONTROL_PROTOCOL__HOST_BUFFER_TYPE_HOST_MANAGED_EXTERNAL_DESC,
-
-    /* must be last*/
+    /* must be last */
     CONTROL_PROTOCOL__HOST_BUFFER_TYPE_COUNT
 } CONTROL_PROTOCOL__HOST_BUFFER_TYPE_t;
 
@@ -1136,6 +1133,8 @@ typedef struct {
     uint8_t is_first_control_per_context;
     uint32_t is_last_control_per_context_length;
     uint8_t is_last_control_per_context;
+    uint32_t context_type_length;
+    uint8_t context_type; // CONTROL_PROTOCOL__context_switch_context_type_t
     uint32_t cfg_channels_count_length;
     uint8_t cfg_channels_count;
     uint32_t config_channel_infos_length;
@@ -1331,6 +1330,10 @@ typedef struct {
 } CONTROL_PROTOCOL__idle_time_get_measurement_response_t;
 
 typedef struct {
+    uint32_t network_group_id_length;
+    uint32_t network_group_id;
+    uint32_t context_type_length;
+    uint8_t context_type; // CONTROL_PROTOCOL__context_switch_context_type_t
     uint32_t context_index_length;
     uint8_t context_index;
     uint32_t action_list_offset_length;
@@ -1721,9 +1724,39 @@ typedef struct {
 #define CONTROL_PROTOCOL__CONTEXT_NETWORK_DATA_SINGLE_CONTROL_MAX_SIZE \
     (CONTROL_PROTOCOL__MAX_REQUEST_PARAMETERS_LENGTH - sizeof(CONTROL_PROTOCOL__context_switch_set_context_info_request_t))
 
+typedef enum {
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_CONTEXT_TYPE_PRELIMINARY,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_CONTEXT_TYPE_DYNAMIC,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_CONTEXT_TYPE_BATCH_SWITCHING,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_CONTEXT_TYPE_ACTIVATION,
+
+    /* must be last*/
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_CONTEXT_TYPE_COUNT,
+} CONTROL_PROTOCOL__context_switch_context_type_t;
+
+// TODO: After HRT-8111, CONTROL_PROTOCOL__CONTEXT_SWITCH_NUMBER_OF_NON_DYNAMIC_CONTEXTS should include the
+//       batch switching context. I.e. it's index should come before the first dynamic context; probably:
+//       - ACTIVATION
+//       - BATCH_SWITCHING
+//       - PRELIMINARY
+//       - first dynamic... (i.e. CONTROL_PROTOCOL__CONTEXT_SWITCH_NUMBER_OF_NON_DYNAMIC_CONTEXTS == 3)
+typedef enum {
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_INDEX_ACTIVATION_CONTEXT = 0,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_INDEX_PRELIMINARY_CONTEXT,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_NUMBER_OF_NON_DYNAMIC_CONTEXTS,
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_INDEX_FIRST_DYNAMIC_CONTEXT = CONTROL_PROTOCOL__CONTEXT_SWITCH_NUMBER_OF_NON_DYNAMIC_CONTEXTS,
+
+    /* must be last*/
+    CONTROL_PROTOCOL__CONTEXT_SWITCH_INDEX_COUNT,
+} CONTROL_PROTOCOL__context_switch_context_index_t;
+
+#define CONTROL_PROTOCOL__MAX_DYNAMIC_CONTEXTS (128)
+#define CONTROL_PROTOCOL__MAX_TOTAL_CONTEXTS (CONTROL_PROTOCOL__MAX_DYNAMIC_CONTEXTS + CONTROL_PROTOCOL__MAX_CONTEXT_SWITCH_APPLICATIONS * CONTROL_PROTOCOL__CONTEXT_SWITCH_NUMBER_OF_NON_DYNAMIC_CONTEXTS)
+
 typedef struct {
     bool is_first_control_per_context;
     bool is_last_control_per_context;
+    uint8_t context_type; // CONTROL_PROTOCOL__context_switch_context_type_t
     uint8_t cfg_channels_count;
     CONTROL_PROTOCOL__config_channel_info_t config_channel_infos[CONTROL_PROTOCOL__MAX_CFG_CHANNELS];
     CONTROL_PROTOCOL__stream_remap_data_t context_stream_remap_data;
@@ -1733,6 +1766,10 @@ typedef struct {
     uint8_t context_network_data[CONTROL_PROTOCOL__CONTEXT_NETWORK_DATA_SINGLE_CONTROL_MAX_SIZE];
 } CONTROL_PROTOCOL__context_switch_context_info_single_control_t;
 
+CASSERT(sizeof(CONTROL_PROTOCOL__context_switch_context_index_t)<=UINT8_MAX, control_protocol_h);
+CASSERT(sizeof(CONTROL_PROTOCOL__context_switch_context_type_t)<=UINT8_MAX, control_protocol_h);
+
+
 /* Context switch user structs */
 
 #define CONTROL_PROTOCOL__CONTEXT_NETWORK_DATA_MAX_SIZE (8 * 1024)

diff --git a/common/include/firmware_header_utils.h b/common/include/firmware_header_utils.h
@@ -22,26 +22,28 @@ extern "C" {
 #define REVISION_NUMBER_SHIFT (0)
 #define REVISION_APP_CORE_FLAG_BIT_SHIFT (27)
 #define REVISION_RESERVED_0_FLAG_BIT_SHIFT (28)
-#define REVISION_RESERVED_1_FLAG_BIT_SHIFT (29)
+#define REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_SHIFT (29)
 #define REVISION_DEV_FLAG_BIT_SHIFT (30)
 #define REVISION_SECOND_STAGE_FLAG_BIT_SHIFT (31)
 
 #define REVISION_NUMBER_WIDTH (27U)
 #define REVISION_APP_CORE_FLAG_BIT_WIDTH (1U)
 #define REVISION_RESERVED_0_FLAG_BIT_WIDTH (1U)
-#define REVISION_RESERVED_1_FLAG_BIT_WIDTH (1U)
+#define REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_WIDTH (1U)
 #define REVISION_DEV_FLAG_BIT_WIDTH (1U)
 #define REVISION_SECOND_STAGE_FLAG_BIT_WIDTH (1U)
 
 #define REVISION_NUMBER_MASK (GET_MASK(REVISION_NUMBER_WIDTH, REVISION_NUMBER_SHIFT))
 #define REVISION_APP_CORE_FLAG_BIT_MASK (GET_MASK(REVISION_APP_CORE_FLAG_BIT_WIDTH, REVISION_APP_CORE_FLAG_BIT_SHIFT))
 #define REVISION_RESERVED_0_FLAG_BIT_MASK (GET_MASK(REVISION_RESERVED_0_FLAG_BIT_WIDTH, REVISION_RESERVED_0_FLAG_BIT_SHIFT))
-#define REVISION_RESERVED_1_FLAG_BIT_MASK (GET_MASK(REVISION_RESERVED_1_FLAG_BIT_WIDTH, REVISION_RESERVED_1_FLAG_BIT_SHIFT))
+#define REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_MASK (GET_MASK(REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_WIDTH, REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_SHIFT))
 #define REVISION_DEV_FLAG_BIT_MASK (GET_MASK(REVISION_DEV_FLAG_BIT_WIDTH, REVISION_DEV_FLAG_BIT_SHIFT))
 #define REVISION_SECOND_STAGE_FLAG_BIT_MASK (GET_MASK(REVISION_SECOND_STAGE_FLAG_BIT_WIDTH, REVISION_SECOND_STAGE_FLAG_BIT_SHIFT))
 
 #define GET_REVISION_NUMBER_VALUE(binary_revision) (REVISION_NUMBER_MASK & binary_revision)
 #define IS_REVISION_DEV(binary_revision) (REVISION_DEV_FLAG_BIT_MASK == (REVISION_DEV_FLAG_BIT_MASK & binary_revision))
+#define IS_REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER(binary_revision) (REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_MASK == \
+    (REVISION_EXTENDED_CONTEXT_SWITCH_BUFFER_FLAG_BIT_MASK & binary_revision))
 #define DEV_STRING_NOTE(__is_release) ((__is_release)? "" : " (dev)")
 
 /**

diff --git a/common/include/firmware_status.h b/common/include/firmware_status.h
@@ -404,6 +404,8 @@ Updating rules:
    FIRMWARE_STATUS__X(CONTROL_PROTOCOL_STATUS_INVALID_INFER_FEATURES_LENGTH) /* DEPRECATED */\
    FIRMWARE_STATUS__X(CONTROL_PROTOCOL_STATUS_INVALID_CONFIG_CHANNEL_INFOS)\
    FIRMWARE_STATUS__X(CONTROL_PROTOCOL_STATUS_INVALID_IS_BATCH_SIZE_FLOW_LENGTH)\
+   FIRMWARE_STATUS__X(CONTROL_PROTOCOL_STATUS_INVALID_CONTEXT_SWITCH_CONTEXT_TYPE_LENGTH)\
+   FIRMWARE_STATUS__X(CONTROL_PROTOCOL_STATUS_INVALID_CONTEXT_SWITCH_CONTEXT_NETWORK_GROUP_ID_LENGTH)\
    \
    FIRMWARE_MODULE__X(FIRMWARE_MODULE__POWER_MEASUREMENT)\
    FIRMWARE_STATUS__X(HAILO_POWER_MEASUREMENT_STATUS_POWER_INIT_ERROR)\
@@ -745,6 +747,11 @@ Updating rules:
    FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_ADD_ACTION_TO_BATCH_SWITCH_BUFFER_REACHED_FORBIDDEN_MEMORY_SPACE)\
    FIRMWARE_STATUS__X(CONTEXT_SWITCH_TASK_STATUS_WAIT_FOR_INTERRUPT_INTERRUPTED_BY_BATCH_CHANGE_REQUEST)\
    FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_CANT_CLEAR_CONFIGURED_APPS_WHILE_ACTIVATED)\
+   FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_INVALID_CONTEXT_TYPE)\
+   FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_UNEXPECTED_CONTEXT_ORDER)\
+   FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_INVALID_DYNAMIC_CONTEXT_COUNT)\
+   FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_CONTEXT_INDEX_OUT_OF_RANGE)\
+   FIRMWARE_STATUS__X(CONTEXT_SWITCH_STATUS_TOTAL_PROVIDED_EDGE_LAYERS_LARGER_THEN_EXPECTED)\
    \
    FIRMWARE_MODULE__X(FIRMWARE_MODULE__D2H_EVENT_MANAGER)\
    FIRMWARE_STATUS__X(HAILO_D2H_EVENT_MANAGER_STATUS_MESSAGE_HIGH_PRIORITY_QUEUE_CREATE_FAILED)\
@@ -1049,6 +1056,12 @@ Updating rules:
    FIRMWARE_STATUS__X(BURST_CREDITS_TASK_STATUS_INFER_REACHED_TIMEOUT)\
    FIRMWARE_STATUS__X(BURST_CREDITS_TASK_STATUS_TASK_DEACTIVATED)\
    \
+   FIRMWARE_MODULE__X(FIRMWARE_MODULE__TASK_SYNC_EVENTS)\
+   FIRMWARE_STATUS__X(TASK_SYNC_EVENTS_STATUS_START_TASK_WHILE_IT_IS_RUNNING)\
+   FIRMWARE_STATUS__X(TASK_SYNC_EVENTS_STATUS_START_TASK_WHILE_TASK_NOT_DONE)\
+   FIRMWARE_STATUS__X(TASK_SYNC_EVENTS_STATUS_FAILED_TO_RESET_STATE_MACHINE)\
+   FIRMWARE_STATUS__X(TASK_SYNC_EVENTS_STATUS_DONE_TASK_WHILE_IT_IS_NOT_RUNNING)\
+
 
 
 typedef enum {

diff --git a/hailort/CMakeLists.txt b/hailort/CMakeLists.txt
@@ -1,8 +1,28 @@
 cmake_minimum_required(VERSION 3.0.0)
 
+option(HAILO_BUILD_PYBIND "Build Python binding" OFF)
+option(HAILO_BUILD_PYHAILORT_VENV "Build pyhailort in venv. Only used if HAILO_BUILD_PYBIND is on" ON)
+option(HAILO_BUILD_EMULATOR "Build hailort for emulator" OFF)
+option(HAILO_BUILD_UT "Build Unit Tests" OFF)
+option(HAILO_BUILD_HW_DEBUG_TOOL "Build hw debug tool" OFF)
+option(HAILO_BUILD_GSTREAMER "Compile gstreamer plugins" OFF)
+option(HAILO_BUILD_EXAMPLES "Build examples" OFF)
+option(HAILO_OFFLINE_COMPILATION "Don't download external dependencies" OFF)
+option(HAILO_BUILD_SERVICE "Build hailort service" OFF)
+
+if(WIN32 AND ${HAILO_BUILD_SERVICE})
+    message(FATAL_ERROR "HailoRT service is not supported on Windows")
+endif()
+
+# Flag for emulator (FPGA/Veloce)
+if(HAILO_BUILD_EMULATOR)
+    message(WARNING "HailoRT is building with Emulator flag on")
+    set(HAILORT_COMPILE_OPTIONS ${HAILORT_COMPILE_OPTIONS} -DHAILO_EMULATOR)
+endif()
+
 # Set firmware version
 add_definitions( -DFIRMWARE_VERSION_MAJOR=4 )
-add_definitions( -DFIRMWARE_VERSION_MINOR=10 )
+add_definitions( -DFIRMWARE_VERSION_MINOR=11 )
 add_definitions( -DFIRMWARE_VERSION_REVISION=0 )
 if(HAILO_BUILD_SERVICE)
     add_definitions( -DHAILO_SUPPORT_MULTI_PROCESS )
@@ -116,32 +136,12 @@ if(HAILO_BUILD_SERVICE)
     add_subdirectory(rpc)
 endif()
 
-# microprofile
-if(HAILO_MICROPROFILE)
-    add_library(microprofile STATIC EXCLUDE_FROM_ALL external/microprofile/microprofile.cpp)
-    set_target_properties(microprofile PROPERTIES
-        CXX_STANDARD              11
-        CXX_STANDARD_REQUIRED     YES
-        POSITION_INDEPENDENT_CODE ON
-    )
-    target_compile_definitions(microprofile
-        PRIVATE
-        -DMICROPROFILE_WEBSERVER=1
-        -DMICROPROFILE_GPU_TIMERS=0
-        -DMICROPROFILE_NAME_MAX_LEN=256
-        PUBLIC
-        -DMICROPROFILE_ENABLED=1
-    )
-    target_include_directories(microprofile PUBLIC external/microprofile)
-else()
-    add_library(microprofile INTERFACE)
-    target_compile_definitions(microprofile INTERFACE -DMICROPROFILE_ENABLED=0)
-    target_include_directories(microprofile INTERFACE external/microprofile)
-endif()
-
 add_subdirectory(common)
 add_subdirectory(libhailort)
 add_subdirectory(hailortcli)
+if(HAILO_BUILD_HW_DEBUG_TOOL)
+    add_subdirectory(tools/hw_debug)
+endif()
 
 if(HAILO_BUILD_SERVICE)
     add_subdirectory(hailort_service)

diff --git a/hailort/LICENSE-3RD-PARTY.md b/hailort/LICENSE-3RD-PARTY.md
@@ -12,5 +12,4 @@
 | benchmark                        | Google Inc.                       | Apache License 2.0 | 1.6.0          | Cloned entire package                      | https://github.com/google/benchmark.git                                       |
 | md5                              | Alexander Peslyak                 | cut-down BSD       | -              | Copied code from website                   | http://openwall.info/wiki/people/solar/software/public-domain-source-code/md5 |
 | pevents                          | Mahmoud Al-Qudsi                  | MIT License        | master         | Cloned entire package                      | https://github.com/neosmart/pevents.git                                       |
-| microprofile                     | Jonas Meyer                       | Unlicense License  | 3.1            | Cloned entire package                      | https://github.com/jonasmr/microprofile                                       |
 | grpc                             | Google Inc.                       | Apache License 2.0 | 1.46.0         | Cloned entire package                      | https://github.com/grpc/grpc                                                  |
diff --git a/hailort/common/filesystem.hpp b/hailort/common/filesystem.hpp
@@ -117,6 +117,11 @@ class LockedFile {
     static Expected<LockedFile> create(const std::string &file_path, const std::string &mode);
     ~LockedFile();
 
+    LockedFile(const LockedFile &other) = delete;
+    LockedFile &operator=(const LockedFile &other) = delete;
+    LockedFile &operator=(LockedFile &&other) = delete;
+    LockedFile(LockedFile &&other);
+
     int get_fd() const;
 
 private:

diff --git a/hailort/common/os/posix/filesystem.cpp b/hailort/common/os/posix/filesystem.cpp
@@ -145,7 +145,12 @@ static_assert(false, "Unsupported Platform!");
 Expected<bool> Filesystem::is_directory(const std::string &path)
 {
     struct stat path_stat{};
-    CHECK(0 == stat(path.c_str(), &path_stat), make_unexpected(HAILO_FILE_OPERATION_FAILURE),
+    auto ret_Val = stat(path.c_str(), &path_stat);
+    if (ret_Val != 0 && (errno == ENOENT)) {
+        // Directory path does not exist
+        return false;
+    }
+    CHECK(0 == ret_Val, make_unexpected(HAILO_FILE_OPERATION_FAILURE),
         "stat() on path \"{}\" failed. errno {}", path.c_str(), errno);
 
    return S_ISDIR(path_stat.st_mode);
@@ -211,12 +216,18 @@ LockedFile::LockedFile(FILE *fp, int fd) : m_fp(fp), m_fd(fd)
 
 LockedFile::~LockedFile()
 {
-    if (-1 == flock(m_fd, LOCK_UN)) {
-        LOGGER__ERROR("Failed to unlock file with errno {}", errno);
+    if (m_fp != nullptr) {
+        // The lock is released when all descriptors are closed.
+        // Since we use LOCK_EX, this is the only fd open and the lock will be release after d'tor.
         fclose(m_fp);
     }
 }
 
+LockedFile::LockedFile(LockedFile &&other) :
+    m_fp(std::exchange(other.m_fp, nullptr)),
+    m_fd(other.m_fd)
+{}
+
 int LockedFile::get_fd() const
 {
     return m_fd;

diff --git a/hailort/common/utils.hpp b/hailort/common/utils.hpp
@@ -250,16 +250,18 @@ _ISEMPTY(                                                               \
     } while(0)
 #define CHECK_AS_RPC_STATUS(cond, reply, ret_val, ...) _CHECK_AS_RPC_STATUS((cond), (reply), (ret_val), ISEMPTY(__VA_ARGS__), "" __VA_ARGS__)
 
-#define _CHECK_GRPC_STATUS(status, ret_val)                                                               \
-    do {                                                                                                  \
-        if (!status.ok()) {                                                                               \
-            LOGGER__ERROR("CHECK_GRPC_STATUS failed with error massage: {}.", status.error_message());    \
-            return ret_val;                                                                               \
-        }                                                                                                 \
+#define _CHECK_GRPC_STATUS(status, ret_val, warning_msg)                                                    \
+    do {                                                                                                    \
+        if (!status.ok()) {                                                                                 \
+            LOGGER__ERROR("CHECK_GRPC_STATUS failed with error massage: {}.", status.error_message());      \
+            LOGGER__WARNING(warning_msg);                                                                   \
+            return ret_val;                                                                                 \
+        }                                                                                                   \
     } while(0)
 
-#define CHECK_GRPC_STATUS(status) _CHECK_GRPC_STATUS(status, HAILO_RPC_FAILED)
-#define CHECK_GRPC_STATUS_AS_EXPECTED(status) _CHECK_GRPC_STATUS(status, make_unexpected(HAILO_RPC_FAILED))
+#define SERVICE_WARNING_MSG ("Make sure HailoRT service is enabled and active!")
+#define CHECK_GRPC_STATUS(status) _CHECK_GRPC_STATUS(status, HAILO_RPC_FAILED, SERVICE_WARNING_MSG)
+#define CHECK_GRPC_STATUS_AS_EXPECTED(status) _CHECK_GRPC_STATUS(status, make_unexpected(HAILO_RPC_FAILED), SERVICE_WARNING_MSG)
 #endif
 
 #define _CHECK_EXPECTED(obj, is_default, fmt, ...)                                                                                      \