From: rppt@linux.ibm.com (Mike Rapoport)
To: linux-snps-arc@lists.infradead.org
Subject: [PATCH v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Tue, 6 Nov 2018 16:06:42 +0200 [thread overview]
Message-ID: <20181106140642.GA19280@rapoport-lnx> (raw)
In-Reply-To: <20181105225431.24485-1-f.fainelli@gmail.com>
On Mon, Nov 05, 2018@02:54:25PM -0800, Florian Fainelli wrote:
> Hi all,
>
> Changes in v4:
>
> - dropped initrd_below_start_ok assignment in ARM64, not necessary at
> all (Ard)
> - replace #ifdef CONFIG_BLK_DEV_INITRD with if
> (IS_ENABLED(CONFIG_BLK_DEV_INITRD) for consistency with other parts
> of arm64_memblock_init() (Rob)
>
> 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
For the series:
Reviewed-by: Mike Rapoport <rppt at linux.ibm.com>
> arch/arc/mm/init.c | 25 +++++-------------------
> arch/arm/mm/init.c | 28 ++-------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 34 ++++++---------------------------
> 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, 53 insertions(+), 108 deletions(-)
>
> --
> 2.17.1
>
--
Sincerely yours,
Mike.
WARNING: multiple messages have this Message-ID (diff)
From: rppt@linux.ibm.com (Mike Rapoport)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Tue, 6 Nov 2018 16:06:42 +0200 [thread overview]
Message-ID: <20181106140642.GA19280@rapoport-lnx> (raw)
In-Reply-To: <20181105225431.24485-1-f.fainelli@gmail.com>
On Mon, Nov 05, 2018 at 02:54:25PM -0800, Florian Fainelli wrote:
> Hi all,
>
> Changes in v4:
>
> - dropped initrd_below_start_ok assignment in ARM64, not necessary at
> all (Ard)
> - replace #ifdef CONFIG_BLK_DEV_INITRD with if
> (IS_ENABLED(CONFIG_BLK_DEV_INITRD) for consistency with other parts
> of arm64_memblock_init() (Rob)
>
> 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
For the series:
Reviewed-by: Mike Rapoport <rppt@linux.ibm.com>
> arch/arc/mm/init.c | 25 +++++-------------------
> arch/arm/mm/init.c | 28 ++-------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 34 ++++++---------------------------
> 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, 53 insertions(+), 108 deletions(-)
>
> --
> 2.17.1
>
--
Sincerely yours,
Mike.
WARNING: multiple messages have this Message-ID (diff)
From: Mike Rapoport <rppt@linux.ibm.com>
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>,
Rob Herring <robh+dt@kernel.org>,
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 ARCHITECTUR
Subject: Re: [PATCH v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Tue, 6 Nov 2018 16:06:42 +0200 [thread overview]
Message-ID: <20181106140642.GA19280@rapoport-lnx> (raw)
In-Reply-To: <20181105225431.24485-1-f.fainelli@gmail.com>
On Mon, Nov 05, 2018 at 02:54:25PM -0800, Florian Fainelli wrote:
> Hi all,
>
> Changes in v4:
>
> - dropped initrd_below_start_ok assignment in ARM64, not necessary at
> all (Ard)
> - replace #ifdef CONFIG_BLK_DEV_INITRD with if
> (IS_ENABLED(CONFIG_BLK_DEV_INITRD) for consistency with other parts
> of arm64_memblock_init() (Rob)
>
> 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
For the series:
Reviewed-by: Mike Rapoport <rppt@linux.ibm.com>
> arch/arc/mm/init.c | 25 +++++-------------------
> arch/arm/mm/init.c | 28 ++-------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 34 ++++++---------------------------
> 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, 53 insertions(+), 108 deletions(-)
>
> --
> 2.17.1
>
--
Sincerely yours,
Mike.
WARNING: multiple messages have this Message-ID (diff)
From: Mike Rapoport <rppt@linux.ibm.com>
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>,
Rob Herring <robh+dt@kernel.org>,
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>,
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 v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd()
Date: Tue, 6 Nov 2018 16:06:42 +0200 [thread overview]
Message-ID: <20181106140642.GA19280@rapoport-lnx> (raw)
In-Reply-To: <20181105225431.24485-1-f.fainelli@gmail.com>
On Mon, Nov 05, 2018 at 02:54:25PM -0800, Florian Fainelli wrote:
> Hi all,
>
> Changes in v4:
>
> - dropped initrd_below_start_ok assignment in ARM64, not necessary at
> all (Ard)
> - replace #ifdef CONFIG_BLK_DEV_INITRD with if
> (IS_ENABLED(CONFIG_BLK_DEV_INITRD) for consistency with other parts
> of arm64_memblock_init() (Rob)
>
> 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
For the series:
Reviewed-by: Mike Rapoport <rppt@linux.ibm.com>
> arch/arc/mm/init.c | 25 +++++-------------------
> arch/arm/mm/init.c | 28 ++-------------------------
> arch/arm64/include/asm/memory.h | 8 --------
> arch/arm64/mm/init.c | 34 ++++++---------------------------
> 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, 53 insertions(+), 108 deletions(-)
>
> --
> 2.17.1
>
--
Sincerely yours,
Mike.
next prev parent reply other threads:[~2018-11-06 14:06 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-05 22:54 [PATCH v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd() Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` [PATCH v4 1/6] nds32: Remove phys_initrd_start and phys_initrd_size Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` [PATCH v4 2/6] arch: Make phys_initrd_start and phys_initrd_size global variables Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` [PATCH v4 3/6] of/fdt: Populate phys_initrd_start/phys_initrd_size from FDT Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` [PATCH v4 4/6] arm64: Utilize phys_initrd_start/phys_initrd_size Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-15 17:45 ` Will Deacon
2018-11-15 17:45 ` Will Deacon
2018-11-15 17:45 ` Will Deacon
2018-11-15 17:45 ` Will Deacon
2018-11-05 22:54 ` [PATCH v4 5/6] of/fdt: Remove custom __early_init_dt_declare_initrd() implementation Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` [PATCH v4 6/6] arch: Move initrd= parsing into do_mounts_initrd.c Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-05 22:54 ` Florian Fainelli
2018-11-13 0:34 ` Vineet Gupta
2018-11-13 0:34 ` Vineet Gupta
2018-11-13 0:34 ` Vineet Gupta
2018-11-13 0:34 ` Vineet Gupta
2018-11-13 0:37 ` Florian Fainelli
2018-11-13 0:37 ` Florian Fainelli
2018-11-13 0:37 ` Florian Fainelli
2018-11-13 0:37 ` Florian Fainelli
2018-11-13 0:40 ` Vineet Gupta
2018-11-13 0:40 ` Vineet Gupta
2018-11-13 0:40 ` Vineet Gupta
2018-11-13 0:40 ` Vineet Gupta
2018-11-13 0:52 ` Florian Fainelli
2018-11-13 0:52 ` Florian Fainelli
2018-11-13 0:52 ` Florian Fainelli
2018-11-13 0:52 ` Florian Fainelli
2018-11-13 0:57 ` Vineet Gupta
2018-11-13 0:57 ` Vineet Gupta
2018-11-13 0:57 ` Vineet Gupta
2018-11-13 0:57 ` Vineet Gupta
2018-11-15 19:35 ` Florian Fainelli
2018-11-15 19:35 ` Florian Fainelli
2018-11-15 19:35 ` Florian Fainelli
2018-11-15 19:35 ` Florian Fainelli
2018-11-06 14:06 ` Mike Rapoport [this message]
2018-11-06 14:06 ` [PATCH v4 0/6] arm64: Get rid of __early_init_dt_declare_initrd() Mike Rapoport
2018-11-06 14:06 ` Mike Rapoport
2018-11-06 14:06 ` Mike Rapoport
2018-11-12 20:32 ` Florian Fainelli
2018-11-12 20:32 ` Florian Fainelli
2018-11-12 20:32 ` Florian Fainelli
2018-11-12 20:32 ` Florian Fainelli
2018-11-13 0:22 ` Rob Herring
2018-11-13 0:22 ` Rob Herring
2018-11-13 0:22 ` Rob Herring
2018-11-13 0:22 ` Rob Herring
2018-11-26 21:56 ` Rob Herring
2018-11-26 21:56 ` Rob Herring
2018-11-26 21:56 ` Rob Herring
2018-11-26 21:56 ` 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=20181106140642.GA19280@rapoport-lnx \
--to=rppt@linux.ibm.com \
--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.