On 12/12/2023 11:40 AM, Morten Brørup wrote: >> From: Vipin Varghese [mailto:vipin.vargh...@amd.com] >> Sent: Tuesday, 12 December 2023 11.38 >> >> Replace pktmbuf pool with mempool, this allows increase in MOPS >> especially in lower buffer size. Using Mempool, allows to reduce >> the extra CPU cycles. > > I get the point of this change: It tests the performance of copying raw > memory objects using respectively rte_memcpy and DMA, without the mbuf > indirection overhead. > > However, I still consider the existing test relevant: The performance of > copying packets using respectively rte_memcpy and DMA. >
This is DMA performance test application and packets are not used, using pktmbuf just introduces overhead to the main focus of the application. I am not sure if pktmuf selected intentionally for this test application, but I assume it is there because of historical reasons. > So, instead of replacing mbufs with mempool objects, please add new test > variants using mempool objects. >
