On Fri, 04 Nov 2016 09:39:15 +0100 Greg Kurz <[email protected]> wrote: > With virtio 1, the vring layout is split in 3 separate regions of > contiguous memory for the descriptor table, the available ring and the > used ring, as opposed with legacy virtio which uses a single region. > > In case of memory re-mapping, the code ensures it doesn't affect the > vring mapping. This is done in vhost_verify_ring_mappings() which assumes > the device is legacy. > > This patch changes vhost_verify_ring_mappings() to check the mappings of > each part of the vring separately.
Add "This will work for legacy mappings as well." ? > > Signed-off-by: Greg Kurz <[email protected]> > --- > hw/virtio/vhost.c | 79 > ++++++++++++++++++++++++++++++++++----------- > include/hw/virtio/vhost.h | 4 ++ > 2 files changed, 64 insertions(+), 19 deletions(-) Reviewed-by: Cornelia Huck <[email protected]>
