powerpc/fadump: remove RMA_START and RMA_END macros
authorHari Bathini <hbathini@linux.ibm.com>
Wed, 11 Sep 2019 14:57:26 +0000 (20:27 +0530)
committerMichael Ellerman <mpe@ellerman.id.au>
Fri, 13 Sep 2019 14:04:46 +0000 (00:04 +1000)
RMA_START is defined as '0' and there is even a BUILD_BUG_ON() to
make sure it is never anything else. Remove this macro and use '0'
instead as code change is needed anyway when it has to be something
else. Also, remove unused RMA_END macro.

Signed-off-by: Hari Bathini <hbathini@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/156821384096.5656.15026984053970204652.stgit@hbathini.in.ibm.com
arch/powerpc/include/asm/fadump-internal.h
arch/powerpc/kernel/fadump.c
arch/powerpc/platforms/pseries/rtas-fadump.c

index a669aaa7c6107719e0cc2f37c985dfa990dc8396..aec1515ac8f8cbe61fcb2f2ecc23098fefb79ca2 100644 (file)
 #define _ASM_POWERPC_FADUMP_INTERNAL_H
 
 #ifndef CONFIG_PRESERVE_FA_DUMP
-/*
- * The RMA region will be saved for later dumping when kernel crashes.
- * RMA is Real Mode Area, the first block of logical memory address owned
- * by logical partition, containing the storage that may be accessed with
- * translate off.
- */
-#define RMA_START      0x0
-#define RMA_END                (ppc64_rma_size)
 
 /* The upper limit percentage for user specified boot memory size (25%) */
 #define MAX_BOOT_MEM_RATIO                     4
index bd49b1f200bf07033fcafcf72ba127f7bb6d622e..2e139259474d7ac61ccb9266d80282d53af0e5f8 100644 (file)
@@ -128,18 +128,22 @@ int __init early_init_dt_scan_fw_dump(unsigned long node, const char *uname,
  * If fadump is registered, check if the memory provided
  * falls within boot memory area and reserved memory area.
  */
-int is_fadump_memory_area(u64 addr, ulong size)
+int is_fadump_memory_area(u64 addr, unsigned long size)
 {
-       u64 d_start = fw_dump.reserve_dump_area_start;
-       u64 d_end = d_start + fw_dump.reserve_dump_area_size;
+       u64 d_start, d_end;
 
        if (!fw_dump.dump_registered)
                return 0;
 
+       if (!size)
+               return 0;
+
+       d_start = fw_dump.reserve_dump_area_start;
+       d_end = d_start + fw_dump.reserve_dump_area_size;
        if (((addr + size) > d_start) && (addr <= d_end))
                return 1;
 
-       return (addr + size) > RMA_START && addr <= fw_dump.boot_memory_size;
+       return (addr <= fw_dump.boot_memory_size);
 }
 
 int should_fadump_crash(void)
@@ -771,14 +775,14 @@ static int fadump_setup_crash_memory_ranges(void)
        crash_mrange_info.mem_range_cnt = 0;
 
        /*
-        * add the first memory chunk (RMA_START through boot_memory_size) as
+        * add the first memory chunk (0 through boot_memory_size) as
         * a separate memory chunk. The reason is, at the time crash firmware
         * will move the content of this memory chunk to different location
         * specified during fadump registration. We need to create a separate
         * program header for this chunk with the correct offset.
         */
        ret = fadump_add_mem_range(&crash_mrange_info,
-                                  RMA_START, fw_dump.boot_memory_size);
+                                  0, fw_dump.boot_memory_size);
        if (ret)
                return ret;
 
@@ -787,11 +791,9 @@ static int fadump_setup_crash_memory_ranges(void)
                end = start + (u64)reg->size;
 
                /*
-                * skip the first memory chunk that is already added (RMA_START
-                * through boot_memory_size). This logic needs a relook if and
-                * when RMA_START changes to a non-zero value.
+                * skip the first memory chunk that is already added
+                * (0 through boot_memory_size).
                 */
-               BUILD_BUG_ON(RMA_START != 0);
                if (start < fw_dump.boot_memory_size) {
                        if (end > fw_dump.boot_memory_size)
                                start = fw_dump.boot_memory_size;
@@ -815,7 +817,7 @@ static int fadump_setup_crash_memory_ranges(void)
  */
 static inline unsigned long fadump_relocate(unsigned long paddr)
 {
-       if (paddr > RMA_START && paddr < fw_dump.boot_memory_size)
+       if ((paddr > 0) && (paddr < fw_dump.boot_memory_size))
                return fw_dump.boot_mem_dest_addr + paddr;
        else
                return paddr;
@@ -883,11 +885,11 @@ static int fadump_create_elfcore_headers(char *bufp)
                phdr->p_flags   = PF_R|PF_W|PF_X;
                phdr->p_offset  = mbase;
 
-               if (mbase == RMA_START) {
+               if (mbase == 0) {
                        /*
-                        * The entire RMA region will be moved by firmware
-                        * to the specified destination_address. Hence set
-                        * the correct offset.
+                        * The entire real memory region will be moved by
+                        * firmware to the specified destination_address.
+                        * Hence set the correct offset.
                         */
                        phdr->p_offset = fw_dump.boot_mem_dest_addr;
                }
index 1d8f2871fa344bb09e8b08407d4ace22f7451517..a525180c1f0fc459443eae264ea58439154c0b05 100644 (file)
@@ -106,7 +106,7 @@ static u64 rtas_fadump_init_mem_struct(struct fw_dump *fadump_conf)
        fdm.rmr_region.request_flag = cpu_to_be32(RTAS_FADUMP_REQUEST_FLAG);
        fdm.rmr_region.source_data_type =
                cpu_to_be16(RTAS_FADUMP_REAL_MODE_REGION);
-       fdm.rmr_region.source_address = cpu_to_be64(RMA_START);
+       fdm.rmr_region.source_address = cpu_to_be64(0);
        fdm.rmr_region.source_len = cpu_to_be64(fadump_conf->boot_memory_size);
        fdm.rmr_region.destination_address = cpu_to_be64(addr);
        addr += fadump_conf->boot_memory_size;