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]

Message-ID: <4C36D873.8060901@kernel.org>
Date:	Fri, 09 Jul 2010 01:06:11 -0700
From:	Yinghai Lu <yinghai@...nel.org>
To:	Yinghai Lu <yinghai@...nel.org>, Michal Simek <monstr@...str.eu>,
	microblaze-uclinux@...e.uq.edu.au
CC:	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

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?

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.

Thanks

Yinghai
--
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/