s390/mem_detect: fix memory type of first block

Fix a long-standing but currently irrelevant bug: the memory detection
code performs a tprot instruction on address zero to figure out if the
first memory chunk is readable or writable. Due to low address
protection the result is "read-only". If the memory detection code
would actually care, it would have to ignore the first memory
increment, but it adds the memory increment to writable memory anyway.

If memblock debugging is enabled this leads to an extra rather
surprising call which registers memory. To avoid this get rid of the
first misleading tprot call and simply assume that the first memory
increment is writable. Otherwise we wouldn't have reached the memory
detection code anyway.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

diff --git a/arch/s390/mm/mem_detect.c b/arch/s390/mm/mem_detect.c
index 0389d8d243589d929ff8be2eb273240abd7a8bc1..185e70d3e15397d6bec38e5c5f4d76e5179b18bd 100644 (file)
--- a/arch/s390/mm/mem_detect.c
+++ b/arch/s390/mm/mem_detect.c
@@ -41,7 +41,8 @@ void __init detect_memory_memblock(void)
        memblock_set_bottom_up(true);
        do {
                size = 0;
-               type = tprot(addr);
+               /* assume lowcore is writable */
+               type = addr ? tprot(addr) : CHUNK_READ_WRITE;
                do {
                        size += rzm;
                        if (max_physmem_end && addr + size >= max_physmem_end)