> -----Original Message----- > From: David Marchand <[email protected]> > Sent: Thursday, March 25, 2021 10:58 PM > To: Honnappa Nagarahalli <[email protected]> > Cc: dev <[email protected]>; nd <[email protected]>; [email protected]; > [email protected]; Ruifeng Wang <[email protected]>; Phil Yang > <[email protected]>; Joyce Kong <[email protected]>; Ananyev, > Konstantin <[email protected]> > Subject: Re: [dpdk-dev] [PATCH v2 0/2] Use WFE for spinlock and ring > > Hello, > > On Fri, Sep 25, 2020 at 4:30 PM David Marchand > <[email protected]> wrote: > > > > Hello Honnappa, > > > > On Sun, May 3, 2020 at 11:32 PM Honnappa Nagarahalli > > <[email protected]> wrote: > > > > > > <snip> > > > > > > > Subject: Re: [PATCH v2 0/2] Use WFE for spinlock and ring > > > > > > > > On Sun, Apr 26, 2020 at 10:39 AM Gavin Hu <[email protected]> wrote: > > > > > > > > > > The rte_wait_until_equal_xxx APIs abstract the functionality of > > > > > 'polling for a memory location to become equal to a given value'[1]. > > > > > > > > > > Use the API for the rte spinlock and ring implementations. > > > > > > > > > > [1] http://patches.dpdk.org/cover/62703/ > > > > > > > > > > Gavin Hu (2): > > > > > spinlock: use wfe to reduce contention on aarch64 > > > > > ring: use wfe to wait for ring tail update on aarch64 > > > > > > > > This would result in rte_ring and rte_spinlock APIs becoming > > > > experimental and this breaks compilation for external applications > > > > using stable ring and spinlock APIs. > > > > IIRC, it was the reason why these patches were dropped with the > > > > introduction of the rte_wait_until_equal_* API. > > > Agreed, the rte_ring new sync modes are resulting in different use cases > for these APIs. We need to take a relook at the APIs. > > > > Will we reconsider this series now that the wait until equal is going > > stable? > > Any update? > I suppose this would need some rebasing after the ring library changes.
Sorry for the late response. This series has been rebased and v3 posted. I have problem to mark v2 as superseded. Thank you. > > Thanks. > > > -- > David Marchand
