> -----Original Message----- > From: Jerin Jacob <jerinjac...@gmail.com> > Sent: Wednesday, June 14, 2023 12:09 AM > To: David Marchand <david.march...@redhat.com> > Cc: Yan, Zhirun <zhirun....@intel.com>; jer...@marvell.com; dev@dpdk.org; > kirankum...@marvell.com; ndabilpu...@marvell.com; > step...@networkplumber.org; pbhagavat...@marvell.com; Liang, Cunming > <cunming.li...@intel.com>; Wang, Haiyue <haiyue.w...@intel.com>; > mattias.ronnblom <mattias.ronnb...@ericsson.com> > Subject: Re: [PATCH v14 00/16] graph enhancement for multi-core dispatch > > On Tue, Jun 13, 2023 at 8:10 PM David Marchand > <david.march...@redhat.com> wrote: > > > > On Tue, Jun 13, 2023 at 4:12 PM Zhirun Yan <zhirun....@intel.com> wrote: > > > > > > V14: > > > Rebase to main and fix build issues.(align header name in patch > > > 01,02) > > > > And, again, compilation is broken between some patches. > > Same here. > > Applying: graph: introduce stream moving cross cores > > real 0m17.361s > user 1m39.631s > sys 0m34.884s > HEAD is now at 050de60d8a version: 23.07-rc1 > meson: build failed > > @Zhirun Yan You may use the following script to test the compilation per > patch > and additional build sanity before posting the patch to mailing list. > https://github.com/jerinjacobk/config/blob/master/scripts/build_each_patch.sh > > example: > git pw series apply 28485 && rm -rf /tmp/k/* && git format-patch > HEAD~16 -o /tmp/k/ && git reset --hard HEAD~1 </path/to/build_each_patch.sh > /tmp/k/ > Thanks for the tools. I will double check the patch series.
> > [1202/3000] Compiling C object lib/librte_graph.a.p/graph_graph.c.o > FAILED: lib/librte_graph.a.p/graph_graph.c.o > ccache gcc -Ilib/librte_graph.a.p -Ilib -I../lib -Ilib/graph -I../lib/graph > -I. -I.. - > Iconfig -I../config -Ilib/eal/include -I../lib/eal/include > -Ilib/eal/linux/include - > I../lib/eal/linux/include -Ilib/eal/x86/include -I../lib/eal/x86/inclu de - > Ilib/eal/common -I../lib/eal/common -Ilib/eal -I../lib/eal -Ilib/kvargs - > I../lib/kvargs -Ilib/metrics -I../lib/metrics -Ilib/telemetry > -I../lib/telemetry - > Ilib/pcapng -I../lib/pcapng -Ilib/ethdev -I../lib/ethdev -Ilib/net > -I../lib/net - > Ilib/mbuf -I../lib/mbuf -Ilib/mempool -I../lib/mempool -Ilib/ring > -I../lib/ring - > Ilib/meter -I../lib/meter -fdiagnostics-color=always > -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Werror -O2 -g -include > rte_config.h -Wcast-qual -Wdepr ecated -Wformat -Wformat-nonliteral - > Wformat-security -Wmissing-declarations -Wmissing-prototypes -Wnested- > externs -Wold-style-definition -Wpointer-arith -Wsign-compare -Wstrict- > prototypes -Wundef -Wwrite-strings -Wno-address-of-packed-memb er -Wno- > packed-not-aligned -Wno-missing-field-initializers -Wno-zero-length-bounds - > D_GNU_SOURCE -fPIC -march=native -DALLOW_EXPERIMENTAL_API - > DALLOW_INTERNAL_API -Wno-format-truncation - > DRTE_LOG_DEFAULT_LOGTYPE=lib.graph -MD -MQ lib/librte > _graph.a.p/graph_graph.c.o -MF lib/librte_graph.a.p/graph_graph.c.o.d > -o lib/librte_graph.a.p/graph_graph.c.o -c ../lib/graph/graph.c > ../lib/graph/graph.c: In function ‘graph_clone’: > ../lib/graph/graph.c:476:83: error: unused parameter ‘prm’ > [-Werror=unused-parameter] > 476 | graph_clone(struct graph *parent_graph, const char *name, struct > rte_graph_param *prm) > | > ~~~~~~~~~~~~~~~~~~~~~~~~^~~ > cc1: all warnings being treated as errors > > > > > > > > > -- > > David Marchand > >
