From 95ba2af20d232dcab3f923f0f509684f4767ab3c Mon Sep 17 00:00:00 2001 From: Vasant Hegde Date: Mon, 18 Jul 2016 16:31:43 +0530 Subject: FSP/MDST: Fix TCE alignment issue We have used TCE_MASK value (4095) instead of TCE_PSIZE (4096) to align memory source address. In some corner cases (like source memory size = 4097) we may endup doing wrong mapping and corrupting part of SYSDUMP. This patch uses ALIGN_UP macro with TCE_PSIZE value for alignining memory. Signed-off-by: Vasant Hegde Signed-off-by: Stewart Smith --- hw/fsp/fsp-mdst-table.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'hw/fsp') diff --git a/hw/fsp/fsp-mdst-table.c b/hw/fsp/fsp-mdst-table.c index e6018aa..0f145ba 100644 --- a/hw/fsp/fsp-mdst-table.c +++ b/hw/fsp/fsp-mdst-table.c @@ -90,7 +90,7 @@ static inline uint32_t get_dump_region_map_size(uint64_t addr, uint32_t size) start = addr & ~TCE_MASK; end = addr + size; - end = (end + (TCE_MASK - 1)) & ~TCE_MASK; + end = ALIGN_UP(end, TCE_PSIZE); return (end - start); } -- cgit v1.1