qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [PATCH V3 0/3] vhost-user: sync backend


From: Marcel Apfelbaum
Subject: [Qemu-devel] [PATCH V3 0/3] vhost-user: sync backend
Date: Fri, 31 Jul 2015 19:38:17 +0300

To be used on top of:
 [PATCH 0/4] vhost-user: protocol updates
 https://lists.gnu.org/archive/html/qemu-devel/2015-07/msg03842.html 

v2 -> v3:
 - Addressed Michael S. Tsirkin's comments:
   - Fixed a typo in patch 2/3  message
   - MQ - I left it there for completeness,
     it will be moved once the "protocol" bits
     will be 'active'
   - unit-test - The bits were added to GET_FEATURES,
     not GET_PROTOCOL_FEATURES, no need to make any changes
 - Added a new patch (1/3) that removes the unnecessary
   virtio bits related to vhost-user backend.

v1 -> v2:
 - Addressed Michael S. Tsirkin's comments:
   - Prefer a white-list of supported features
   - Add a unit-test to show the problem we are trying to solve
     (run the unit-test before the patch and it will fail)

Currently the vhost-user supported features are not evaluated.
 The way I see it, and please correct me, the best way to do
 this is to:
  1. Get the backend features on vhost init.
  2. Check them one by one (white-list) against all 
     the currently supported virtio features.
  3. All other code should remain the same.


Marcel Apfelbaum (3):
  hw/vhost-user: remove unnecessary virtio control bits
  hw/vhost-user: sync backend features
  tests/vhost-user: check vhost-user feature negotiation

 hw/net/vhost_net.c      | 22 ----------------------
 hw/virtio/vhost-user.c  | 17 +++++++++++++++++
 tests/vhost-user-test.c | 19 ++++++++++++++++---
 3 files changed, 33 insertions(+), 25 deletions(-)

-- 
2.1.0




reply via email to

[Prev in Thread] Current Thread [Next in Thread]