> -----Original Message----- > From: Jerin Jacob <jerinjac...@gmail.com> > Sent: Monday, February 20, 2023 9:38 PM > To: Yan, Zhirun <zhirun....@intel.com> > Cc: dev@dpdk.org; jer...@marvell.com; kirankum...@marvell.com; > ndabilpu...@marvell.com; Liang, Cunming <cunming.li...@intel.com>; Wang, > Haiyue <haiyue.w...@intel.com> > Subject: Re: [PATCH v1 01/13] graph: split graph worker into common and > default model > > On Thu, Nov 17, 2022 at 10:39 AM Zhirun Yan <zhirun....@intel.com> wrote: > > > > To support multiple graph worker model, split graph into common and > > default. Naming the current walk function as rte_graph_model_rtc cause > > the default model is RTC(Run-to-completion). > > There CI issues with this series. Please check > https://patches.dpdk.org/project/dpdk/patch/20221117050926.136974-2- > zhirun....@intel.com/ > # Please make sure each patch builds with devtools/test-meson-builds.sh # > Please make sure each patch dont any issue with app/test/test_graph.c test # > This series dont have perf issues with app/test/test_graph_perf.c # Both RTC > and new mode runs with l3fwd_graph without any performance regression # > Please Introduce model concept in documentation at > doc/guides/prog_guide/graph_lib.rst and details for this generic mode. > > Also update the maintainers files for new model files. > Yes, I will fix the CI issues and update doc, maintainers files in next version. > > > > Signed-off-by: Haiyue Wang <haiyue.w...@intel.com> > > Signed-off-by: Cunming Liang <cunming.li...@intel.com> > > Signed-off-by: Zhirun Yan <zhirun....@intel.com> > > --- > > lib/graph/rte_graph_model_rtc.h | 57 ++++ > > lib/graph/rte_graph_worker.h | 498 +--------------------------- > > lib/graph/rte_graph_worker_common.h | 456 +++++++++++++++++++++++++ > > > Use git mv to avoid loosing history and reduce the diff.
Actually, it is file A -> file B and file C, I will break it to 2 patch to keep the log history. Got it, Thanks.
