diff options
| author | Eric Auger <eric.auger@redhat.com> | 2022-05-24 05:14:05 -0400 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2022-06-08 08:44:19 -0600 |
| commit | ec6600be0dc16982181c7ad80d94c143c0807dd2 (patch) | |
| tree | a24241945649bcf10ae3fdf5cc8346ea7541b3d1 /hw/vfio/common.c | |
| parent | 9b1f58854959c5a9bdb347e3e04c252ab7fc9ef5 (diff) | |
| download | focaccia-qemu-ec6600be0dc16982181c7ad80d94c143c0807dd2.tar.gz focaccia-qemu-ec6600be0dc16982181c7ad80d94c143c0807dd2.zip | |
vfio/common: remove spurious warning on vfio_listener_region_del
851d6d1a0f ("vfio/common: remove spurious tpm-crb-cmd misalignment
warning") removed the warning on vfio_listener_region_add() path.
However the same warning also hits on region_del path. Let's remove
it and reword the dynamic trace as this can be called on both
map and unmap path.
Signed-off-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Link: https://lore.kernel.org/r/20220524091405.416256-1-eric.auger@redhat.com
Fixes: 851d6d1a0ff2 ("vfio/common: remove spurious tpm-crb-cmd misalignment warning")
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'hw/vfio/common.c')
| -rw-r--r-- | hw/vfio/common.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/hw/vfio/common.c b/hw/vfio/common.c index 29982c7af8..ace9562a9b 100644 --- a/hw/vfio/common.c +++ b/hw/vfio/common.c @@ -1145,7 +1145,15 @@ static void vfio_listener_region_del(MemoryListener *listener, if (unlikely((section->offset_within_address_space & ~qemu_real_host_page_mask()) != (section->offset_within_region & ~qemu_real_host_page_mask()))) { - error_report("%s received unaligned region", __func__); + if (!vfio_known_safe_misalignment(section)) { + error_report("%s received unaligned region %s iova=0x%"PRIx64 + " offset_within_region=0x%"PRIx64 + " qemu_real_host_page_size=0x%"PRIxPTR, + __func__, memory_region_name(section->mr), + section->offset_within_address_space, + section->offset_within_region, + qemu_real_host_page_size()); + } return; } |