Skip to content

Commit d647c30

Browse files
committed
[CPU][core] Remove OV_CPU_NODE_SCOPED_TASK* macros, reuse existing handles in Node::execute tracing
1 parent 792ddf3 commit d647c30

File tree

4 files changed

+2
-46
lines changed

4 files changed

+2
-46
lines changed

src/common/itt/include/openvino/itt.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ typedef struct handle_ {
4141
* @cond
4242
*/
4343
namespace internal {
44-
bool is_initialized();
4544
domain_t domain(const char* name);
4645
handle_t handle(const char* name);
4746
void taskBegin(domain_t d, handle_t t);

src/common/itt/src/itt.cpp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ namespace internal {
2121

2222
static __itt_collection_state state = __itt_get_collection_state();
2323

24-
bool is_initialized() {
24+
static inline bool is_initialized() {
2525
return state == __itt_collection_init_successful;
2626
}
2727

@@ -110,10 +110,6 @@ void regionEnd(domain_t d) {
110110

111111
#else
112112

113-
bool is_initialized() {
114-
return false;
115-
}
116-
117113
domain_t domain(const char*) {
118114
return nullptr;
119115
}

src/plugins/intel_cpu/src/itt.h

Lines changed: 0 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -25,45 +25,6 @@ OV_ITT_DOMAIN(ov_intel_cpu, "ov::intel_cpu");
2525
OV_ITT_DOMAIN(ov_intel_cpu_LT, "ov::intel_cpu::lt");
2626
} // namespace ov::intel_cpu::itt::domains
2727

28-
namespace ov::intel_cpu::itt {
29-
30-
class ScopedOpExecTask {
31-
public:
32-
explicit ScopedOpExecTask(const char* name) noexcept {
33-
if (openvino::itt::internal::is_initialized()) {
34-
m_handle = openvino::itt::handle(name);
35-
openvino::itt::internal::taskBegin(::ov::itt::domains::ov_op_exec(), m_handle);
36-
}
37-
}
38-
explicit ScopedOpExecTask(const std::string& name) noexcept : ScopedOpExecTask(name.c_str()) {}
39-
~ScopedOpExecTask() noexcept {
40-
if (openvino::itt::internal::is_initialized()) {
41-
openvino::itt::internal::taskEnd(::ov::itt::domains::ov_op_exec());
42-
}
43-
}
44-
45-
ScopedOpExecTask(const ScopedOpExecTask&) = delete;
46-
ScopedOpExecTask& operator=(const ScopedOpExecTask&) = delete;
47-
48-
private:
49-
openvino::itt::handle_t m_handle{};
50-
};
51-
52-
} // namespace ov::intel_cpu::itt
53-
54-
#if defined(SELECTIVE_BUILD_ANALYZER) || defined(SELECTIVE_BUILD)
55-
# define OV_CPU_NODE_SCOPED_TASK(taskName) OV_ITT_SCOPED_TASK(::ov::itt::domains::ov_op_exec, taskName)
56-
# define OV_CPU_NODE_SCOPED_TASK_BASE(taskName) OV_ITT_SCOPED_TASK_BASE(::ov::itt::domains::ov_op_exec, taskName)
57-
#else
58-
# define OV_CPU_NODE_SCOPE_CONCAT_IMPL(x, y) x##y
59-
# define OV_CPU_NODE_SCOPE_CONCAT(x, y) OV_CPU_NODE_SCOPE_CONCAT_IMPL(x, y)
60-
# define OV_CPU_NODE_SCOPED_TASK_INTERNAL(varName, taskName) ::ov::intel_cpu::itt::ScopedOpExecTask varName(taskName)
61-
# define OV_CPU_NODE_SCOPED_TASK(taskName) \
62-
OV_CPU_NODE_SCOPED_TASK_INTERNAL(OV_CPU_NODE_SCOPE_CONCAT(cpuNodeScopedTaskGuard, __LINE__), taskName)
63-
# define OV_CPU_NODE_SCOPED_TASK_BASE(taskName) \
64-
OV_CPU_NODE_SCOPED_TASK_INTERNAL(OV_CPU_NODE_SCOPE_CONCAT(cpuNodeScopedTaskGuardBase, __LINE__), taskName)
65-
#endif
66-
6728
#if defined(SELECTIVE_BUILD_ANALYZER)
6829
# define CPU_LPT_SCOPE(region) OV_SCOPE(intel_cpu, region)
6930
# define CPU_GRAPH_OPTIMIZER_SCOPE(region) OV_SCOPE(intel_cpu, region)

src/plugins/intel_cpu/src/node.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -823,7 +823,7 @@ void Node::updateDynamicParams() {
823823
}
824824

825825
void Node::execute(const dnnl::stream& strm, int numaId) {
826-
OV_CPU_NODE_SCOPED_TASK_BASE(getTypeStr());
826+
OV_ITT_SCOPED_TASK_BASE(itt::domains::ov_intel_cpu, profiling.execute);
827827
if (isDynamicNode()) {
828828
executeDynamic(strm, numaId);
829829
} else {

0 commit comments

Comments
 (0)