Mailing List Archive

commit 5ee30222c626fd3224f0b6d59e9856ba77bb89d4
Author: Henry Wang <Henry.Wang@arm.com>
AuthorDate: Tue Mar 28 15:13:32 2023 +0800
Commit: Julien Grall <jgrall@amazon.com>
CommitDate: Sun Apr 16 15:41:08 2023 +0100

xen/arm: Rename vgic_cpu_base and vgic_dist_base for new vGIC

In the follow-up patch from this series, the GICv2 CPU interface
mapping will be deferred until the first access in the stage-2
data abort trap handling code. Since the data abort trap handling
code is common for the current and the new vGIC implementation,
it is necessary to unify the variable names in struct vgic_dist
for these two implementations.

Therefore, this commit renames the vgic_cpu_base and vgic_dist_base
for new vGIC to cbase and dbase. So we can use the same code in
the data abort trap handling code for both vGIC implementations.

Signed-off-by: Henry Wang <Henry.Wang@arm.com>
Acked-by: Julien Grall <jgrall@amazon.com>
---
xen/arch/arm/include/asm/new_vgic.h | 8 ++++----
xen/arch/arm/vgic/vgic-init.c | 4 ++--
xen/arch/arm/vgic/vgic-v2.c | 17 ++++++++---------
3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/xen/arch/arm/include/asm/new_vgic.h b/xen/arch/arm/include/asm/new_vgic.h
index b7fa9ab11a..18ed3f754a 100644
--- a/xen/arch/arm/include/asm/new_vgic.h
+++ b/xen/arch/arm/include/asm/new_vgic.h
@@ -115,11 +115,11 @@ struct vgic_dist {
unsigned int nr_spis;

/* base addresses in guest physical address space: */
- paddr_t vgic_dist_base; /* distributor */
+ paddr_t dbase; /* distributor */
union
{
/* either a GICv2 CPU interface */
- paddr_t vgic_cpu_base;
+ paddr_t cbase;
/* or a number of GICv3 redistributor regions */
struct
{
@@ -188,12 +188,12 @@ struct vgic_cpu {

static inline paddr_t vgic_cpu_base(const struct vgic_dist *vgic)
{
- return vgic->vgic_cpu_base;
+ return vgic->cbase;
}

static inline paddr_t vgic_dist_base(const struct vgic_dist *vgic)
{
- return vgic->vgic_dist_base;
+ return vgic->dbase;
}

#endif /* __ASM_ARM_NEW_VGIC_H */
diff --git a/xen/arch/arm/vgic/vgic-init.c b/xen/arch/arm/vgic/vgic-init.c
index 62ae553699..ea739d081e 100644
--- a/xen/arch/arm/vgic/vgic-init.c
+++ b/xen/arch/arm/vgic/vgic-init.c
@@ -112,8 +112,8 @@ int domain_vgic_register(struct domain *d, int *mmio_count)
BUG();
}

- d->arch.vgic.vgic_dist_base = VGIC_ADDR_UNDEF;
- d->arch.vgic.vgic_cpu_base = VGIC_ADDR_UNDEF;
+ d->arch.vgic.dbase = VGIC_ADDR_UNDEF;
+ d->arch.vgic.cbase = VGIC_ADDR_UNDEF;
d->arch.vgic.vgic_redist_base = VGIC_ADDR_UNDEF;

return 0;
diff --git a/xen/arch/arm/vgic/vgic-v2.c b/xen/arch/arm/vgic/vgic-v2.c
index c90e88fddb..bd51ef95c8 100644
--- a/xen/arch/arm/vgic/vgic-v2.c
+++ b/xen/arch/arm/vgic/vgic-v2.c
@@ -272,7 +272,7 @@ int vgic_v2_map_resources(struct domain *d)
*/
if ( is_hardware_domain(d) )
{
- d->arch.vgic.vgic_dist_base = gic_v2_hw_data.dbase;
+ d->arch.vgic.dbase = gic_v2_hw_data.dbase;
/*
* For the hardware domain, we always map the whole HW CPU
* interface region in order to match the device tree (the "reg"
@@ -280,13 +280,13 @@ int vgic_v2_map_resources(struct domain *d)
* Note that we assume the size of the CPU interface is always
* aligned to PAGE_SIZE.
*/
- d->arch.vgic.vgic_cpu_base = gic_v2_hw_data.cbase;
+ d->arch.vgic.cbase = gic_v2_hw_data.cbase;
csize = gic_v2_hw_data.csize;
vbase = gic_v2_hw_data.vbase;
}
else if ( is_domain_direct_mapped(d) )
{
- d->arch.vgic.vgic_dist_base = gic_v2_hw_data.dbase;
+ d->arch.vgic.dbase = gic_v2_hw_data.dbase;
/*
* For all the direct-mapped domain other than the hardware domain,
* we only map a 8kB CPU interface but we make sure it is at a location
@@ -296,13 +296,13 @@ int vgic_v2_map_resources(struct domain *d)
* address when the GIC is aliased to get a 8kB contiguous
* region.
*/
- d->arch.vgic.vgic_cpu_base = gic_v2_hw_data.cbase;
+ d->arch.vgic.cbase = gic_v2_hw_data.cbase;
csize = GUEST_GICC_SIZE;
vbase = gic_v2_hw_data.vbase + gic_v2_hw_data.aliased_offset;
}
else
{
- d->arch.vgic.vgic_dist_base = GUEST_GICD_BASE;
+ d->arch.vgic.dbase = GUEST_GICD_BASE;
/*
* The CPU interface exposed to the guest is always 8kB. We may
* need to add an offset to the virtual CPU interface base
@@ -310,14 +310,13 @@ int vgic_v2_map_resources(struct domain *d)
* region.
*/
BUILD_BUG_ON(GUEST_GICC_SIZE != SZ_8K);
- d->arch.vgic.vgic_cpu_base = GUEST_GICC_BASE;
+ d->arch.vgic.cbase = GUEST_GICC_BASE;
csize = GUEST_GICC_SIZE;
vbase = gic_v2_hw_data.vbase + gic_v2_hw_data.aliased_offset;
}


- ret = vgic_register_dist_iodev(d, gaddr_to_gfn(dist->vgic_dist_base),
- VGIC_V2);
+ ret = vgic_register_dist_iodev(d, gaddr_to_gfn(dist->dbase), VGIC_V2);
if ( ret )
{
gdprintk(XENLOG_ERR, "Unable to register VGIC MMIO regions\n");
@@ -328,7 +327,7 @@ int vgic_v2_map_resources(struct domain *d)
* Map the gic virtual cpu interface in the gic cpu interface
* region of the guest.
*/
- ret = map_mmio_regions(d, gaddr_to_gfn(d->arch.vgic.vgic_cpu_base),
+ ret = map_mmio_regions(d, gaddr_to_gfn(d->arch.vgic.cbase),
csize / PAGE_SIZE, maddr_to_mfn(vbase));
if ( ret )
{
--
generated by git-patchbot for /home/xen/git/xen.git#master