From cd429605dae214127413d8cd879d9e8a9e65ca71 Mon Sep 17 00:00:00 2001 From: "liyihan2@huawei.com" Date: Tue, 9 Mar 2021 16:55:21 +0800 Subject: [PATCH] log optimize --- ge/common/cust_aicpu_kernel_store.cc | 2 +- .../format_transfers/format_transfer_c1hwncoc0_hwcn.cc | 6 +++--- ge/common/ge/op_tiling_manager.cc | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ge/common/cust_aicpu_kernel_store.cc b/ge/common/cust_aicpu_kernel_store.cc index 6d96eab4..1055989b 100755 --- a/ge/common/cust_aicpu_kernel_store.cc +++ b/ge/common/cust_aicpu_kernel_store.cc @@ -31,7 +31,7 @@ void CustAICPUKernelStore::LoadCustAICPUKernelBinToOpDesc(const std::shared_ptr< if (kernel_bin != nullptr) { GE_IF_BOOL_EXEC(!op_desc->SetExtAttr(ge::OP_EXTATTR_CUSTAICPU_KERNEL, kernel_bin), GELOGW("LoadKernelCustAICPUBinToOpDesc: SetExtAttr for kernel_bin failed");) - GELOGI("Load cust aicpu kernel:%s, %zu.", kernel_bin->GetName().c_str(), kernel_bin->GetBinDataSize()); + GELOGI("Load cust aicpu kernel:%s, %zu", kernel_bin->GetName().c_str(), kernel_bin->GetBinDataSize()); } } GELOGD("LoadCustAICPUKernelBinToOpDesc success."); diff --git a/ge/common/formats/format_transfers/format_transfer_c1hwncoc0_hwcn.cc b/ge/common/formats/format_transfers/format_transfer_c1hwncoc0_hwcn.cc index 0cb581d7..56b40669 100644 --- a/ge/common/formats/format_transfers/format_transfer_c1hwncoc0_hwcn.cc +++ b/ge/common/formats/format_transfers/format_transfer_c1hwncoc0_hwcn.cc @@ -49,7 +49,7 @@ Status CheckArgsForC1hwncoc0ToHwcn(const TransArgs &args) { return UNSUPPORTED; } if (!CheckShapeValid(src_shape, kC1hwncoc0DimsNum)) { - GELOGE(PARAM_INVALID, "Failed to check src shape %s", ShapeToString(src_shape).c_str()); + GELOGE(PARAM_INVALID, "Failed to check src shape %s.", ShapeToString(src_shape).c_str()); return PARAM_INVALID; } if (!CheckShapeValid(dst_shape, kHwcnDimsNum)) { @@ -147,7 +147,7 @@ Status FormatTransferC1hwncoc0Hwcn::TransFormat(const TransArgs &args, TransResu ShapeToString(args.dst_shape).c_str(), ShapeToString(args.src_shape).c_str()); return PARAM_INVALID; } - GELOGD("Begin to trans format from C1HWNCoC0 to HWCN, src shape %s, data type %s, dst shape %s, memory size %ld", + GELOGD("Begin to trans format from C1HWNCoC0 to HWCN, src shape %s, data type %s, dst shape %s, memory size %ld.", ShapeToString(args.src_shape).c_str(), TypeUtils::DataTypeToSerialString(args.src_data_type).c_str(), ShapeToString(args.dst_shape).c_str(), total_size); if (GetDstDataAfterTrans(args, result, size, total_size) != SUCCESS) { @@ -161,7 +161,7 @@ Status FormatTransferC1hwncoc0Hwcn::TransFormat(const TransArgs &args, TransResu Status FormatTransferC1hwncoc0Hwcn::TransShape(Format src_format, const std::vector &src_shape, DataType data_type, Format dst_format, std::vector &dst_shape) { - GELOGD("The shape derivation from C1HWNCoC0 to HWCN is not unique. Trans shape in this direction is not supported"); + GELOGD("The shape derivation from C1HWNCoC0 to HWCN is not unique. Trans shape in this direction is not supported."); return ACL_ERROR_GE_TRANSSHAPE_FORMAT_INVALID; } diff --git a/ge/common/ge/op_tiling_manager.cc b/ge/common/ge/op_tiling_manager.cc index e374dcf4..db959368 100644 --- a/ge/common/ge/op_tiling_manager.cc +++ b/ge/common/ge/op_tiling_manager.cc @@ -33,7 +33,7 @@ void OpTilingManager::ClearHandles() noexcept { if (mmDlclose(handle.second) != 0) { const char *error = mmDlerror(); GE_IF_BOOL_EXEC(error == nullptr, error = ""); - GELOGE(FAILED, "Failed to close handle of %s: %s.", handle.first.c_str(), error); + GELOGE(FAILED, "Failed to close handle of %s: %s", handle.first.c_str(), error); } } handles_.clear();