> -----Original Message----- > From: Van Haaren, Harry > Sent: Thursday, October 8, 2020 9:34 AM > To: David Marchand <david.march...@redhat.com>; Richardson, Bruce > <bruce.richard...@intel.com>; Ananyev, Konstantin > <konstantin.anan...@intel.com>; Jerin Jacob Kollanukkaran > <jer...@marvell.com>; > Ruifeng Wang (Arm Technology China) <ruifeng.w...@arm.com>; Jan Viktorin > <vikto...@rehivetech.com>; David Christensen <d...@linux.vnet.ibm.com>; > Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > Cc: dev <dev@dpdk.org>; Pavan Nikhilesh <pbhagavat...@marvell.com> > Subject: RE: [dpdk-dev] [PATCH v2] eal: add new prefetch write variants > > > -----Original Message----- > > From: David Marchand <david.march...@redhat.com> > > Sent: Thursday, October 8, 2020 8:43 AM > > To: Van Haaren, Harry <harry.van.haa...@intel.com>; Richardson, Bruce > > <bruce.richard...@intel.com>; Ananyev, Konstantin > > <konstantin.anan...@intel.com>; Jerin Jacob Kollanukkaran > <jer...@marvell.com>; > > Ruifeng Wang (Arm Technology China) <ruifeng.w...@arm.com>; Jan Viktorin > > <vikto...@rehivetech.com>; David Christensen <d...@linux.vnet.ibm.com>; > > Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > > Cc: dev <dev@dpdk.org>; Pavan Nikhilesh <pbhagavat...@marvell.com> > > Subject: Re: [dpdk-dev] [PATCH v2] eal: add new prefetch write variants > > > > On Mon, Sep 14, 2020 at 5:09 PM Harry van Haaren > > <harry.van.haa...@intel.com> 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 <harry.van.haa...@intel.com> > > > > 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!