summaryrefslogtreecommitdiffstats
path: root/drivers/vhost
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2015-02-25 15:20:01 +0100
committerDavid S. Miller <davem@davemloft.net>2015-02-27 15:53:44 -0500
commit0d79a493e507437a2135e5ac1a447d4d503488d8 (patch)
treeeeda1dacebc689cf8e39801af7218f1c5a10fb29 /drivers/vhost
parent4c5a84421c7d1c259c3883a404f9a67a2f55b003 (diff)
downloadop-kernel-dev-0d79a493e507437a2135e5ac1a447d4d503488d8.zip
op-kernel-dev-0d79a493e507437a2135e5ac1a447d4d503488d8.tar.gz
vhost: drop hard-coded num_buffers size
The 2 that we use for copy_to_iter comes from sizeof(u16), it used to be that way before the iov iter update. Fix it up, making it obvious the size of stack access is right. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/vhost')
-rw-r--r--drivers/vhost/net.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c
index ca70434..2bbfc25e 100644
--- a/drivers/vhost/net.c
+++ b/drivers/vhost/net.c
@@ -621,7 +621,8 @@ static void handle_rx(struct vhost_net *net)
num_buffers = cpu_to_vhost16(vq, headcount);
if (likely(mergeable) &&
- copy_to_iter(&num_buffers, 2, &fixup) != 2) {
+ copy_to_iter(&num_buffers, sizeof num_buffers,
+ &fixup) != sizeof num_buffers) {
vq_err(vq, "Failed num_buffers write");
vhost_discard_vq_desc(vq, headcount);
break;
OpenPOWER on IntegriCloud