From 22c3cdb43f12fed549fdb6d4ce54b285172123c8 Mon Sep 17 00:00:00 2001 From: Leo Chen Date: Fri, 16 Sep 2022 16:06:05 +0800 Subject: [PATCH] add interpretercore for jit engine (#46092) * add interpretercore for jit engine * add ut --- paddle/fluid/jit/CMakeLists.txt | 13 ++- paddle/fluid/jit/engine/CMakeLists.txt | 5 + paddle/fluid/jit/engine/interpreter_engine.cc | 102 ++++++++++++++++++ paddle/fluid/jit/engine/interpreter_engine.h | 61 +++++++++++ paddle/fluid/jit/layer_test.cc | 1 + paddle/fluid/jit/serializer.cc | 7 ++ paddle/fluid/platform/flags.cc | 1 + 7 files changed, 189 insertions(+), 1 deletion(-) create mode 100644 paddle/fluid/jit/engine/interpreter_engine.cc create mode 100644 paddle/fluid/jit/engine/interpreter_engine.h diff --git a/paddle/fluid/jit/CMakeLists.txt b/paddle/fluid/jit/CMakeLists.txt index 565bd670b9..872845d079 100644 --- a/paddle/fluid/jit/CMakeLists.txt +++ b/paddle/fluid/jit/CMakeLists.txt @@ -34,7 +34,8 @@ cc_library( cc_library( jit_function SRCS function.cc - DEPS jit_function_utils jit_executor_engine jit_pe_engine) + DEPS jit_function_utils jit_executor_engine jit_pe_engine + jit_interpreter_engine) cc_library( jit_layer @@ -46,6 +47,7 @@ cc_library( jit_function_schema jit_executor_engine jit_pe_engine + jit_interpreter_engine jit_function) if(WITH_TESTING AND NOT WIN32) @@ -65,10 +67,19 @@ if(WITH_TESTING AND NOT WIN32) feed_op fetch_op scale_op + transfer_layout_op jit_layer) cc_test( layer_test SRCS layer_test.cc DEPS ${JIT_DEPS}) add_dependencies(layer_test jit_download_program) + + cc_test( + layer_test_new + SRCS layer_test.cc + DEPS ${JIT_DEPS}) + add_dependencies(layer_test_new jit_download_program) + set_tests_properties(layer_test_new PROPERTIES ENVIRONMENT + "FLAGS_jit_engine_type=New") endif() diff --git a/paddle/fluid/jit/engine/CMakeLists.txt b/paddle/fluid/jit/engine/CMakeLists.txt index 92a1f9582c..5626e9eb1f 100644 --- a/paddle/fluid/jit/engine/CMakeLists.txt +++ b/paddle/fluid/jit/engine/CMakeLists.txt @@ -7,3 +7,8 @@ cc_library( jit_pe_engine SRCS pe_engine.cc DEPS parallel_executor) + +cc_library( + jit_interpreter_engine + SRCS interpreter_engine.cc + DEPS standalone_executor) diff --git a/paddle/fluid/jit/engine/interpreter_engine.cc b/paddle/fluid/jit/engine/interpreter_engine.cc new file mode 100644 index 0000000000..410fd4dc01 --- /dev/null +++ b/paddle/fluid/jit/engine/interpreter_engine.cc @@ -0,0 +1,102 @@ +// Copyright (c) 2022 PaddlePaddle Authors. All Rights Reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +#include "paddle/fluid/jit/engine/interpreter_engine.h" + +#include "paddle/fluid/framework/block_desc.h" +#include "paddle/fluid/framework/ir/graph.h" +#include "paddle/fluid/framework/ir/graph_helper.h" +#include "paddle/fluid/framework/ir/pass.h" +#include "paddle/fluid/framework/new_executor/interpretercore.h" +#include "paddle/fluid/framework/program_desc.h" +#include "paddle/phi/core/enforce.h" + +namespace paddle { +namespace jit { + +InterpreterEngine::InterpreterEngine(const std::shared_ptr &info, + const VariableMap ¶ms_dict, + const phi::Place &place) + : info_(info), place_(place) { + info_->RemoveDescFeedFetch(); + PADDLE_ENFORCE_GT( + static_cast(info_->ProgramDesc().Block(0).OpSize()), + 0, + platform::errors::PreconditionNotMet( + "There is no operator in ProgramDesc.")); + utils::ShareParamsIntoScope(info_->ParamNames(), params_dict, &scope_); + VLOG(6) << framework::GenScopeTreeDebugInfo(&scope_); + CreateInterpreterCore(); +} + +void InterpreterEngine::CreateInterpreterCore() { + auto &program_desc = info_->ProgramDesc(); + + // apply inference pass + framework::ir::Graph graph{program_desc}; + auto pass = + framework::ir::PassRegistry::Instance().Get("delete_dropout_op_x_pass"); + pass->Apply(&graph); +#ifdef PADDLE_WITH_MKLDNN + auto mkldnn_pass = + framework::ir::PassRegistry::Instance().Get("mkldnn_placement_pass"); + mkldnn_pass->Set("mkldnn_enabled_op_types", + new std::unordered_set({})); + mkldnn_pass->Apply(&graph); +#endif + + GraphToProgram(graph, &converted_prog_, nullptr); + + auto in_names = info_->InputArgNames(); + auto out_names = info_->OutputArgNames(); + std::set skip_gc_vars; + skip_gc_vars.insert(in_names.begin(), in_names.end()); + skip_gc_vars.insert(out_names.begin(), out_names.end()); + + inner_interpreter_ = + std::make_shared(place_, + converted_prog_.Block(0), + /*skip_gc_vars=*/skip_gc_vars, + &scope_, + /*used_for_jit=*/true); +} + +std::vector InterpreterEngine::operator()( + const std::vector &inputs) { + auto dense_tensors = utils::ToDenseTensors(inputs); + return utils::ToTensors(this->operator()(dense_tensors)); +} + +std::vector InterpreterEngine::operator()( + const std::vector &inputs) { + utils::ShareIntoScope(info_->InputArgNames(), inputs, &scope_); + + // the latter can be moved to python side. + auto &feed_names = info_->InputArgNames(); + auto &fetch_names = info_->OutputArgNames(); + paddle::framework::FetchList outs = inner_interpreter_->Run(feed_names); + + std::vector outputs; + utils::FetchOuts(info_->OutputArgNames(), scope_, &outputs); + scope_.DropKids(); + + return outputs; +} + +const std::shared_ptr &InterpreterEngine::Info() const { + return info_; +} + +} // namespace jit +} // namespace paddle diff --git a/paddle/fluid/jit/engine/interpreter_engine.h b/paddle/fluid/jit/engine/interpreter_engine.h new file mode 100644 index 0000000000..8c7f43f297 --- /dev/null +++ b/paddle/fluid/jit/engine/interpreter_engine.h @@ -0,0 +1,61 @@ +// Copyright (c) 2022 PaddlePaddle Authors. All Rights Reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +#pragma once + +#include + +#include "paddle/fluid/framework/program_desc.h" +#include "paddle/fluid/framework/scope.h" + +#include "paddle/fluid/jit/engine/base_engine.h" +#include "paddle/fluid/jit/function_schema.h" +#include "paddle/fluid/jit/function_utils.h" + +namespace paddle { + +namespace framework { +class InterpreterCore; +} // namespace framework + +namespace jit { +using InterpreterCore = framework::InterpreterCore; +// using Graph = framework::ir::Graph; + +class InterpreterEngine : public BaseEngine { + public: + InterpreterEngine(const std::shared_ptr &info, + const VariableMap ¶ms_dict, + const phi::Place &place); + + ~InterpreterEngine() noexcept {} + + void CreateInterpreterCore(); + + std::vector operator()(const std::vector &inputs); + + std::vector operator()(const std::vector &inputs); + + const std::shared_ptr &Info() const; + + private: + std::shared_ptr info_; + framework::Scope scope_; + phi::Place place_; + std::shared_ptr inner_interpreter_; + framework::ProgramDesc converted_prog_; +}; + +} // namespace jit +} // namespace paddle diff --git a/paddle/fluid/jit/layer_test.cc b/paddle/fluid/jit/layer_test.cc index 360eebe8b4..7007693aa8 100644 --- a/paddle/fluid/jit/layer_test.cc +++ b/paddle/fluid/jit/layer_test.cc @@ -38,6 +38,7 @@ USE_OP_ITSELF(reduce_mean); USE_OP_ITSELF(feed); USE_OP_ITSELF(fetch); USE_OP_ITSELF(scale); +USE_OP_ITSELF(transfer_layout); PD_DECLARE_KERNEL(add, CPU, ALL_LAYOUT); PD_DECLARE_KERNEL(matmul, CPU, ALL_LAYOUT); diff --git a/paddle/fluid/jit/serializer.cc b/paddle/fluid/jit/serializer.cc index 65a39bc7f9..8e8bb370e8 100644 --- a/paddle/fluid/jit/serializer.cc +++ b/paddle/fluid/jit/serializer.cc @@ -21,6 +21,7 @@ #include "paddle/fluid/platform/device_context.h" #include "paddle/fluid/jit/engine/executor_engine.h" +#include "paddle/fluid/jit/engine/interpreter_engine.h" #include "paddle/fluid/jit/engine/pe_engine.h" #include "paddle/fluid/jit/layer.h" #include "paddle/fluid/jit/property.h" @@ -79,6 +80,12 @@ Layer Deserializer::operator()(const std::string& path, VLOG(3) << "Add function type: PEEngine. Function name: " << func_name; layer.SetEngine(func_name, utils::MakeEngine(info, params_dict, place)); + } else if (FLAGS_jit_engine_type == "New") { + VLOG(3) << "Add function type: InterpreterEngine. Function name: " + << func_name; + layer.SetEngine( + func_name, + utils::MakeEngine(info, params_dict, place)); } else { PD_THROW("Invalid JitLayer engine type."); } diff --git a/paddle/fluid/platform/flags.cc b/paddle/fluid/platform/flags.cc index dd66ab179b..534309942a 100644 --- a/paddle/fluid/platform/flags.cc +++ b/paddle/fluid/platform/flags.cc @@ -1011,6 +1011,7 @@ PADDLE_DEFINE_EXPORTED_bool( * Note: * FLAGS_jit_engine_type == Executor, using ExecutorEngine by default * FLAGS_jit_engine_type == PE, using PEEngine by default + * FLAGS_jit_engine_type == New, using InterpreterEngine by default */ PADDLE_DEFINE_EXPORTED_string(jit_engine_type, "PE", -- GitLab