From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Laura Abbott <lauraa@codeaurora.org>,
Russell King <linux@arm.linux.org.uk>,
David Brown <davidb@codeaurora.org>,
Daniel Walker <dwalker@fifo99.com>,
Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
Eric Miao <eric.y.miao@gmail.com>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Ben Dooks <ben-linux@fluff.org>,
Kukjin Kim <kgene.kim@samsung.com>,
linux-arm-kernel@lists.infradead.org
Cc: Nicolas Pitre <nicolas.pitre@linaro.org>,
Grygorii Strashko <grygorii.strashko@ti.com>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
linux-arm-msm@vger.kernel.org, Will Deacon <will.deacon@arm.com>,
linux-kernel@vger.kernel.org,
Leif Lindholm <leif.lindholm@linaro.org>,
Catalin Marinas <catalin.marinas@arm.com>,
Santosh Shilimkar <santosh.shilimkar@ti.com>
Subject: Re: [PATCHv2 2/2] arm: Get rid of meminfo
Date: Tue, 11 Feb 2014 10:47:54 +0100 [thread overview]
Message-ID: <52F9F1CA.2070501@samsung.com> (raw)
In-Reply-To: <1391558551-31395-3-git-send-email-lauraa@codeaurora.org>
Hello,
On 2014-02-05 01:02, Laura Abbott wrote:
> memblock is now fully integrated into the kernel and is the prefered
> method for tracking memory. Rather than reinvent the wheel with
> meminfo, migrate to using memblock directly instead of meminfo as
> an intermediate.
>
> Signed-off-by: Laura Abbott <lauraa@codeaurora.org>
This patch fails with SPARSEMEM enabled (tested with v3.14-rc1,
exynos4_defconfig + oldnoconfig):
arch/arm/mm/init.c: In function ‘free_unused_memmap’:
arch/arm/mm/init.c:442:10: error: ‘prev_bank_end’ undeclared (first use
in this function)
arch/arm/mm/init.c:442:10: note: each undeclared identifier is reported
only once for each function it appears in
make[1]: *** [arch/arm/mm/init.o] Error 1
make[1]: *** Waiting for unfinished jobs....
> ---
> arch/arm/include/asm/mach/arch.h | 4 +-
> arch/arm/include/asm/memblock.h | 3 +-
> arch/arm/include/asm/setup.h | 23 ------
> arch/arm/kernel/atags_parse.c | 5 +-
> arch/arm/kernel/setup.c | 30 ++------
> arch/arm/mach-clps711x/board-clep7312.c | 7 +-
> arch/arm/mach-clps711x/board-edb7211.c | 10 +--
> arch/arm/mach-clps711x/board-p720t.c | 2 +-
> arch/arm/mach-footbridge/cats-hw.c | 2 +-
> arch/arm/mach-footbridge/netwinder-hw.c | 2 +-
> arch/arm/mach-msm/board-halibut.c | 6 --
> arch/arm/mach-msm/board-mahimahi.c | 13 +---
> arch/arm/mach-msm/board-msm7x30.c | 3 +-
> arch/arm/mach-msm/board-sapphire.c | 13 ++--
> arch/arm/mach-msm/board-trout.c | 8 +--
> arch/arm/mach-orion5x/common.c | 3 +-
> arch/arm/mach-orion5x/common.h | 3 +-
> arch/arm/mach-pxa/cm-x300.c | 3 +-
> arch/arm/mach-pxa/corgi.c | 10 +--
> arch/arm/mach-pxa/eseries.c | 9 +--
> arch/arm/mach-pxa/poodle.c | 8 +--
> arch/arm/mach-pxa/spitz.c | 9 +--
> arch/arm/mach-pxa/tosa.c | 8 +--
> arch/arm/mach-realview/core.c | 11 +--
> arch/arm/mach-realview/core.h | 3 +-
> arch/arm/mach-realview/realview_pb1176.c | 8 +--
> arch/arm/mach-realview/realview_pbx.c | 17 ++---
> arch/arm/mach-s3c24xx/mach-smdk2413.c | 8 +--
> arch/arm/mach-s3c24xx/mach-vstms.c | 8 +--
> arch/arm/mach-sa1100/assabet.c | 2 +-
> arch/arm/mm/init.c | 61 ++++++----------
> arch/arm/mm/mmu.c | 115 +++++++++---------------------
> 32 files changed, 133 insertions(+), 284 deletions(-)
(snipped)
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
WARNING: multiple messages have this Message-ID (diff)
From: m.szyprowski@samsung.com (Marek Szyprowski)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2 2/2] arm: Get rid of meminfo
Date: Tue, 11 Feb 2014 10:47:54 +0100 [thread overview]
Message-ID: <52F9F1CA.2070501@samsung.com> (raw)
In-Reply-To: <1391558551-31395-3-git-send-email-lauraa@codeaurora.org>
Hello,
On 2014-02-05 01:02, Laura Abbott wrote:
> memblock is now fully integrated into the kernel and is the prefered
> method for tracking memory. Rather than reinvent the wheel with
> meminfo, migrate to using memblock directly instead of meminfo as
> an intermediate.
>
> Signed-off-by: Laura Abbott <lauraa@codeaurora.org>
This patch fails with SPARSEMEM enabled (tested with v3.14-rc1,
exynos4_defconfig + oldnoconfig):
arch/arm/mm/init.c: In function ?free_unused_memmap?:
arch/arm/mm/init.c:442:10: error: ?prev_bank_end? undeclared (first use
in this function)
arch/arm/mm/init.c:442:10: note: each undeclared identifier is reported
only once for each function it appears in
make[1]: *** [arch/arm/mm/init.o] Error 1
make[1]: *** Waiting for unfinished jobs....
> ---
> arch/arm/include/asm/mach/arch.h | 4 +-
> arch/arm/include/asm/memblock.h | 3 +-
> arch/arm/include/asm/setup.h | 23 ------
> arch/arm/kernel/atags_parse.c | 5 +-
> arch/arm/kernel/setup.c | 30 ++------
> arch/arm/mach-clps711x/board-clep7312.c | 7 +-
> arch/arm/mach-clps711x/board-edb7211.c | 10 +--
> arch/arm/mach-clps711x/board-p720t.c | 2 +-
> arch/arm/mach-footbridge/cats-hw.c | 2 +-
> arch/arm/mach-footbridge/netwinder-hw.c | 2 +-
> arch/arm/mach-msm/board-halibut.c | 6 --
> arch/arm/mach-msm/board-mahimahi.c | 13 +---
> arch/arm/mach-msm/board-msm7x30.c | 3 +-
> arch/arm/mach-msm/board-sapphire.c | 13 ++--
> arch/arm/mach-msm/board-trout.c | 8 +--
> arch/arm/mach-orion5x/common.c | 3 +-
> arch/arm/mach-orion5x/common.h | 3 +-
> arch/arm/mach-pxa/cm-x300.c | 3 +-
> arch/arm/mach-pxa/corgi.c | 10 +--
> arch/arm/mach-pxa/eseries.c | 9 +--
> arch/arm/mach-pxa/poodle.c | 8 +--
> arch/arm/mach-pxa/spitz.c | 9 +--
> arch/arm/mach-pxa/tosa.c | 8 +--
> arch/arm/mach-realview/core.c | 11 +--
> arch/arm/mach-realview/core.h | 3 +-
> arch/arm/mach-realview/realview_pb1176.c | 8 +--
> arch/arm/mach-realview/realview_pbx.c | 17 ++---
> arch/arm/mach-s3c24xx/mach-smdk2413.c | 8 +--
> arch/arm/mach-s3c24xx/mach-vstms.c | 8 +--
> arch/arm/mach-sa1100/assabet.c | 2 +-
> arch/arm/mm/init.c | 61 ++++++----------
> arch/arm/mm/mmu.c | 115 +++++++++---------------------
> 32 files changed, 133 insertions(+), 284 deletions(-)
(snipped)
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
next prev parent reply other threads:[~2014-02-11 9:47 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-05 0:02 [PATCHv2 0/2] Remove ARM meminfo Laura Abbott
2014-02-05 0:02 ` Laura Abbott
2014-02-05 0:02 ` [PATCHv2 1/2] mm/memblock: add memblock_get_current_limit Laura Abbott
2014-02-05 0:02 ` Laura Abbott
2014-02-05 22:01 ` Andrew Morton
2014-02-05 22:01 ` Andrew Morton
2014-02-05 23:19 ` Santosh Shilimkar
2014-02-05 23:19 ` Santosh Shilimkar
2014-02-05 0:02 ` [PATCHv2 2/2] arm: Get rid of meminfo Laura Abbott
2014-02-05 0:02 ` Laura Abbott
2014-02-05 2:00 ` Jason Cooper
2014-02-05 2:00 ` Jason Cooper
2014-02-05 13:43 ` Leif Lindholm
2014-02-05 13:43 ` Leif Lindholm
2014-02-05 23:21 ` Santosh Shilimkar
2014-02-05 23:21 ` Santosh Shilimkar
2014-02-05 23:21 ` Santosh Shilimkar
2014-02-07 2:09 ` Courtney Cavin
2014-02-07 2:09 ` Courtney Cavin
2014-02-07 2:09 ` Courtney Cavin
2014-02-10 3:25 ` Laura Abbott
2014-02-10 3:25 ` Laura Abbott
2014-02-10 3:25 ` Laura Abbott
2014-02-10 10:11 ` Catalin Marinas
2014-02-10 10:11 ` Catalin Marinas
2014-02-10 10:11 ` Catalin Marinas
2014-02-10 17:28 ` Courtney Cavin
2014-02-10 17:28 ` Courtney Cavin
2014-02-10 17:28 ` Courtney Cavin
2014-02-10 21:59 ` Laura Abbott
2014-02-10 21:59 ` Laura Abbott
2014-02-10 21:59 ` Laura Abbott
2014-02-10 21:58 ` Grant Likely
2014-02-10 21:58 ` Grant Likely
2014-02-10 21:58 ` Grant Likely
2014-02-11 9:47 ` Marek Szyprowski [this message]
2014-02-11 9:47 ` Marek Szyprowski
2014-02-05 13:55 ` [PATCHv2 0/2] Remove ARM meminfo Catalin Marinas
2014-02-05 13:55 ` Catalin Marinas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52F9F1CA.2070501@samsung.com \
--to=m.szyprowski@samsung.com \
--cc=andrew@lunn.ch \
--cc=ard.biesheuvel@linaro.org \
--cc=ben-linux@fluff.org \
--cc=catalin.marinas@arm.com \
--cc=davidb@codeaurora.org \
--cc=dwalker@fifo99.com \
--cc=eric.y.miao@gmail.com \
--cc=grygorii.strashko@ti.com \
--cc=haojian.zhuang@gmail.com \
--cc=jason@lakedaemon.net \
--cc=kgene.kim@samsung.com \
--cc=lauraa@codeaurora.org \
--cc=leif.lindholm@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=nicolas.pitre@linaro.org \
--cc=santosh.shilimkar@ti.com \
--cc=sebastian.hesselbarth@gmail.com \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.