>-----Original Message----- >From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of gavin hu >Sent: Wednesday, January 9, 2019 3:32 AM >To: dev@dpdk.org >Cc: n...@arm.com; tho...@monjalon.net; jer...@marvell.com; >hemant.agra...@nxp.com; honnappa.nagaraha...@arm.com; >gavin...@arm.com; olivier.m...@6wind.com; Richardson, Bruce ><bruce.richard...@intel.com>; sta...@dpdk.org >Subject: [dpdk-dev] [PATCH v6 3/3] hash: flush the rings instead of dequeuing >one by one > >From: Gavin Hu <gavin...@arm.com> > >Within rte_hash_reset, calling a while loop to dequeue one by >one from the ring, while not using them at all, is wasting cycles, >The patch just flush the ring by resetting the indices can save cpu >cycles. > >Fixes: b26473ff8f4a ("hash: add reset function") >Fixes: 75706568a7eb ("hash: add extendable bucket feature") >Cc: sta...@dpdk.org [Wang, Yipeng] I don't think it is a bug fix applicable to previous stable versions. > >Signed-off-by: Gavin Hu <gavin...@arm.com> >Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
[Wang, Yipeng] Otherwise the hash table change looks good to me and unit test passed on my setup. Acked-by: Yipeng Wang <yipeng1.w...@intel.com>
