| Message ID | 20190227085149.38596-4-aik@ozlabs.ru |
|---|---|
| State | New |
| Headers | show |
| Series | spapr_pci, vfio: NVIDIA V100 + POWER9 passthrough | expand |
On Wed, Feb 27, 2019 at 07:51:46PM +1100, Alexey Kardashevskiy wrote: > The "systempagesize" name suggests that it is the host system page size > while it is the smallest page size of memory backing the guest RAM so > let's rename it to stop confusion. This should cause no behavioral change. > > Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> Definitely a good idea, applied. > --- > hw/vfio/spapr.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/hw/vfio/spapr.c b/hw/vfio/spapr.c > index 88437a7..57fe758 100644 > --- a/hw/vfio/spapr.c > +++ b/hw/vfio/spapr.c > @@ -148,14 +148,14 @@ int vfio_spapr_create_window(VFIOContainer *container, > uint64_t pagesize = memory_region_iommu_get_min_page_size(iommu_mr); > unsigned entries, bits_total, bits_per_level, max_levels; > struct vfio_iommu_spapr_tce_create create = { .argsz = sizeof(create) }; > - long systempagesize = qemu_getrampagesize(); > + long rampagesize = qemu_getrampagesize(); > > /* > * The host might not support the guest supported IOMMU page size, > * so we will use smaller physical IOMMU pages to back them. > */ > - if (pagesize > systempagesize) { > - pagesize = systempagesize; > + if (pagesize > rampagesize) { > + pagesize = rampagesize; > } > pagesize = 1ULL << (63 - clz64(container->pgsizes & > (pagesize | (pagesize - 1))));
diff --git a/hw/vfio/spapr.c b/hw/vfio/spapr.c index 88437a7..57fe758 100644 --- a/hw/vfio/spapr.c +++ b/hw/vfio/spapr.c @@ -148,14 +148,14 @@ int vfio_spapr_create_window(VFIOContainer *container, uint64_t pagesize = memory_region_iommu_get_min_page_size(iommu_mr); unsigned entries, bits_total, bits_per_level, max_levels; struct vfio_iommu_spapr_tce_create create = { .argsz = sizeof(create) }; - long systempagesize = qemu_getrampagesize(); + long rampagesize = qemu_getrampagesize(); /* * The host might not support the guest supported IOMMU page size, * so we will use smaller physical IOMMU pages to back them. */ - if (pagesize > systempagesize) { - pagesize = systempagesize; + if (pagesize > rampagesize) { + pagesize = rampagesize; } pagesize = 1ULL << (63 - clz64(container->pgsizes & (pagesize | (pagesize - 1))));
The "systempagesize" name suggests that it is the host system page size while it is the smallest page size of memory backing the guest RAM so let's rename it to stop confusion. This should cause no behavioral change. Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> --- hw/vfio/spapr.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)