From: Souptick Joarder <jrdr.linux@gmail.com>
To: Christophe Leroy <christophe.leroy@csgroup.eu>
Cc: Kefeng Wang <wangkefeng.wang@huawei.com>,
linux-kernel@vger.kernel.org, Linux-MM <linux-mm@kvack.org>,
Andrew Morton <akpm@linux-foundation.org>,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH v3 resend 11/15] powerpc: convert to setup_initial_init_mm()
Date: Tue, 8 Jun 2021 20:41:47 +0530 [thread overview]
Message-ID: <CAFqt6zbk3rYPx9Qa+pZ3Q5XNLfQZ4w01CCGuq7LYhVYDwweYpw@mail.gmail.com> (raw)
In-Reply-To: <215985fd-67d5-731f-743c-ea446b55bb8d@csgroup.eu>
On Tue, Jun 8, 2021 at 8:24 PM Christophe Leroy
<christophe.leroy@csgroup.eu> wrote:
>
>
>
> Le 08/06/2021 à 16:36, Souptick Joarder a écrit :
> > On Tue, Jun 8, 2021 at 1:56 PM Kefeng Wang <wangkefeng.wang@huawei.com> wrote:
> >>
> >> Use setup_initial_init_mm() helper to simplify code.
> >>
> >> Note klimit is (unsigned long) _end, with new helper,
> >> will use _end directly.
> >
> > With this change klimit left with no user in this file and can be
> > moved to some appropriate header.
> > But in a separate series.
>
> I have a patch to remove klimit, see
> https://patchwork.ozlabs.org/project/linuxppc-dev/patch/9fa9ba6807c17f93f35a582c199c646c4a8bfd9c.1622800638.git.christophe.leroy@csgroup.eu/
Got it. Thanks :)
>
> Christophe
>
>
> >
> >>
> >> Cc: Michael Ellerman <mpe@ellerman.id.au>
> >> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> >> Cc: linuxppc-dev@lists.ozlabs.org
> >> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
> >> ---
> >> arch/powerpc/kernel/setup-common.c | 5 +----
> >> 1 file changed, 1 insertion(+), 4 deletions(-)
> >>
> >> diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup-common.c
> >> index 74a98fff2c2f..96697c6e1e16 100644
> >> --- a/arch/powerpc/kernel/setup-common.c
> >> +++ b/arch/powerpc/kernel/setup-common.c
> >> @@ -927,10 +927,7 @@ void __init setup_arch(char **cmdline_p)
> >>
> >> klp_init_thread_info(&init_task);
> >>
> >> - init_mm.start_code = (unsigned long)_stext;
> >> - init_mm.end_code = (unsigned long) _etext;
> >> - init_mm.end_data = (unsigned long) _edata;
> >> - init_mm.brk = klimit;
> >> + setup_initial_init_mm(_stext, _etext, _edata, _end);
> >>
> >> mm_iommu_init(&init_mm);
> >> irqstack_early_init();
> >> --
> >> 2.26.2
> >>
> >>
next prev parent reply other threads:[~2021-06-08 15:12 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-08 8:34 [PATCH v3 resend 00/15] init_mm: cleanup ARCH's text/data/brk setup code Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 01/15] mm: add setup_initial_init_mm() helper Kefeng Wang
2021-06-08 14:53 ` Souptick Joarder
2021-06-08 14:57 ` Christophe Leroy
2021-06-08 15:14 ` Souptick Joarder
2021-06-08 8:34 ` [PATCH v3 resend 02/15] arc: convert to setup_initial_init_mm() Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 03/15] arm: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 04/15] arm64: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 05/15] csky: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 06/15] h8300: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 07/15] m68k: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 08/15] nds32: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 09/15] nios2: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 10/15] openrisc: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 11/15] powerpc: " Kefeng Wang
2021-06-08 14:36 ` Souptick Joarder
2021-06-08 14:54 ` Christophe Leroy
2021-06-08 15:11 ` Souptick Joarder [this message]
2021-06-08 8:34 ` [PATCH v3 resend 12/15] riscv: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 13/15] s390: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 14/15] sh: " Kefeng Wang
2021-06-08 8:34 ` [PATCH v3 resend 15/15] x86: " Kefeng Wang
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=CAFqt6zbk3rYPx9Qa+pZ3Q5XNLfQZ4w01CCGuq7LYhVYDwweYpw@mail.gmail.com \
--to=jrdr.linux@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=christophe.leroy@csgroup.eu \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=wangkefeng.wang@huawei.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).