diff mbox series

[v2,15/15] vhost: increase number of async IO vectors

Message ID	20211026162904.482987-16-maxime.coquelin@redhat.com (mailing list archive)
State	Accepted, archived
Delegated to:	Maxime Coquelin
Headers	From: Maxime Coquelin <maxime.coquelin@redhat.com> To: dev@dpdk.org, chenbo.xia@intel.com, jiayu.hu@intel.com, yuanx.wang@intel.com, wenwux.ma@intel.com, bruce.richardson@intel.com, john.mcnamara@intel.com Cc: Maxime Coquelin <maxime.coquelin@redhat.com> Date: Tue, 26 Oct 2021 18:29:04 +0200 Message-Id: <20211026162904.482987-16-maxime.coquelin@redhat.com> In-Reply-To: <20211026162904.482987-1-maxime.coquelin@redhat.com> References: <20211026162904.482987-1-maxime.coquelin@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" Subject: [dpdk-dev] [PATCH v2 15/15] vhost: increase number of async IO vectors Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	vhost: clean-up and simplify async implementation \| [v2,00/15] vhost: clean-up and simplify async implementation [v2,01/15] vhost: move async data in a dedicated structure [v2,02/15] vhost: hide inflight async structure [v2,03/15] vhost: simplify async IO vectors [v2,04/15] vhost: simplify async IO vectors iterators [v2,05/15] vhost: remove async batch threshold [v2,06/15] vhost: introduce specific iovec structure [v2,07/15] vhost: remove useless fields in async iterator struct [v2,08/15] vhost: improve IO vector logic [v2,09/15] vhost: remove notion of async descriptor [v2,10/15] vhost: simplify async enqueue completion [v2,11/15] vhost: simplify getting the first in-flight index [v2,12/15] vhost: prepare async for mbuf to desc refactoring [v2,13/15] vhost: prepare sync for mbuf to desc refactoring [v2,14/15] vhost: merge sync and async mbuf to desc filling [v2,15/15] vhost: increase number of async IO vectors

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK
ci/github-robot: build	success	github build: passed
ci/Intel-compilation	success	Compilation OK
ci/iol-broadcom-Functional	success	Functional Testing PASS
ci/iol-broadcom-Performance	success	Performance Testing PASS
ci/iol-x86_64-compile-testing	success	Testing PASS
ci/iol-x86_64-unit-testing	success	Testing PASS
ci/intel-Testing	success	Testing PASS
ci/iol-aarch64-compile-testing	success	Testing PASS
ci/iol-mellanox-Performance	success	Performance Testing PASS
ci/iol-intel-Performance	success	Performance Testing PASS
ci/iol-intel-Functional	success	Functional Testing PASS
ci/iol-aarch64-unit-testing	success	Testing PASS

Commit Message

Maxime Coquelin Oct. 26, 2021, 4:29 p.m. UTC

  This patch increases the number of IO vectors for the
asynchronous data path from 512 to 2048. It has been
reported during testing the startvation of IO vectors
during iperf benchmark with 64KB packet size.

As there are no direct relationship between
VHOST_MAX_ASYNC_VEC and BUF_VECTOR_MAX, this patch also
assign VHOST_MAX_ASYNC_VEC value directly instyead of being
a multiple of BUF_VECTOR_MAX.

Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
---
 lib/vhost/vhost.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/lib/vhost/vhost.h b/lib/vhost/vhost.h
index ae5bc257cc..eaee59c0cc 100644
--- a/lib/vhost/vhost.h
+++ b/lib/vhost/vhost.h
@@ -49,7 +49,7 @@ 
 #define MAX_PKT_BURST 32
 
 #define VHOST_MAX_ASYNC_IT (MAX_PKT_BURST)
-#define VHOST_MAX_ASYNC_VEC (BUF_VECTOR_MAX * 2)
+#define VHOST_MAX_ASYNC_VEC 2048
 
 #define PACKED_DESC_ENQUEUE_USED_FLAG(w)	\
 	((w) ? (VRING_DESC_F_AVAIL | VRING_DESC_F_USED | VRING_DESC_F_WRITE) : \