[Skiboot] [PATCH v2 08/11] xive/p9: Clarify indirect table allocation

Cédric Le Goater clg at kaod.org
Fri Jun 12 21:37:29 AEST 2020


The XIVE interrupt controller uses a set of Virtualization Structure
Tables (VST) which characteristics, type, address, size, are described
by Virtual Structure Descriptors (VSD). A VSD is 64bit wide.

The EQ and VP tables are indirect tables. The VSD points to a single
page of VSDs each pointing to a page of virtual structures. Indirect
tables are limited to a single top page which is enough to cover the
whole range of EQs (24 bits) and VPs (19bits).

Signed-off-by: Cédric Le Goater <clg at kaod.org>
---
 hw/xive.c | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/hw/xive.c b/hw/xive.c
index 9bc78de9542c..8e7a51125f44 100644
--- a/hw/xive.c
+++ b/hw/xive.c
@@ -91,6 +91,7 @@
  * local block of that chip
  */
 
+#define XIVE_VSD_SIZE		sizeof(u64)
 
 /* BAR default values (should be initialized by HostBoot but for
  * now we do it). Based on the memory map document by Dave Larson
@@ -180,7 +181,7 @@
 
 #define XIVE_EQ_ORDER		20 /* 1M ENDs */
 #define XIVE_EQ_COUNT		(1ul << XIVE_EQ_ORDER)
-#define IND_EQ_TABLE_SIZE	((XIVE_EQ_COUNT / EQ_PER_PAGE) * 8)
+#define XIVE_EQ_TABLE_SIZE	((XIVE_EQ_COUNT / EQ_PER_PAGE) * XIVE_VSD_SIZE)
 
 #define XIVE_EQ_SHIFT		(16 + 1) /* ESn + ESe pages */
 
@@ -217,7 +218,7 @@
 
 #define MAX_VP_ORDER		NVT_SHIFT /* 512k */
 #define MAX_VP_COUNT		(1ul << MAX_VP_ORDER)
-#define IND_VP_TABLE_SIZE	((MAX_VP_COUNT / VP_PER_PAGE) * 8)
+#define XIVE_VP_TABLE_SIZE	((MAX_VP_COUNT / VP_PER_PAGE) * XIVE_VSD_SIZE)
 
 /* Initial number of VPs (XXX Make it a variable ?). Round things
  * up to a max of 32 cores per chip
@@ -1661,10 +1662,12 @@ static bool xive_prealloc_tables(struct xive *x)
 	memset(x->ivt_base, 0, IVT_SIZE);
 	xive_dbg(x, "IVT at %p size 0x%lx\n", x->ivt_base, IVT_SIZE);
 
-	/* Indirect EQ table. (XXX Align to 64K until I figure out the
-	 * HW requirements)
-	 */
-	al = (IND_EQ_TABLE_SIZE + 0xffff) & ~0xffffull;
+	/* Indirect EQ table.  Limited to one top page. */
+	al = ALIGN_UP(XIVE_EQ_TABLE_SIZE, 0x10000);
+	if (al > 0x10000) {
+		xive_err(x, "EQ indirect table is too big !\n");
+		return false;
+	}
 	x->eq_ind_base = local_alloc(x->chip_id, al, al);
 	if (!x->eq_ind_base) {
 		xive_err(x, "Failed to allocate EQ indirect table\n");
@@ -1672,19 +1675,21 @@ static bool xive_prealloc_tables(struct xive *x)
 	}
 	memset(x->eq_ind_base, 0, al);
 	xive_dbg(x, "EQi at %p size 0x%llx\n", x->eq_ind_base, al);
-	x->eq_ind_count = IND_EQ_TABLE_SIZE / 8;
+	x->eq_ind_count = XIVE_EQ_TABLE_SIZE / 8;
 
-	/* Indirect VP table. (XXX Align to 64K until I figure out the
-	 * HW requirements)
-	 */
-	al = (IND_VP_TABLE_SIZE + 0xffff) & ~0xffffull;
+	/* Indirect VP table.  Limited to one top page. */
+	al = ALIGN_UP(XIVE_VP_TABLE_SIZE, 0x10000);
+	if (al > 0x10000) {
+		xive_err(x, "VP indirect table is too big !\n");
+		return false;
+	}
 	x->vp_ind_base = local_alloc(x->chip_id, al, al);
 	if (!x->vp_ind_base) {
 		xive_err(x, "Failed to allocate VP indirect table\n");
 		return false;
 	}
 	xive_dbg(x, "VPi at %p size 0x%llx\n", x->vp_ind_base, al);
-	x->vp_ind_count = IND_VP_TABLE_SIZE / 8;
+	x->vp_ind_count = XIVE_VP_TABLE_SIZE / 8;
 	memset(x->vp_ind_base, 0, al);
 
 	/* Populate/initialize VP/EQs indirect backing */
-- 
2.25.4



More information about the Skiboot mailing list