[flashrom-gerrit] Change in flashrom[master]: flashrom.c: Warn when mapping >=16M flash chips from high mem

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/63534 )

Change subject: flashrom.c: Warn when mapping >=16M flash chips from high mem
......................................................................

flashrom.c: Warn when mapping >=16M flash chips from high mem

Change-Id: Ibf59c243fb9c6e0db68f17ee87fc1bb354a3f886
Signed-off-by: Edward O'Callaghan 
---
M flashrom.c
1 file changed, 4 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/34/63534/1

diff --git a/flashrom.c b/flashrom.c
index ac61259..ccdf7f2 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -694,6 +694,10 @@
    if (flash->chip->total_size == 0)
    	return 0;

+	/* FIXME: The internal master hooks 'map_flash_region = physmap'. */
+	if (flash->chip->total_size > 16 * 1024)
+		msg_cwarn("Warning: maping %d kB sized flash chip. This could cause issue on x86.\n", flash->chip->total_size * 1024);
+
    const chipsize_t size = flash->chip->total_size * 1024;
    uintptr_t base = flashbase ? flashbase : (0xffffffff - size + 1);
    void *addr = programmer_map_flash_region(flash->chip->name, base, size);