[Skiboot] [PATCH] hdata: Fix SP attention area address

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Wed Feb 5 21:58:05 AEDT 2020


SP attention area is aligned. We were sending wrong address.
Hence `attn` on FSP based system is failing. Align SP attention
area so that FSP can locate attention data.

Fixes: 518e554 (spira: fix endian conversions in spira data structures)
CC: Nicholas Piggin <npiggin at gmail.com>

Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
---
 hdata/spira.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/hdata/spira.c b/hdata/spira.c
index 7c5918d2d..76c3d60a5 100644
--- a/hdata/spira.c
+++ b/hdata/spira.c
@@ -11,6 +11,7 @@
 #include <opal-dump.h>
 #include <fsp-attn.h>
 #include <fsp-leds.h>
+#include <skiboot.h>
 
 #include "hdata.h"
 #include "hostservices.h"
@@ -58,7 +59,7 @@ extern struct hsr_data_area cpu_ctl_hsr_area;
  */
 #define CPU_CTL_INIT_DATA_OFF		(CPU_CTL_OFF)
 #define CPU_CTL_SPAT_AREA_OFF		(CPU_CTL_INIT_DATA_OFF + sizeof(struct cpu_ctl_init_data) + SKIBOOT_BASE)
-#define CPU_CTL_SP_ATTN_AREA1_OFF	(CPU_CTL_SPAT_AREA_OFF + sizeof(struct sp_addr_table))
+#define CPU_CTL_SP_ATTN_AREA1_OFF	(ALIGN_UP((CPU_CTL_SPAT_AREA_OFF + sizeof(struct sp_addr_table)), ATTN_AREA_SZ))
 #define CPU_CTL_SP_ATTN_AREA2_OFF	(CPU_CTL_SP_ATTN_AREA1_OFF + sizeof(struct sp_attn_area))
 #define CPU_CTL_HSR_AREA_OFF		(CPU_CTL_SP_ATTN_AREA2_OFF + sizeof(struct sp_attn_area))
 
-- 
2.21.1



More information about the Skiboot mailing list