|
 |
76daa3 |
From 966857f341d1e5935546a92c5712b637bf64021d Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Jens Freimann <jfreiman@redhat.com>
|
|
|
76daa3 |
Date: Thu, 1 Jun 2017 13:17:31 +0200
|
|
|
76daa3 |
Subject: [PATCH 08/13] vhost-user: pass message as a pointer to
|
|
|
76daa3 |
process_message_reply()
|
|
|
76daa3 |
MIME-Version: 1.0
|
|
|
76daa3 |
Content-Type: text/plain; charset=UTF-8
|
|
|
76daa3 |
Content-Transfer-Encoding: 8bit
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Jens Freimann <jfreiman@redhat.com>
|
|
|
76daa3 |
Message-id: <20170601131731.17338-2-jfreiman@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75456
|
|
|
76daa3 |
O-Subject: [RHV-7.4 qemu-kvm-rhev PATCH 1/1] vhost-user: pass message as a pointer to process_message_reply()
|
|
|
76daa3 |
Bugzilla: 1447592
|
|
|
76daa3 |
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Xiao Wang <jasowang@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Vlad Yasevich <vyasevic@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
From: Maxime Coquelin <maxime.coquelin@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
process_message_reply() was recently updated to get full message
|
|
|
76daa3 |
content instead of only its request field.
|
|
|
76daa3 |
|
|
|
76daa3 |
There is no need to copy all the struct content into the stack,
|
|
|
76daa3 |
so just pass its pointer as const.
|
|
|
76daa3 |
|
|
|
76daa3 |
Reviewed-by: Jens Freimann <jfreiman@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Zhiyong Yang <zhiyong.yang@intel.com>
|
|
|
76daa3 |
Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
|
|
|
76daa3 |
(cherry-picked from commit 60cd11024f41cc73175e651a2dfe09a3cade56bb)
|
|
|
76daa3 |
|
|
|
76daa3 |
This cleans up commit 15a523d
|
|
|
76daa3 |
"hw/virtio: fix vhost user fails to startup when MQ"
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Jens Freimann <jfreiman@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
hw/virtio/vhost-user.c | 12 ++++++------
|
|
|
76daa3 |
1 file changed, 6 insertions(+), 6 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
|
|
|
76daa3 |
index 32a95a8..24ca863 100644
|
|
|
76daa3 |
--- a/hw/virtio/vhost-user.c
|
|
|
76daa3 |
+++ b/hw/virtio/vhost-user.c
|
|
|
76daa3 |
@@ -163,11 +163,11 @@ fail:
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
static int process_message_reply(struct vhost_dev *dev,
|
|
|
76daa3 |
- VhostUserMsg msg)
|
|
|
76daa3 |
+ const VhostUserMsg *msg)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
VhostUserMsg msg_reply;
|
|
|
76daa3 |
|
|
|
76daa3 |
- if ((msg.flags & VHOST_USER_NEED_REPLY_MASK) == 0) {
|
|
|
76daa3 |
+ if ((msg->flags & VHOST_USER_NEED_REPLY_MASK) == 0) {
|
|
|
76daa3 |
return 0;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
@@ -175,10 +175,10 @@ static int process_message_reply(struct vhost_dev *dev,
|
|
|
76daa3 |
return -1;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
- if (msg_reply.request != msg.request) {
|
|
|
76daa3 |
+ if (msg_reply.request != msg->request) {
|
|
|
76daa3 |
error_report("Received unexpected msg type."
|
|
|
76daa3 |
"Expected %d received %d",
|
|
|
76daa3 |
- msg.request, msg_reply.request);
|
|
|
76daa3 |
+ msg->request, msg_reply.request);
|
|
|
76daa3 |
return -1;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
@@ -325,7 +325,7 @@ static int vhost_user_set_mem_table(struct vhost_dev *dev,
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
if (reply_supported) {
|
|
|
76daa3 |
- return process_message_reply(dev, msg);
|
|
|
76daa3 |
+ return process_message_reply(dev, &msg;;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
return 0;
|
|
|
76daa3 |
@@ -717,7 +717,7 @@ static int vhost_user_net_set_mtu(struct vhost_dev *dev, uint16_t mtu)
|
|
|
76daa3 |
|
|
|
76daa3 |
/* If reply_ack supported, slave has to ack specified MTU is valid */
|
|
|
76daa3 |
if (reply_supported) {
|
|
|
76daa3 |
- return process_message_reply(dev, msg);
|
|
|
76daa3 |
+ return process_message_reply(dev, &msg;;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
return 0;
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|