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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Fri, 09 Jul 2010 11:49:42 +0200
From:	Michal Simek <monstr@...str.eu>
To:	Yinghai Lu <yinghai@...nel.org>
CC:	microblaze-uclinux@...e.uq.edu.au, Ingo Molnar <mingo@...e.hu>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	David Miller <davem@...emloft.net>,
	Benjamin Herrenschmidt <benh@...nel.crashing.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Johannes Weiner <hannes@...xchg.org>,
	linux-kernel@...r.kernel.org, linux-arch@...r.kernel.org
Subject: Re: [PATCH 01/49] lmb: rename to memblock

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@...e.hu>
>> Acked-by: "H. Peter Anvin" <hpa@...or.com>
>> Acked-by: Benjamin Herrenschmidt <benh@...nel.crashing.org>
>> Acked-by: Linus Torvalds <torvalds@...ux-foundation.org>
>> Signed-off-by: Yinghai Lu <yinghai@...nel.org>
>> ---
>>  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@...str.eu>

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/