@@ -60,7 +60,7 @@ Status InsertReshapeIfNeed(const NodePtr &node) { | |||||
node->GetName().c_str(), src_anchor->GetIdx(), dst_node->GetName().c_str(), dst_anchor->GetIdx()); | node->GetName().c_str(), src_anchor->GetIdx(), dst_node->GetName().c_str(), dst_anchor->GetIdx()); | ||||
GE_CHECK_NOTNULL(dst_node); | GE_CHECK_NOTNULL(dst_node); | ||||
GE_CHECK_NOTNULL(dst_node->GetOpDesc()); | GE_CHECK_NOTNULL(dst_node->GetOpDesc()); | ||||
auto dst_tensor = dst_node->GetOpDesc()->GetInputDescPtr(dst_anchor->GetIdx()); | |||||
auto dst_tensor = dst_node->GetOpDesc()->MutableInputDesc(dst_anchor->GetIdx()); | |||||
GE_CHECK_NOTNULL(dst_tensor); | GE_CHECK_NOTNULL(dst_tensor); | ||||
bool is_dynamic = false; | bool is_dynamic = false; | ||||
const auto &src_tensor_dims = src_tensor->GetShape().GetDims(); | const auto &src_tensor_dims = src_tensor->GetShape().GetDims(); | ||||
@@ -71,6 +71,12 @@ Status InsertReshapeIfNeed(const NodePtr &node) { | |||||
dst_node->GetName().c_str()); | dst_node->GetName().c_str()); | ||||
is_dynamic = true; | is_dynamic = true; | ||||
} | } | ||||
if (dst_node->GetType() == NETOUTPUT && is_dynamic) { | |||||
// NetOutput shape must be continuous when dynamic shape. | |||||
// Otherwise, there may be an error waiting for the shape refresh to time out during execution. | |||||
dst_tensor->SetShape(src_tensor->GetShape()); | |||||
continue; | |||||
} | |||||
bool is_need_insert_reshape = src_tensor_dims != dst_tensor_dims && | bool is_need_insert_reshape = src_tensor_dims != dst_tensor_dims && | ||||
!is_dynamic; | !is_dynamic; | ||||
if (is_need_insert_reshape) { | if (is_need_insert_reshape) { | ||||
@@ -354,6 +354,8 @@ Status AiCoreOpTask::PrepareWithShape(TaskContext &context) { | |||||
Status AiCoreOpTask::UpdateTilingInfo(TaskContext &context) { | Status AiCoreOpTask::UpdateTilingInfo(TaskContext &context) { | ||||
auto node = context.GetNodeItem().node; | auto node = context.GetNodeItem().node; | ||||
GE_CHECK_NOTNULL(node); | GE_CHECK_NOTNULL(node); | ||||
auto op_desc = node->GetOpDesc(); | |||||
GE_CHECK_NOTNULL(op_desc); | |||||
GELOGD("[%s] Start to update tiling info for task: [%s]", node->GetName().c_str(), stub_name_.c_str()); | GELOGD("[%s] Start to update tiling info for task: [%s]", node->GetName().c_str(), stub_name_.c_str()); | ||||
OpRunInfo tiling_info; | OpRunInfo tiling_info; | ||||
@@ -368,16 +370,14 @@ Status AiCoreOpTask::UpdateTilingInfo(TaskContext &context) { | |||||
// update op args by tiling info | // update op args by tiling info | ||||
block_dim_ = static_cast<uint32_t>(tiling_info.block_dim); | block_dim_ = static_cast<uint32_t>(tiling_info.block_dim); | ||||
op_desc->SetWorkspaceBytes(tiling_info.workspaces); | |||||
clear_atomic_ = tiling_info.clear_atomic; | clear_atomic_ = tiling_info.clear_atomic; | ||||
tiling_data_ = tiling_info.tiling_data.str(); | tiling_data_ = tiling_info.tiling_data.str(); | ||||
tiling_key_ = tiling_info.tiling_key; | tiling_key_ = tiling_info.tiling_key; | ||||
GELOGD("Successfully getting [tiling_key] : %u", tiling_key_); | GELOGD("Successfully getting [tiling_key] : %u", tiling_key_); | ||||
auto op_desc = node->GetOpDesc(); | |||||
GE_CHECK_NOTNULL(op_desc); | |||||
op_desc->SetWorkspaceBytes(tiling_info.workspaces); | |||||
if (tiling_data_.empty()) { | if (tiling_data_.empty()) { | ||||
GELOGD("[%s] Tiling data is empty.", op_desc->GetName().c_str()); | |||||
GELOGD("[%s] Tiling data is empty.", op_desc->GsetName().c_str()); | |||||
return SUCCESS; | return SUCCESS; | ||||
} | } | ||||
if (tiling_buffer_ == nullptr) { | if (tiling_buffer_ == nullptr) { | ||||
@@ -42,8 +42,8 @@ ut::GraphBuilder Graph1Builder() { | |||||
auto var1 = builder.AddNode("var1", "Variable", 0, 1, FORMAT_ND, DT_FLOAT, {-1}); | auto var1 = builder.AddNode("var1", "Variable", 0, 1, FORMAT_ND, DT_FLOAT, {-1}); | ||||
auto const1 = builder.AddNode("const1", "Const", 0, 1, FORMAT_ND, DT_FLOAT, {1, 1, 224, 224}); | auto const1 = builder.AddNode("const1", "Const", 0, 1, FORMAT_ND, DT_FLOAT, {1, 1, 224, 224}); | ||||
auto transdata2 = builder.AddNode("transdata2", "Transdata", 1, 1, FORMAT_ND, DT_FLOAT, {224, 224}); | auto transdata2 = builder.AddNode("transdata2", "Transdata", 1, 1, FORMAT_ND, DT_FLOAT, {224, 224}); | ||||
auto transdata1 = builder.AddNode("transdata1", "Transdata", 1, 1, FORMAT_ND, DT_FLOAT, {224, 224}); | |||||
auto netoutput1 = builder.AddNode("netoutput1", "Netoutput", 2, 0); | |||||
auto transdata1 = builder.AddNode("transdata1", "Transdata", 1, 1, FORMAT_ND, DT_FLOAT, {-1, 224}); | |||||
auto netoutput1 = builder.AddNode("netoutput1", "NetOutput", 2, 0); | |||||
builder.AddDataEdge(var1, 0, transdata1, 0); | builder.AddDataEdge(var1, 0, transdata1, 0); | ||||
builder.AddDataEdge(const1, 0, transdata2, 0); | builder.AddDataEdge(const1, 0, transdata2, 0); | ||||
@@ -58,10 +58,10 @@ TEST_F(UtestReshapeRecoveryPass, reshape_recovery_with_dynamic_shape) { | |||||
auto builder = Graph1Builder(); | auto builder = Graph1Builder(); | ||||
auto graph = builder.GetGraph(); | auto graph = builder.GetGraph(); | ||||
ReshapeRecoveryPass reshape_recovery_pass; | ReshapeRecoveryPass reshape_recovery_pass; | ||||
EXPECT_EQ(graph->GetDirectNodesSize(),5); | |||||
EXPECT_EQ(graph->GetDirectNodesSize(), 5); | |||||
Status ret = reshape_recovery_pass.Run(graph); | Status ret = reshape_recovery_pass.Run(graph); | ||||
EXPECT_EQ(ret, SUCCESS); | EXPECT_EQ(ret, SUCCESS); | ||||
EXPECT_EQ(graph->GetDirectNodesSize(),8); | |||||
EXPECT_EQ(graph->GetDirectNodesSize(), 7); | |||||
auto reshape1 = graph->FindNode("Reshape_ReshapeRecoveryPass_0"); | auto reshape1 = graph->FindNode("Reshape_ReshapeRecoveryPass_0"); | ||||
EXPECT_NE(reshape1, nullptr); | EXPECT_NE(reshape1, nullptr); | ||||