> -----Original Message----- > From: Van Haaren, Harry > Sent: Thursday, October 8, 2020 9:34 AM > To: David Marchand <[email protected]>; Richardson, Bruce > <[email protected]>; Ananyev, Konstantin > <[email protected]>; Jerin Jacob Kollanukkaran > <[email protected]>; > Ruifeng Wang (Arm Technology China) <[email protected]>; Jan Viktorin > <[email protected]>; David Christensen <[email protected]>; > Honnappa Nagarahalli <[email protected]> > Cc: dev <[email protected]>; Pavan Nikhilesh <[email protected]> > Subject: RE: [dpdk-dev] [PATCH v2] eal: add new prefetch write variants > > > -----Original Message----- > > From: David Marchand <[email protected]> > > Sent: Thursday, October 8, 2020 8:43 AM > > To: Van Haaren, Harry <[email protected]>; Richardson, Bruce > > <[email protected]>; Ananyev, Konstantin > > <[email protected]>; Jerin Jacob Kollanukkaran > <[email protected]>; > > Ruifeng Wang (Arm Technology China) <[email protected]>; Jan Viktorin > > <[email protected]>; David Christensen <[email protected]>; > > Honnappa Nagarahalli <[email protected]> > > Cc: dev <[email protected]>; Pavan Nikhilesh <[email protected]> > > Subject: Re: [dpdk-dev] [PATCH v2] eal: add new prefetch write variants > > > > On Mon, Sep 14, 2020 at 5:09 PM Harry van Haaren > > <[email protected]> wrote: > > > > > > This commit adds a new rte_prefetch0_write() variants, suggesting to the > > > compiler to use a prefetch instruction with intention to write. As a > > > compiler builtin, the compiler can choose based on compilation target > > > what the best implementation for this instruction is. > > > > > > Signed-off-by: Harry van Haaren <[email protected]> > > > > No review, copying arch maintainers. > > > > -- > > David Marchand > > Hey All, > > Actually Pavan provided some feedback to add prefetch1 and prefetch2 variants: > https://mails.dpdk.org/archives/dev/2020-September/thread.html#180501 > > I can push a v3 with those changes later today, will CC folks here. > > Thanks, -Harry
... apologies - David you're right - the v2 on list already has those changes: http://patches.dpdk.org/patch/77632/ reviews welcome!
