From 62660056d7a80cdc3f2e1c14cc4da9a7ad0bc5b3 Mon Sep 17 00:00:00 2001 From: liudingyan Date: Thu, 11 Mar 2021 17:48:08 +0800 Subject: [PATCH 1/2] report errmsg --- ge/host_cpu_engine/engine/host_cpu_engine.cc | 2 ++ .../ops_kernel_store/host_cpu_ops_kernel_builder.cc | 10 ++++++++++ ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc | 2 ++ 3 files changed, 14 insertions(+) diff --git a/ge/host_cpu_engine/engine/host_cpu_engine.cc b/ge/host_cpu_engine/engine/host_cpu_engine.cc index cdbad1ed..657ccfbd 100644 --- a/ge/host_cpu_engine/engine/host_cpu_engine.cc +++ b/ge/host_cpu_engine/engine/host_cpu_engine.cc @@ -35,6 +35,8 @@ Status HostCpuEngine::Initialize(const std::map &options) { ops_kernel_store_ = MakeShared(); if (ops_kernel_store_ == nullptr) { GELOGE(FAILED, "Make HostCpuOpsKernelInfoStore failed."); + + REPORT_INNER_ERROR("E19999", "Make HostCpuOpsKernelInfoStore failed."); return FAILED; } } diff --git a/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc b/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc index adb252bc..bb528008 100644 --- a/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc +++ b/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc @@ -40,6 +40,7 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { OpDescPtr op_desc = ge_node.GetOpDesc(); if (op_desc == nullptr) { GELOGE(FAILED, "CalcOpRunningParam failed, as op desc is null"); + REPORT_INPUT_ERROR("E19999", "CalcOpRunningParam failed, as op desc is null"); return FAILED; } @@ -76,6 +77,11 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { GELOGE(FAILED, "Calc op[%s:%s] out[%zu] mem size failed, mem_size=%ld, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, output_mem_size, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); + + REPORT_INNER_ERROR("E19999", "Calc op[%s:%s] out[%zu] mem size failed, mem_size=%ld, format=%s, data_type=%s.", + name.c_str(), type.c_str(), i, output_mem_size, TypeUtils::FormatToSerialString(format).c_str(), + TypeUtils::DataTypeToSerialString(data_type).c_str()); + return FAILED; } GELOGI("Calc op[%s:%s] out[%zu] mem size is %ld, format=%s, data_type=%s.", @@ -86,6 +92,10 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { if (op_desc->UpdateOutputDesc(static_cast(i), output_tensor) != GRAPH_SUCCESS) { GELOGE(FAILED, "Update op[%s:%s] out[%zu] desc failed, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); + + REPORT_INNER_ERROR("E19999", "Update op[%s:%s] out[%zu] desc failed, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, + TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); + return FAILED; } } diff --git a/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc b/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc index 176ae579..f26aa154 100644 --- a/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc +++ b/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc @@ -33,6 +33,8 @@ std::shared_ptr OpFactory::CreateOp(const Node &node, RunContext &run_contex } GELOGE(FAILED, "Not supported OP, type = %s, name = %s", node.GetType().c_str(), node.GetName().c_str()); + REPORT_INPUT_ERROR("E19999", "CreateOp failed, node_type = %s, node_name = %s", node.GetType().c_str(), node.GetName().c_str()); + return nullptr; } From 000c5694a6c40dbc9964a4650f34ee65c1717504 Mon Sep 17 00:00:00 2001 From: liudingyan Date: Thu, 11 Mar 2021 19:18:57 +0800 Subject: [PATCH 2/2] update and report errmsg --- ge/host_cpu_engine/engine/host_cpu_engine.cc | 2 +- .../ops_kernel_store/host_cpu_ops_kernel_builder.cc | 6 +++--- ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/ge/host_cpu_engine/engine/host_cpu_engine.cc b/ge/host_cpu_engine/engine/host_cpu_engine.cc index 657ccfbd..6f17cfcd 100644 --- a/ge/host_cpu_engine/engine/host_cpu_engine.cc +++ b/ge/host_cpu_engine/engine/host_cpu_engine.cc @@ -36,7 +36,7 @@ Status HostCpuEngine::Initialize(const std::map &options) { if (ops_kernel_store_ == nullptr) { GELOGE(FAILED, "Make HostCpuOpsKernelInfoStore failed."); - REPORT_INNER_ERROR("E19999", "Make HostCpuOpsKernelInfoStore failed."); + REPORT_INNER_ERROR("E19999", "Make HostCpuOpsKernelInfoStore failed."); return FAILED; } } diff --git a/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc b/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc index bb528008..db6dfda0 100644 --- a/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc +++ b/ge/host_cpu_engine/ops_kernel_store/host_cpu_ops_kernel_builder.cc @@ -40,7 +40,7 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { OpDescPtr op_desc = ge_node.GetOpDesc(); if (op_desc == nullptr) { GELOGE(FAILED, "CalcOpRunningParam failed, as op desc is null"); - REPORT_INPUT_ERROR("E19999", "CalcOpRunningParam failed, as op desc is null"); + REPORT_INNER_ERROR("E19999", "CalcOpRunningParam failed, as op desc is null"); return FAILED; } @@ -78,7 +78,7 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { name.c_str(), type.c_str(), i, output_mem_size, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); - REPORT_INNER_ERROR("E19999", "Calc op[%s:%s] out[%zu] mem size failed, mem_size=%ld, format=%s, data_type=%s.", + REPORT_INNER_ERROR("E19999", "Calc op[%s:%s] out[%zu] mem size failed, mem_size=%ld, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, output_mem_size, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); @@ -93,7 +93,7 @@ Status HostCpuOpsKernelBuilder::CalcOpRunningParam(Node &ge_node) { GELOGE(FAILED, "Update op[%s:%s] out[%zu] desc failed, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); - REPORT_INNER_ERROR("E19999", "Update op[%s:%s] out[%zu] desc failed, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, + REPORT_INNER_ERROR("E19999", "Update op[%s:%s] out[%zu] desc failed, format=%s, data_type=%s.", name.c_str(), type.c_str(), i, TypeUtils::FormatToSerialString(format).c_str(), TypeUtils::DataTypeToSerialString(data_type).c_str()); return FAILED; diff --git a/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc b/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc index f26aa154..3b0cb555 100644 --- a/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc +++ b/ge/host_cpu_engine/ops_kernel_store/op/op_factory.cc @@ -32,8 +32,8 @@ std::shared_ptr OpFactory::CreateOp(const Node &node, RunContext &run_contex return iter->second(node, run_context); } - GELOGE(FAILED, "Not supported OP, type = %s, name = %s", node.GetType().c_str(), node.GetName().c_str()); - REPORT_INPUT_ERROR("E19999", "CreateOp failed, node_type = %s, node_name = %s", node.GetType().c_str(), node.GetName().c_str()); + GELOGE(FAILED, "Create OP failed ! node_type = %s, node_name = %s", node.GetType().c_str(), node.GetName().c_str()); + REPORT_INNER_ERROR("E19999", "CreateOp failed, node_type = %s, node_name = %s", node.GetType().c_str(), node.GetName().c_str()); return nullptr; }