From: Christian Marangi <ansuelsmth@gmail.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Russell King <linux@armlinux.org.uk>,
Arnd Bergmann <arnd@arndb.de>,
Andrew Morton <akpm@linux-foundation.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
"Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
Jonathan Corbet <corbet@lwn.net>,
Thomas Gleixner <tglx@linutronix.de>,
Randy Dunlap <rdunlap@infradead.org>,
"Mike Rapoport (IBM)" <rppt@kernel.org>,
Eric DeVolder <eric.devolder@oracle.com>,
Nathan Chancellor <nathan@kernel.org>,
"Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
Kees Cook <keescook@chromium.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
John Crispin <john@phrozen.org>
Subject: Re: [PATCH 2/2] ARM: decompressor: add option to ignore MEM ATAGs
Date: Sat, 20 Jan 2024 17:57:09 +0100 [thread overview]
Message-ID: <65abfb68.050a0220.a9c65.ecbc@mx.google.com> (raw)
In-Reply-To: <CACRpkdavA+OL-ek0JGBRFX=n_H4=j8DbB+nCr7=8htscAzrzFg@mail.gmail.com>
On Sat, Jan 20, 2024 at 12:51:06PM +0100, Linus Walleij wrote:
> On Fri, Jan 19, 2024 at 9:14 PM Christian Marangi <ansuelsmth@gmail.com> wrote:
>
> > Some bootloaders can pass broken MEM ATAGs that provide hardcoded
> > information about mounted RAM size and physical location.
> > Example booloader provide RAM of size 1.7Gb but actual mounted RAM
> > size is 512Mb causing kernel panic.
> >
> > Add option CONFIG_ARM_ATAG_DTB_COMPAT_IGNORE_MEM to ignore these ATAG
> > and not augument appended DTB memory node.
> >
> > Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
>
> So you cannot just ignore all the ATAGs I guess?
> If it's the command line you need, you can pass an identical one in
> chosen.
Ehhh it's not that trivial... Downstream we have a patch that takes just
the bootargs from ATAGs but reality is that only MEM is broken.
Also duplicating the bootargs from bootloader in chosen is problematic
as have tons of device that use cmdlinepart to declare partitions and we
are not aware of the partitions of every device.
And also there are some device that supports dual partition and the
value is provided by the bootloader bootargs so duplicating it would
result in not having a good way to support this.
>
> But if you really need this, it should be there.
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
>
--
Ansuel
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-01-20 16:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-19 20:12 [PATCH 0/2] ARM: decompressor: support AUTO_ZRELADDR and appended DTB Christian Marangi
2024-01-19 20:12 ` [PATCH 1/2] ARM: decompressor: support memory start validation for " Christian Marangi
2024-01-20 11:15 ` Geert Uytterhoeven
2024-01-20 11:48 ` Linus Walleij
2024-01-19 20:12 ` [PATCH 2/2] ARM: decompressor: add option to ignore MEM ATAGs Christian Marangi
2024-01-20 11:51 ` Linus Walleij
2024-01-20 16:57 ` Christian Marangi [this message]
2024-01-20 11:52 ` Linus Walleij
2024-01-20 17:00 ` Christian Marangi
2024-01-21 19:37 ` Linus Walleij
2024-01-20 11:19 ` [PATCH 0/2] ARM: decompressor: support AUTO_ZRELADDR and appended DTB Geert Uytterhoeven
2024-01-20 17:10 ` Christian Marangi
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=65abfb68.050a0220.a9c65.ecbc@mx.google.com \
--to=ansuelsmth@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=andersson@kernel.org \
--cc=arnd@arndb.de \
--cc=corbet@lwn.net \
--cc=eric.devolder@oracle.com \
--cc=geert+renesas@glider.be \
--cc=john@phrozen.org \
--cc=keescook@chromium.org \
--cc=kirill.shutemov@linux.intel.com \
--cc=konrad.dybcio@somainline.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=nathan@kernel.org \
--cc=rdunlap@infradead.org \
--cc=rmk+kernel@armlinux.org.uk \
--cc=rppt@kernel.org \
--cc=tglx@linutronix.de \
/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).