Hi Thomas, > -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Monday, July 1, 2019 3:43 PM > To: Gavin Hu (Arm Technology China) <[email protected]> > Cc: Stephen Hemminger <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; Honnappa > Nagarahalli <[email protected]>; nd <[email protected]>; > [email protected] > Subject: Re: [dpdk-dev] [RFC 1/5] eal: add the APIs to wait until equal > > 01/07/2019 09:16, Gavin Hu (Arm Technology China): > > From: Stephen Hemminger <[email protected]> > > > Gavin Hu <[email protected]> wrote: > > > > > > > +#ifdef RTE_USE_WFE > > > > +#define rte_wait_until_equal_relaxed(addr, expected) do {\ > [...] > > > That is a hot mess. > > > Macro's are harder to maintain and offer no benefit over inline functions. > > > > During our internal review, I ever used C11 _Generic to generalize the API > to take different types of arguments. > > Gavin, the question is about macros versus functions. > Please, could you convert it into an inline function? Sure, I will do it in next version.
Re: [dpdk-dev] [RFC 1/5] eal: add the APIs to wait until equal
Gavin Hu (Arm Technology China) Tue, 02 Jul 2019 07:18:20 -0700
- [dpdk-dev] [RFC 0/5] use WFE for locks and... Gavin Hu
- [dpdk-dev] [RFC 2/5] ticketlock: use ... Gavin Hu
- [dpdk-dev] [RFC 3/5] ring: use wfe to... Gavin Hu
- [dpdk-dev] [RFC 1/5] eal: add the API... Gavin Hu
- Re: [dpdk-dev] [RFC 1/5] eal: add... Stephen Hemminger
- Re: [dpdk-dev] [RFC 1/5] eal:... Gavin Hu (Arm Technology China)
- Re: [dpdk-dev] [RFC 1/5] ... Thomas Monjalon
- Re: [dpdk-dev] [RFC ... Gavin Hu (Arm Technology China)
- Re: [dpdk-dev] [RFC 1/5] eal: add... Pavan Nikhilesh Bhagavatula
- Re: [dpdk-dev] [RFC 1/5] eal:... Gavin Hu (Arm Technology China)
- [dpdk-dev] [RFC 4/5] spinlock: use wf... Gavin Hu
- [dpdk-dev] [RFC 5/5] config: add WFE ... Gavin Hu
- Re: [dpdk-dev] [RFC 0/5] use WFE for ... Stephen Hemminger
- Re: [dpdk-dev] [RFC 0/5] use WFE ... Gavin Hu (Arm Technology China)
- [dpdk-dev] [RFC v2 0/5] use WFE for l... Gavin Hu
- [dpdk-dev] [RFC v2 1/5] eal: add the ... Gavin Hu
- Re: [dpdk-dev] [EXT] [RFC v2 1/5]... Pavan Nikhilesh Bhagavatula
- [dpdk-dev] [RFC v2 4/5] spinlock: use... Gavin Hu

