On 10/9/19 3:38 PM, Marvin Liu wrote: > Batch enqueue function will first check whether descriptors are cache > aligned. It will also check prerequisites in the beginning. Batch > enqueue function not support chained mbufs, single packet enqueue > function will handle it. > > Signed-off-by: Marvin Liu <yong....@intel.com> > Thinking again about this patch and series in general... So this series improves performance by 40% in cases where: - descriptors are cache aligned - single mbuf But my understanding is that it will cause performance regression for the other cases, which may not be that uncommon, no? Do you have some number about the performance impact on these other cases? Thanks, Maxime
- [dpdk-dev] [PATCH v3 11/15] vhost: add batch and single ze... Marvin Liu
- [dpdk-dev] [PATCH v3 15/15] vhost: optimize packed ring de... Marvin Liu
- [dpdk-dev] [PATCH v3 14/15] vhost: check whether disable s... Marvin Liu
- [dpdk-dev] [PATCH v4 00/14] vhost packed ring performance ... Marvin Liu
- [dpdk-dev] [PATCH v4 02/14] vhost: unify unroll pragm... Marvin Liu
- Re: [dpdk-dev] [PATCH v4 02/14] vhost: unify unro... Maxime Coquelin
- [dpdk-dev] [PATCH v4 03/14] vhost: add batch enqueue ... Marvin Liu
- Re: [dpdk-dev] [PATCH v4 03/14] vhost: add batch ... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v4 03/14] vhost: add batch ... Maxime Coquelin
- [dpdk-dev] [PATCH v4 04/14] vhost: add single packet ... Marvin Liu
- Re: [dpdk-dev] [PATCH v4 04/14] vhost: add single... Maxime Coquelin
- [dpdk-dev] [PATCH v4 01/14] vhost: add single packet ... Marvin Liu
- Re: [dpdk-dev] [PATCH v4 01/14] vhost: add single... Maxime Coquelin
- [dpdk-dev] [PATCH v4 06/14] vhost: flush vhost enqueu... Marvin Liu
- [dpdk-dev] [PATCH v4 05/14] vhost: add batch dequeue ... Marvin Liu
- [dpdk-dev] [PATCH v4 08/14] vhost: buffer vhost deque... Marvin Liu
- [dpdk-dev] [PATCH v4 10/14] vhost: optimize enqueue f... Marvin Liu
- [dpdk-dev] [PATCH v4 07/14] vhost: add flush function... Marvin Liu