Re: [PATCH 01/49] lmb: rename to memblock

From: Michal Simek
Date: Fri Jul 09 2010 - 05:53:10 EST


Yinghai Lu wrote:
On 07/06/2010 03:38 PM, Yinghai Lu wrote:
via following scripts

FILES=$(find * -type f | grep -vE 'oprofile|[^K]config')

sed -i \
-e 's/lmb/memblock/g' \
-e 's/LMB/MEMBLOCK/g' \
$FILES

for N in $(find . -name lmb.[ch]); do
M=$(echo $N | sed 's/lmb/memblock/g')
mv $N $M
done

and remove some wrong change like lmbench and dlmb etc.

also move memblock.c from lib/ to mm/

Suggested-by: Ingo Molnar <mingo@xxxxxxx>
Acked-by: "H. Peter Anvin" <hpa@xxxxxxxxx>
Acked-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
Acked-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
Signed-off-by: Yinghai Lu <yinghai@xxxxxxxxxx>
---
arch/microblaze/platform/generic/system.dts | 4 +-

...
diff --git a/arch/microblaze/platform/generic/system.dts b/arch/microblaze/platform/generic/system.dts
index 2d5c417..2936ae6 100644
--- a/arch/microblaze/platform/generic/system.dts
+++ b/arch/microblaze/platform/generic/system.dts
@@ -70,7 +70,7 @@
xlnx,allow-icache-wr = <0x1>;
xlnx,area-optimized = <0x0>;
xlnx,cache-byte-size = <0x2000>;
- xlnx,d-lmb = <0x1>;
+ xlnx,d-memblock = <0x1>;
xlnx,d-opb = <0x0>;
xlnx,d-plb = <0x1>;
xlnx,data-size = <0x20>;
@@ -89,7 +89,7 @@
xlnx,fsl-data-size = <0x20>;
xlnx,fsl-exception = <0x0>;
xlnx,fsl-links = <0x0>;
- xlnx,i-lmb = <0x1>;
+ xlnx,i-memblock = <0x1>;
xlnx,i-opb = <0x0>;
xlnx,i-plb = <0x1>;
xlnx,icache-always-used = <0x1>;
...

looks like that change to microblaze dts is wrong.

Michal, it seems lmb in microblaze have lmb special?

DTS change is definitely wrong. LMB means local memory bus - which means that CPU has this connection.


in arch/microblaze/

include/asm/cpuinfo.h: u32 use_dlmb;
include/asm/cpuinfo.h: u32 use_ilmb;
kernel/cpu/cpuinfo-pvr-full.c: CI(use_dlmb, D_LMB);
kernel/cpu/cpuinfo-pvr-full.c: CI(use_ilmb, I_LMB);
kernel/cpu/cpuinfo-static.c: ci->use_dlmb = fcpu(cpu, "xlnx,d-lmb");
kernel/cpu/cpuinfo-static.c: ci->use_ilmb = fcpu(cpu, "xlnx,i-lmb");
kernel/prom.c:#include <linux/lmb.h>
kernel/prom.c: lmb_add(base, size);
kernel/prom.c: return lmb_alloc(size, align);
kernel/prom.c: lmb_init();
kernel/prom.c: lmb_analyze();
kernel/prom.c: pr_debug("Phys. mem: %lx\n", (unsigned long) lmb_phys_mem_size());
mm/init.c:#include <linux/lmb.h>
mm/init.c: for (i = 0; i < lmb.memory.cnt; i++) {
mm/init.c: memory_start = (u32) lmb.memory.region[i].base;
mm/init.c: memory_end = (u32) lmb.memory.region[i].base
mm/init.c: + (u32) lmb.memory.region[i].size;
mm/init.c: lmb_reserve(kernel_align_start, kernel_align_size);
mm/init.c: lmb_reserve(PFN_UP(TOPHYS((u32)klimit)) << PAGE_SHIFT, map_size);
mm/init.c: for (i = 0; i < lmb.reserved.cnt; i++) {
mm/init.c: (u32) lmb.reserved.region[i].base,
mm/init.c: (u32) lmb_size_bytes(&lmb.reserved, i));
mm/init.c: reserve_bootmem(lmb.reserved.region[i].base,
mm/init.c: lmb_size_bytes(&lmb.reserved, i) - 1, BOOTMEM_DEFAULT);
mm/init.c: lmb.memory.region[0].size = memory_size;
mm/init.c: if (!lmb.reserved.cnt) {
mm/init.c: if ((u32) lmb.memory.region[0].size < 0x1000000) {
mm/init.c: memory_start = (u32) lmb.memory.region[0].base;
mm/init.c: memory_end = (u32) lmb.memory.region[0].base +
mm/init.c: (u32) lmb.memory.region[0].size;
mm/init.c: lmb_reserve(kstart, ksize);
mm/init.c: p = __va(lmb_alloc_base(PAGE_SIZE, PAGE_SIZE,
platform/generic/system.dts: xlnx,d-lmb = <0x1>;
platform/generic/system.dts: xlnx,i-lmb = <0x1>;

thought we should keep d-lmb and i-lmb, and change other lmb to memblock.


The rest of microblaze changes look ok.

Please revert this DTS part and for the rest:

Acked-by: Michal Simek <monstr@xxxxxxxxx>

Thanks,
Michal



--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/