On Thu, Apr 28 2022, Eric Auger <eric.au...@redhat.com> wrote: > The CRB command buffer currently is a RAM MemoryRegion and given > its base address alignment, it causes an error report on > vfio_listener_region_add(). This region could have been a RAM device > region, easing the detection of such safe situation but this option > was not well received. So let's add a helper function that uses the > memory region owner type to detect the situation is safe wrt > the assignment. Other device types can be checked here if such kind > of problem occurs again. > > Signed-off-by: Eric Auger <eric.au...@redhat.com> > Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > Acked-by: Stefan Berger <stef...@linux.ibm.com> > Reviewed-by: Cornelia Huck <coh...@redhat.com> > > --- > > v3 -> v4: > - rebase on top of qemu_real_host_page_size() and > qemu_real_host_page_size(). Print the size and make the message > consistent > - Added Stefan's A-b and Connie R-b (despite the changes)
Still fine with me. > --- > hw/vfio/common.c | 27 ++++++++++++++++++++++++++- > hw/vfio/trace-events | 1 + > 2 files changed, 27 insertions(+), 1 deletion(-)