hw/virtio: Propagate page_mask to vhost_vdpa_listener_skipped_section()
In order to make vhost-vdpa.c a target-agnostic source unit, we need to remove the TARGET_PAGE_SIZE / TARGET_PAGE_MASK / TARGET_PAGE_ALIGN uses. TARGET_PAGE_SIZE will be replaced by the runtime qemu_target_page_size(). The other ones will be deduced from TARGET_PAGE_SIZE. Since the 3 macros are used in 3 related functions (sharing the same call tree), we'll refactor them to only depend on TARGET_PAGE_MASK. Having the following call tree: vhost_vdpa_listener_region_del() -> vhost_vdpa_listener_skipped_section() -> vhost_vdpa_section_end() The first step is to propagate TARGET_PAGE_MASK to vhost_vdpa_listener_skipped_section(). Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20230710094931.84402-2-philmd@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
parent
4565917bb0
commit
961d60e934
@ -42,7 +42,8 @@ static Int128 vhost_vdpa_section_end(const MemoryRegionSection *section)
|
||||
|
||||
static bool vhost_vdpa_listener_skipped_section(MemoryRegionSection *section,
|
||||
uint64_t iova_min,
|
||||
uint64_t iova_max)
|
||||
uint64_t iova_max,
|
||||
int page_mask)
|
||||
{
|
||||
Int128 llend;
|
||||
|
||||
@ -313,7 +314,7 @@ static void vhost_vdpa_listener_region_add(MemoryListener *listener,
|
||||
int ret;
|
||||
|
||||
if (vhost_vdpa_listener_skipped_section(section, v->iova_range.first,
|
||||
v->iova_range.last)) {
|
||||
v->iova_range.last, TARGET_PAGE_MASK)) {
|
||||
return;
|
||||
}
|
||||
if (memory_region_is_iommu(section->mr)) {
|
||||
@ -398,7 +399,7 @@ static void vhost_vdpa_listener_region_del(MemoryListener *listener,
|
||||
int ret;
|
||||
|
||||
if (vhost_vdpa_listener_skipped_section(section, v->iova_range.first,
|
||||
v->iova_range.last)) {
|
||||
v->iova_range.last, TARGET_PAGE_MASK)) {
|
||||
return;
|
||||
}
|
||||
if (memory_region_is_iommu(section->mr)) {
|
||||
|
Loading…
Reference in New Issue
Block a user