]> git.proxmox.com Git - mirror_qemu.git/commit - exec.c
vhost-user: fix regions provied with VHOST_USER_SET_MEM_TABLE message
authorDamjan Marion <damarion@cisco.com>
Thu, 26 Jun 2014 21:01:32 +0000 (23:01 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Sun, 29 Jun 2014 16:39:40 +0000 (19:39 +0300)
commit3fd74b84076488ae44ba5f3cfed22ff056c5199c
tree5780453adca5f1b8f6ad52b572d06330c323f873
parent12d6e4640ce28d002e96d58c31cd5e7256258626
vhost-user: fix regions provied with VHOST_USER_SET_MEM_TABLE message

Old code was affected by memory gaps which resulted in buffer pointers
pointing to address outside of the mapped regions.

Here we are introducing following changes:
 - new function qemu_get_ram_block_host_ptr() returns host pointer
   to the ram block, it is needed to calculate offset of specific
   region in the host memory
 - new field mmap_offset is added to the VhostUserMemoryRegion. It
   contains offset where specific region starts in the mapped memory.
   As there is stil no wider adoption of vhost-user agreement was made
   that we will not bump version number due to this change
 - other fileds in VhostUserMemoryRegion struct are not changed, as
   they are all needed for usermode app implementation
 - region data is not taken from ram_list.blocks anymore, instead we
   use region data which is alredy calculated for use in vhost-net
 - Now multiple regions can have same FD and user applicaton can call
   mmap() multiple times with the same FD but with different offset
   (user needs to take care for offset page alignment)

Signed-off-by: Damjan Marion <damarion@cisco.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Damjan Marion <damarion@cisco.com>
docs/specs/vhost-user.txt
exec.c
hw/virtio/vhost-user.c
include/exec/ram_addr.h