On Thu, Feb 16, 2023 at 12:47:50PM +0100, David Hildenbrand wrote: > Following up on my previous work to make virtio-mem consume multiple > memslots dynamically [1] that requires precise accounting between used vs. > reserved memslots, I realized that vhost makes this extra hard by > filtering out some memory region sections (so they don't consume a > memslot) in the vhost-user case, which messes up the whole memslot > accounting. > > This series fixes what I found to be broken and prepares for more work on > [1]. Further, it cleanes up the merge checks that I consider unnecessary. > > [1] https://lkml.kernel.org/r/[email protected] > > Cc: "Michael S. Tsirkin" <[email protected]> > Cc: Stefan Hajnoczi <[email protected]> > Cc: Dr. David Alan Gilbert <[email protected]> > > David Hildenbrand (2): > vhost: Defer filtering memory sections until building the vhost memory > structure > vhost: Remove vhost_backend_can_merge() callback > > hw/virtio/vhost-user.c | 14 ----- > hw/virtio/vhost-vdpa.c | 1 - > hw/virtio/vhost.c | 85 ++++++++++++++++++++----------- > include/hw/virtio/vhost-backend.h | 4 -- > 4 files changed, 56 insertions(+), 48 deletions(-) > > -- > 2.39.1 >
I'm not familiar enough with the memslot code to review this properly, but overall it looks okay: Acked-by: Stefan Hajnoczi <[email protected]>
signature.asc
Description: PGP signature
