> -----Original Message----- > From: Jerin Jacob Kollanukkaran <jer...@marvell.com> > Sent: Monday, July 22, 2019 10:15 PM > To: Phil Yang (Arm Technology China) <phil.y...@arm.com>; dev@dpdk.org > Cc: tho...@monjalon.net; gage.e...@intel.com; > hemant.agra...@nxp.com; Honnappa Nagarahalli > <honnappa.nagaraha...@arm.com>; Gavin Hu (Arm Technology China) > <gavin...@arm.com>; nd <n...@arm.com> > Subject: RE: [EXT] [PATCH v5 3/3] eal/stack: enable lock-free stack for > aarch64 > > > -----Original Message----- > > From: Phil Yang <phil.y...@arm.com> > > Sent: Monday, July 22, 2019 6:36 PM > > To: dev@dpdk.org > > Cc: tho...@monjalon.net; Jerin Jacob Kollanukkaran > <jer...@marvell.com>; > > gage.e...@intel.com; hemant.agra...@nxp.com; > > honnappa.nagaraha...@arm.com; gavin...@arm.com; n...@arm.com > > Subject: [EXT] [PATCH v5 3/3] eal/stack: enable lock-free stack for aarch64 > > > > Enable both c11 atomic and non c11 atomic lock-free stack for aarch64. > > > > Introduced a new header, rte_stack_lf_stubs.h, to reduce the ifdef clutter > > across generic and c11 files. The rte_stack_lf_stubs.h contains stub > > implementations of __rte_stack_lf_count, __rte_stack_lf_push_elems > and > > __rte_stack_lf_pop_elems. > > > > Suggested-by: Gage Eads <gage.e...@intel.com> > > Suggested-by: Jerin Jacob <jer...@marvell.com> > > Signed-off-by: Phil Yang <phil.y...@arm.com> > > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > > Tested-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > > # Build issue with 32 bit build, Looks like new header file missing in > SYMLINK- > $(CONFIG_RTE_LIBRTE_STACK)-include > > In file included from /home/jerin/dpdk.org/build/include/rte_stack.h:98, > from > /home/jerin/dpdk.org/drivers/mempool/stack/rte_mempool_stack.c:7: > /home/jerin/dpdk.org/build/include/rte_stack_lf.h:9:10: fatal error: > rte_stack_lf_stubs.h: No such file or directory > 9 | #include "rte_stack_lf_stubs.h" > | ^~~~~~~~~~~~~~~~~~~~~~ >
Yes. Thanks. Will fix it in next version.
