On Wed, Oct 27, 2021 at 1:40 PM Feifei Wang <feifei.wa...@arm.com> wrote: > > Add new definitions for wait scheme, and apply this new definitions into > lib to replace rte_pause. > > v2: > 1. use macro to create new wait scheme (Stephen) > > v3: > 1. delete unnecessary bug fix in bpf (Konstantin) > > v4: > 1. put size into the macro body (Konstantin) > 2. replace assert with BUILD_BUG_ON (Stephen) > 3. delete unnecessary compiler barrier for bpf (Konstantin) > > v5: > 1. 'size' is not the parameter (Konstantin) > 2. put () around macro parameters (Konstantin) > 3. fix some original typo issue (Jerin) > 4. swap 'rte_wait_event' parameter location (Jerin) > 5. add new macro '__LOAD_EXC' > 6. delete 'undef' to prevent compilation warning
+ David, Konstantin, Stephen, Please make a practice to add exiting reviewers. undef the local marco may result in conflict with other libraries. Please add __RTE_ARM_ for existing macros (mark as internal) to fix the namespace if we are taking that path > > v6: > 1. fix patch style check warning > 2. delete 'bpf' patch due to 'const' limit > > Feifei Wang (4): > eal: add new definitions for wait scheme > eal: use wait event for read pflock > eal: use wait event scheme for mcslock > lib/distributor: use wait event scheme > > lib/distributor/rte_distributor_single.c | 10 +- > lib/eal/arm/include/rte_pause_64.h | 136 +++++++++++++---------- > lib/eal/include/generic/rte_mcslock.h | 9 +- > lib/eal/include/generic/rte_pause.h | 28 +++++ > lib/eal/include/generic/rte_pflock.h | 4 +- > 5 files changed, 119 insertions(+), 68 deletions(-) > > -- > 2.25.1 >
