>-----Original Message----- >From: dev [mailto:[email protected]] On Behalf Of gavin hu >Sent: Wednesday, January 9, 2019 3:32 AM >To: [email protected] >Cc: [email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; Richardson, Bruce ><[email protected]>; [email protected] >Subject: [dpdk-dev] [PATCH v6 3/3] hash: flush the rings instead of dequeuing >one by one > >From: Gavin Hu <[email protected]> > >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: [email protected] [Wang, Yipeng] I don't think it is a bug fix applicable to previous stable versions. > >Signed-off-by: Gavin Hu <[email protected]> >Reviewed-by: Honnappa Nagarahalli <[email protected]>
[Wang, Yipeng] Otherwise the hash table change looks good to me and unit test passed on my setup. Acked-by: Yipeng Wang <[email protected]>
