From: robh@kernel.org (Rob Herring)
To: linux-snps-arc@lists.infradead.org
Subject: [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Mon, 5 Nov 2018 14:31:39 -0600 [thread overview]
Message-ID: <20181105203139.GA21852@bogus> (raw)
In-Reply-To: <20181031192843.13230-1-f.fainelli@gmail.com>
On Wed, Oct 31, 2018@12:28:37PM -0700, Florian Fainelli wrote:
> Hi all,
>
> Changes in v3:
>
> - use C conditionals in drivers/of/fdt.c
> - added check on phys_initrd_size in arch/arm64/mm/init.c to determine
> whether initrd_start must be populated
> - fixed a build warning with ARC that was just missing an (unsigned
> long) cast
>
> Changes in v2:
>
> - get rid of ARCH_HAS_PHYS_INITRD and instead define
> phys_initrd_start/phys_initrd_size in init/do_mounts_initrd.c
>
> - make __early_init_dt_declare_initrd() account for ARM64 specific
> behavior with __va() when having CONFIG_DEBUG_VM enabled
>
> - consolidate early_initrd() command line parsing into
> init/do_mounts_initrd.c
>
> Because phys_initrd_start/phys_initrd_size are now compiled in
> ini/do_mounts_initrd.c which is only built with CONFIG_BLK_DEV_INITRD=y,
> we need to be a bit careful about the uses throughout architecture
> specific code.
>
> Previous discussions/submissions list here:
>
> v3:
> https://www.spinics.net/lists/arm-kernel/msg683566.html
> v2:
> https://lkml.org/lkml/2018/10/25/4
>
> Florian Fainelli (6):
> nds32: Remove phys_initrd_start and phys_initrd_size
> arch: Make phys_initrd_start and phys_initrd_size global variables
> of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT
> arm64: Utilize phys_initrd_start/phys_initrd_size
> of/fdt: Remove custom __early_init_dt_declare_initrd() implementation
> arch: Move initrd= parsing into do_mounts_initrd.c
This all looks good to me. I can take it via the DT if you want. I'll
give folks some more time to review though.
Rob
>
> arch/arc/mm/init.c | 25 +++++--------------------
> arch/arm/mm/init.c | 28 ++--------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 33 +++++++--------------------------
> arch/nds32/mm/init.c | 2 --
> arch/unicore32/mm/init.c | 24 +++++-------------------
> drivers/of/fdt.c | 17 ++++++++++++-----
> include/linux/initrd.h | 3 +++
> init/do_mounts_initrd.c | 20 ++++++++++++++++++++
> 9 files changed, 54 insertions(+), 106 deletions(-)
>
> --
> 2.17.1
>
WARNING: multiple messages have this Message-ID (diff)
From: robh@kernel.org (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Mon, 5 Nov 2018 14:31:39 -0600 [thread overview]
Message-ID: <20181105203139.GA21852@bogus> (raw)
In-Reply-To: <20181031192843.13230-1-f.fainelli@gmail.com>
On Wed, Oct 31, 2018 at 12:28:37PM -0700, Florian Fainelli wrote:
> Hi all,
>
> Changes in v3:
>
> - use C conditionals in drivers/of/fdt.c
> - added check on phys_initrd_size in arch/arm64/mm/init.c to determine
> whether initrd_start must be populated
> - fixed a build warning with ARC that was just missing an (unsigned
> long) cast
>
> Changes in v2:
>
> - get rid of ARCH_HAS_PHYS_INITRD and instead define
> phys_initrd_start/phys_initrd_size in init/do_mounts_initrd.c
>
> - make __early_init_dt_declare_initrd() account for ARM64 specific
> behavior with __va() when having CONFIG_DEBUG_VM enabled
>
> - consolidate early_initrd() command line parsing into
> init/do_mounts_initrd.c
>
> Because phys_initrd_start/phys_initrd_size are now compiled in
> ini/do_mounts_initrd.c which is only built with CONFIG_BLK_DEV_INITRD=y,
> we need to be a bit careful about the uses throughout architecture
> specific code.
>
> Previous discussions/submissions list here:
>
> v3:
> https://www.spinics.net/lists/arm-kernel/msg683566.html
> v2:
> https://lkml.org/lkml/2018/10/25/4
>
> Florian Fainelli (6):
> nds32: Remove phys_initrd_start and phys_initrd_size
> arch: Make phys_initrd_start and phys_initrd_size global variables
> of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT
> arm64: Utilize phys_initrd_start/phys_initrd_size
> of/fdt: Remove custom __early_init_dt_declare_initrd() implementation
> arch: Move initrd= parsing into do_mounts_initrd.c
This all looks good to me. I can take it via the DT if you want. I'll
give folks some more time to review though.
Rob
>
> arch/arc/mm/init.c | 25 +++++--------------------
> arch/arm/mm/init.c | 28 ++--------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 33 +++++++--------------------------
> arch/nds32/mm/init.c | 2 --
> arch/unicore32/mm/init.c | 24 +++++-------------------
> drivers/of/fdt.c | 17 ++++++++++++-----
> include/linux/initrd.h | 3 +++
> init/do_mounts_initrd.c | 20 ++++++++++++++++++++
> 9 files changed, 54 insertions(+), 106 deletions(-)
>
> --
> 2.17.1
>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: linux-kernel@vger.kernel.org,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
Frank Rowand <frowand.list@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Marc Zyngier <marc.zyngier@arm.com>,
Russell King <rmk+kernel@armlinux.org.uk>,
Andrey Ryabinin <aryabinin@virtuozzo.com>,
Andrey Konovalov <andreyknvl@google.com>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Robin Murphy <robin.murphy@arm.com>,
Laura Abbott <labbott@redhat.com>, Stefan Agner <stefan@agner.ch>,
Johannes Weiner <hannes@cmpxchg.org>,
Greg Hackmann <ghackmann@android.com>,
Kristina Martsenko <kristina.martsenko@arm.com>,
CHANDAN VN <chandan.vn@samsung.com>,
"moderated list:ARM64 PORT (AARCH64 ARCHITECTURE)"
<linux-arm-kernel@lists.infra>
Subject: Re: [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Mon, 5 Nov 2018 14:31:39 -0600 [thread overview]
Message-ID: <20181105203139.GA21852@bogus> (raw)
In-Reply-To: <20181031192843.13230-1-f.fainelli@gmail.com>
On Wed, Oct 31, 2018 at 12:28:37PM -0700, Florian Fainelli wrote:
> Hi all,
>
> Changes in v3:
>
> - use C conditionals in drivers/of/fdt.c
> - added check on phys_initrd_size in arch/arm64/mm/init.c to determine
> whether initrd_start must be populated
> - fixed a build warning with ARC that was just missing an (unsigned
> long) cast
>
> Changes in v2:
>
> - get rid of ARCH_HAS_PHYS_INITRD and instead define
> phys_initrd_start/phys_initrd_size in init/do_mounts_initrd.c
>
> - make __early_init_dt_declare_initrd() account for ARM64 specific
> behavior with __va() when having CONFIG_DEBUG_VM enabled
>
> - consolidate early_initrd() command line parsing into
> init/do_mounts_initrd.c
>
> Because phys_initrd_start/phys_initrd_size are now compiled in
> ini/do_mounts_initrd.c which is only built with CONFIG_BLK_DEV_INITRD=y,
> we need to be a bit careful about the uses throughout architecture
> specific code.
>
> Previous discussions/submissions list here:
>
> v3:
> https://www.spinics.net/lists/arm-kernel/msg683566.html
> v2:
> https://lkml.org/lkml/2018/10/25/4
>
> Florian Fainelli (6):
> nds32: Remove phys_initrd_start and phys_initrd_size
> arch: Make phys_initrd_start and phys_initrd_size global variables
> of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT
> arm64: Utilize phys_initrd_start/phys_initrd_size
> of/fdt: Remove custom __early_init_dt_declare_initrd() implementation
> arch: Move initrd= parsing into do_mounts_initrd.c
This all looks good to me. I can take it via the DT if you want. I'll
give folks some more time to review though.
Rob
>
> arch/arc/mm/init.c | 25 +++++--------------------
> arch/arm/mm/init.c | 28 ++--------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 33 +++++++--------------------------
> arch/nds32/mm/init.c | 2 --
> arch/unicore32/mm/init.c | 24 +++++-------------------
> drivers/of/fdt.c | 17 ++++++++++++-----
> include/linux/initrd.h | 3 +++
> init/do_mounts_initrd.c | 20 ++++++++++++++++++++
> 9 files changed, 54 insertions(+), 106 deletions(-)
>
> --
> 2.17.1
>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: linux-kernel@vger.kernel.org,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
Frank Rowand <frowand.list@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Marc Zyngier <marc.zyngier@arm.com>,
Russell King <rmk+kernel@armlinux.org.uk>,
Andrey Ryabinin <aryabinin@virtuozzo.com>,
Andrey Konovalov <andreyknvl@google.com>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Robin Murphy <robin.murphy@arm.com>,
Laura Abbott <labbott@redhat.com>, Stefan Agner <stefan@agner.ch>,
Johannes Weiner <hannes@cmpxchg.org>,
Greg Hackmann <ghackmann@android.com>,
Kristina Martsenko <kristina.martsenko@arm.com>,
CHANDAN VN <chandan.vn@samsung.com>,
"moderated list:ARM64 PORT (AARCH64 ARCHITECTURE)"
<linux-arm-kernel@lists.infradead.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE"
<devicetree@vger.kernel.org>,
rppt@linux.ibm.com, linux@armlinux.org.uk, green.hu@gmail.com,
deanbo422@gmail.com, gxt@pku.edu.cn, ard.biesheuvel@linaro.org,
linux-snps-arc@lists.infradead.org, vgupta@synopsys.com
Subject: Re: [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Mon, 5 Nov 2018 14:31:39 -0600 [thread overview]
Message-ID: <20181105203139.GA21852@bogus> (raw)
In-Reply-To: <20181031192843.13230-1-f.fainelli@gmail.com>
On Wed, Oct 31, 2018 at 12:28:37PM -0700, Florian Fainelli wrote:
> Hi all,
>
> Changes in v3:
>
> - use C conditionals in drivers/of/fdt.c
> - added check on phys_initrd_size in arch/arm64/mm/init.c to determine
> whether initrd_start must be populated
> - fixed a build warning with ARC that was just missing an (unsigned
> long) cast
>
> Changes in v2:
>
> - get rid of ARCH_HAS_PHYS_INITRD and instead define
> phys_initrd_start/phys_initrd_size in init/do_mounts_initrd.c
>
> - make __early_init_dt_declare_initrd() account for ARM64 specific
> behavior with __va() when having CONFIG_DEBUG_VM enabled
>
> - consolidate early_initrd() command line parsing into
> init/do_mounts_initrd.c
>
> Because phys_initrd_start/phys_initrd_size are now compiled in
> ini/do_mounts_initrd.c which is only built with CONFIG_BLK_DEV_INITRD=y,
> we need to be a bit careful about the uses throughout architecture
> specific code.
>
> Previous discussions/submissions list here:
>
> v3:
> https://www.spinics.net/lists/arm-kernel/msg683566.html
> v2:
> https://lkml.org/lkml/2018/10/25/4
>
> Florian Fainelli (6):
> nds32: Remove phys_initrd_start and phys_initrd_size
> arch: Make phys_initrd_start and phys_initrd_size global variables
> of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT
> arm64: Utilize phys_initrd_start/phys_initrd_size
> of/fdt: Remove custom __early_init_dt_declare_initrd() implementation
> arch: Move initrd= parsing into do_mounts_initrd.c
This all looks good to me. I can take it via the DT if you want. I'll
give folks some more time to review though.
Rob
>
> arch/arc/mm/init.c | 25 +++++--------------------
> arch/arm/mm/init.c | 28 ++--------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 33 +++++++--------------------------
> arch/nds32/mm/init.c | 2 --
> arch/unicore32/mm/init.c | 24 +++++-------------------
> drivers/of/fdt.c | 17 ++++++++++++-----
> include/linux/initrd.h | 3 +++
> init/do_mounts_initrd.c | 20 ++++++++++++++++++++
> 9 files changed, 54 insertions(+), 106 deletions(-)
>
> --
> 2.17.1
>
next prev parent reply other threads:[~2018-11-05 20:31 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-31 19:28 [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd() Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` [PATCH v3 1/6] nds32: Remove phys_initrd_start and phys_initrd_size Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` [PATCH v3 2/6] arch: Make phys_initrd_start and phys_initrd_size global variables Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` [PATCH v3 3/6] of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` [PATCH v3 4/6] arm64: Utilize phys_initrd_start/phys_initrd_size Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-11-05 20:33 ` Rob Herring
2018-11-05 20:33 ` Rob Herring
2018-11-05 20:33 ` Rob Herring
2018-11-05 20:33 ` Rob Herring
2018-11-05 20:39 ` Ard Biesheuvel
2018-11-05 20:39 ` Ard Biesheuvel
2018-11-05 20:39 ` Ard Biesheuvel
2018-11-05 20:39 ` Ard Biesheuvel
2018-11-05 20:41 ` Florian Fainelli
2018-11-05 20:41 ` Florian Fainelli
2018-11-05 20:41 ` Florian Fainelli
2018-11-05 20:41 ` Florian Fainelli
2018-11-05 20:44 ` Ard Biesheuvel
2018-11-05 20:44 ` Ard Biesheuvel
2018-11-05 20:44 ` Ard Biesheuvel
2018-11-05 20:44 ` Ard Biesheuvel
2018-11-05 20:51 ` Florian Fainelli
2018-11-05 20:51 ` Florian Fainelli
2018-11-05 20:51 ` Florian Fainelli
2018-11-05 20:51 ` Florian Fainelli
2018-11-05 21:00 ` Ard Biesheuvel
2018-11-05 21:00 ` Ard Biesheuvel
2018-11-05 21:00 ` Ard Biesheuvel
2018-11-05 21:00 ` Ard Biesheuvel
2018-11-05 21:05 ` Florian Fainelli
2018-11-05 21:05 ` Florian Fainelli
2018-11-05 21:05 ` Florian Fainelli
2018-11-05 21:05 ` Florian Fainelli
2018-11-05 21:07 ` Ard Biesheuvel
2018-11-05 21:07 ` Ard Biesheuvel
2018-11-05 21:07 ` Ard Biesheuvel
2018-11-05 21:07 ` Ard Biesheuvel
2018-10-31 19:28 ` [PATCH v3 5/6] of/fdt: Remove custom __early_init_dt_declare_initrd() implementation Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` [PATCH v3 6/6] arch: Move initrd= parsing into do_mounts_initrd.c Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-10-31 19:28 ` Florian Fainelli
2018-11-05 20:31 ` Rob Herring [this message]
2018-11-05 20:31 ` [PATCH v3 0/6] arm64: Get rid of __early_init_dt_declare_initrd() Rob Herring
2018-11-05 20:31 ` Rob Herring
2018-11-05 20:31 ` Rob Herring
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=20181105203139.GA21852@bogus \
--to=robh@kernel.org \
--cc=linux-snps-arc@lists.infradead.org \
/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.