> -----Original Message----- > From: Thomas Monjalon <tho...@monjalon.net> > Sent: Thursday, October 15, 2020 6:03 AM > To: Liu, Yong <yong....@intel.com> > Cc: Richardson, Bruce <bruce.richard...@intel.com>; > step...@networkplumber.org; dev@dpdk.org; > david.march...@redhat.com; Yigit, Ferruh <ferruh.yi...@intel.com>; > maxime.coque...@redhat.com; honnappa.nagaraha...@arm.com; David > Christensen <d...@linux.vnet.ibm.com>; ruifeng.w...@arm.com > Subject: Re: [dpdk-dev] [PATCH v2] config: enable packet data prefetch > > 23/09/2020 03:51, Marvin Liu: > > Data prefetch instruction can preload data into cpu’s hierarchical > > cache before data access. Virtualized data paths like virtio utilized > > this feature for acceleration. Since most modern cpus have support > > prefetch function, we can enable packet data prefetch as default. > > > > Signed-off-by: Marvin Liu <yong....@intel.com> > > --- > > +#define RTE_PMD_PACKET_PREFETCH 1 > > We could also remove the related #ifdefs. > > What can be the drawback of always enable those prefetches? >
Hi Thomas, I think the potential drawback is that current prefetch location cannot guarantee the best performance across different platforms. Each developer has tuned the performance by adding prefetch instruction and verified the result on himself platform. So prefetch location is based on certain platform, also it will be hard for developer to compare the results across platforms. Thanks, Marvin
