]> Gentwo Git Trees - linux/.git/commit
vhost-scsi: Dynamically allocate scatterlists
authorMike Christie <michael.christie@oracle.com>
Tue, 3 Dec 2024 19:15:12 +0000 (13:15 -0600)
committerMichael S. Tsirkin <mst@redhat.com>
Mon, 24 Feb 2025 21:47:59 +0000 (16:47 -0500)
commit6285ac7dcd93cc96f60215207178638038fd97f0
treedc982092681e49f3f175cd46e854a036dc90529a
parentb7b349f06c3d2c6c32c049a954ff418410244093
vhost-scsi: Dynamically allocate scatterlists

We currently preallocate scatterlists which have 2048 entries for each
command. For a small device with just 1 queue this results in:

8 MB = 32 bytes per sg * 2048 entries * 128 cmd

When mq is turned on and we increase the virtqueue_size so we can handle
commands from multiple queues in parallel, then this can sky rocket.

This patch allows us to dynamically allocate the scatterlist like is done
with drivers like NVMe and SCSI.

For small IO (4-16K) IOPs testing, we didn't see any regressions, but
for throughput testing we sometimes saw a 2-5% regression when the
backend device was very fast (8 NVMe drives in a MD RAID0 config or a
memory backed device). As a result this patch makes the dynamic
allocation feature a modparam so userspace can decide how it wants to
balance mem use and perf.

Signed-off-by: Mike Christie <michael.christie@oracle.com>
Message-Id: <20241203191705.19431-6-michael.christie@oracle.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
drivers/vhost/Kconfig
drivers/vhost/scsi.c