Skip to content

vhost-user fixes

Nikolay Nikolaev qemu-devel 2 versions – In review

Get series with b4
b4 am 20140712014119.10636.81503.stgit@3820

Versions newest first · linked by author and title

  1. v2 Latest2 patches 2 repliesIn review
  2. v13 patches 3 repliesSuperseded

Changes in v2 from the cover letter

- dropped the patches 1 and 2 and just fixed the "real issue"

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2vhost-user: Fix VHOST_SET_MEM_TABLE processinghw/virtio/vhost-user.c+3 -12 replies
2/2qtest: Adapt vhost-user-test to latehs vhost-user changestests/vhost-user-test.c+9 -20 replies

Reviews trailers per patch

1 of 2 patches has a Reviewed-by or Acked-by.

PatchPaolo Bonzini
1/2R-b
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [Qemu-devel] [PATCH v2 0/2 for-2.1] vhost-user fixes

The latest vhost-user changes changed the VHOST_SET_MEM_TABLE handling. Now the memory regions are mapped from dev->mem. There is an error when calling qemu_get_ram_fd with reg->guest_phys_addr. The right approach is to find the ram_addr_t corresponding to the userspace_addr using qemu_ram_addr_from_host first.

Thanks to Paolo Bonzini for poinitng the real problem.

The related vhost-user qtest is also updated to reflect the changes in vhost-user message structures.

Changes since v1:
- dropped the patches 1 and 2 and just fixed the "real issue"

---

Nikolay Nikolaev (2):
vhost-user: Fix VHOST_SET_MEM_TABLE processing
qtest: Adapt vhost-user-test to latehs vhost-user changes

 hw/virtio/vhost-user.c  |    4 +++-
 tests/vhost-user-test.c |   11 +++++++++--
 2 files changed, 12 insertions(+), 3 deletions(-)

--
Signature