From: catalin.marinas@arm.com (Catalin Marinas)
To: linux-arm-kernel@lists.infradead.org
Subject: [Linaro-acpi] [PATCH v7 04/17] ARM64 / ACPI: Introduce early_param for "acpi" and pass acpi=force to enable ACPI
Date: Thu, 29 Jan 2015 23:11:20 +0000 [thread overview]
Message-ID: <20150129231120.GA23786@e104818-lin.cambridge.arm.com> (raw)
In-Reply-To: <54CA7F94.6000305@redhat.com>
On Thu, Jan 29, 2015 at 06:44:36PM +0000, Jon Masters wrote:
> On 01/29/2015 01:34 PM, Timur Tabi wrote:
> > On 01/29/2015 12:28 PM, Ard Biesheuvel wrote:
> >> The UEFI stub in the kernel uses the DTB file format (FDT) to pass
> >> information about the UEFI memory map and system table to the kernel.
> >> It does so even if there is no device tree that describes the
> >> platform. In this case, the file only contains a /chosen DT node, and
> >> nothing else, and it is up to the kernel to figure out that it can ask
> >> UEFI for a set of ACPI tables that it can use instead to configure the
> >> system. Otherwise, the /chosen node properties are added to a device
> >> tree that contains the full platform description.
> >>
> >> The problem is that we have to decide how to distinguish a
> >> conventional device tree DTB from a DTB that only exists to
> >> communicate the UEFI entry points.
> >
> > Ah, that's exactly what I'm seeing. The UEFI stub in our kernel
> > generates a DTB, and therefore I always need to put acpi=force on our
> > kernel command line.
To Timur: that's because you use a set of patches that are still under
development and not yet agreed as being upstream ready. There is a
sub-thread on this topic and even a patch from Ard on how EFI stub can
tell the kernel whether DT as any SoC description or not. This needs
further discussion since similar feature is needed by kexec and Xen.
> I expect some of the distros to patch ACPI always enabled. So from my
> point of view this affects only those wanting to follow upstream.
Sorry Jon but statements like this make me wonder whether we should
simply let the whole ARM ACPI be an out of tree distro business. We
spend a long time discussing OS-agnostic firmware implementation,
planning mini-summits, just to get certain Linux distro representative
stating that the kernel-firmware interface we discuss here only matters
for those planning to follow upstream. Certain Linux distros will play
by other rules.
I'm trying to get some consensus here, coming with arguments why DT
has priority over ACPI while still allowing ACPI-only firmware and you
pretty much state that vendors picking a distro kernel rather than
mainline don't need to bother. How does this work with Red Hat's stand
on upstream first? Not having ACPI in mainline yet is not an excuse;
there have been reasonable technical arguments and it's now a matter of
time until they (well, part of them) are sorted, nothing political
(what's political is distros patching the kernel to disable DT booting
on purpose).
--
Catalin
next prev parent reply other threads:[~2015-01-29 23:11 UTC|newest]
Thread overview: 156+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-14 15:04 [PATCH v7 00/17] Introduce ACPI for ARM64 based on ACPI 5.1 Hanjun Guo
2015-01-14 15:04 ` [PATCH v7 01/17] arm64: allow late use of early_ioremap Hanjun Guo
2015-01-15 18:44 ` Mark Langsdorf
2015-01-14 15:04 ` [PATCH v7 02/17] ARM64 / ACPI: Get RSDP and ACPI boot-time tables Hanjun Guo
2015-01-15 18:45 ` Mark Langsdorf
2015-01-14 15:04 ` [PATCH v7 03/17] ARM64 / ACPI: Introduce sleep-arm.c Hanjun Guo
2015-01-15 18:45 ` Mark Langsdorf
2015-01-14 15:04 ` [PATCH v7 04/17] ARM64 / ACPI: Introduce early_param for "acpi" and pass acpi=force to enable ACPI Hanjun Guo
2015-01-15 18:46 ` Mark Langsdorf
2015-01-19 11:42 ` Catalin Marinas
2015-01-19 11:55 ` Ard Biesheuvel
2015-01-19 13:51 ` Catalin Marinas
2015-01-19 14:00 ` Ard Biesheuvel
2015-01-19 14:22 ` Catalin Marinas
2015-01-19 15:13 ` Grant Likely
2015-01-19 16:59 ` Jon Masters
2015-01-19 17:52 ` Catalin Marinas
2015-01-19 18:01 ` Mark Rutland
2015-01-20 9:29 ` Hanjun Guo
2015-01-20 10:56 ` Catalin Marinas
2015-01-20 11:10 ` Mark Rutland
2015-01-20 12:17 ` Hanjun Guo
2015-01-20 12:31 ` Leif Lindholm
2015-01-20 19:20 ` Stefano Stabellini
2015-01-21 9:43 ` Parth Dixit
2015-01-21 15:23 ` Catalin Marinas
2015-01-21 15:29 ` Jon Masters
2015-01-21 15:42 ` Catalin Marinas
2015-01-21 15:56 ` Graeme Gregory
2015-01-21 16:05 ` Jon Masters
2015-01-21 16:16 ` Catalin Marinas
2015-01-21 16:51 ` Parth Dixit
2015-01-21 16:10 ` Stefano Stabellini
2015-01-22 12:29 ` Daniel Kiper
2015-01-28 17:58 ` [Linaro-acpi] " Timur Tabi
2015-01-28 18:08 ` Catalin Marinas
2015-01-28 18:08 ` Timur Tabi
2015-01-28 18:14 ` Catalin Marinas
2015-01-28 18:18 ` Timur Tabi
2015-01-29 15:19 ` Catalin Marinas
2015-01-29 18:20 ` Ard Biesheuvel
2015-01-29 18:21 ` Timur Tabi
2015-01-29 18:28 ` Ard Biesheuvel
2015-01-29 18:34 ` Timur Tabi
2015-01-29 18:44 ` Jon Masters
2015-01-29 23:11 ` Catalin Marinas [this message]
2015-01-29 23:16 ` Jon Masters
2015-01-29 23:30 ` Catalin Marinas
2015-01-30 11:13 ` Catalin Marinas
2015-01-30 14:48 ` Timur Tabi
2015-01-30 15:12 ` Ard Biesheuvel
2015-01-14 15:04 ` [PATCH v7 05/17] ARM64 / ACPI: If we chose to boot from acpi then disable FDT Hanjun Guo
2015-01-15 18:46 ` Mark Langsdorf
2015-01-19 11:45 ` Catalin Marinas
2015-01-14 15:04 ` [PATCH v7 06/17] ARM64 / ACPI: Make PCI optional for ACPI on ARM64 Hanjun Guo
2015-01-15 18:46 ` Mark Langsdorf
2015-01-16 9:49 ` Catalin Marinas
2015-01-18 6:25 ` Hanjun Guo
2015-01-18 6:31 ` Jon Masters
2015-01-18 6:46 ` Hanjun Guo
2015-01-18 9:29 ` Graeme Gregory
2015-01-18 12:32 ` Jon Masters
2015-01-19 4:26 ` Hanjun Guo
2015-01-19 10:37 ` Catalin Marinas
2015-01-19 10:42 ` Catalin Marinas
2015-01-20 2:39 ` Hanjun Guo
2015-01-20 11:00 ` Catalin Marinas
2015-01-20 11:56 ` Hanjun Guo
2015-01-20 12:26 ` [Linaro-acpi] " Tomasz Nowicki
2015-01-20 15:10 ` Catalin Marinas
2015-01-14 15:04 ` [PATCH v7 07/17] ARM64 / ACPI: Disable ACPI if FADT revision is less than 5.1 Hanjun Guo
2015-01-15 18:47 ` Mark Langsdorf
2015-01-16 14:33 ` Lorenzo Pieralisi
2015-01-18 5:49 ` Hanjun Guo
2015-01-19 11:50 ` Catalin Marinas
2015-01-20 3:05 ` Hanjun Guo
2015-01-14 15:04 ` [PATCH v7 08/17] ARM64 / ACPI: Get PSCI flags in FADT for PSCI init Hanjun Guo
2015-01-15 18:47 ` Mark Langsdorf
2015-01-14 15:04 ` [PATCH v7 09/17] ACPI / table: Print GIC information when MADT is parsed Hanjun Guo
2015-01-15 18:47 ` Mark Langsdorf
2015-01-14 15:04 ` [PATCH v7 10/17] ARM64 / ACPI: Parse MADT for SMP initialization Hanjun Guo
2015-01-15 18:48 ` Mark Langsdorf
2015-01-16 18:18 ` Lorenzo Pieralisi
2015-01-20 13:09 ` Hanjun Guo
2015-01-20 15:16 ` Lorenzo Pieralisi
2015-01-14 15:04 ` [PATCH v7 11/17] ACPI / processor: Make it possible to get CPU hardware ID via GICC Hanjun Guo
2015-01-15 18:48 ` Mark Langsdorf
2015-01-20 11:17 ` Catalin Marinas
2015-01-20 12:26 ` Hanjun Guo
2015-01-20 16:16 ` Lorenzo Pieralisi
2015-01-14 15:05 ` [PATCH v7 12/17] ARM64 / ACPI: Introduce ACPI_IRQ_MODEL_GIC and register device's gsi Hanjun Guo
2015-01-15 18:48 ` Mark Langsdorf
2015-01-16 10:45 ` Marc Zyngier
2015-01-14 15:05 ` [PATCH v7 13/17] ARM64 / ACPI: Add GICv2 specific ACPI boot support Hanjun Guo
2015-01-15 18:50 ` Mark Langsdorf
2015-01-16 11:15 ` Marc Zyngier
2015-01-16 13:54 ` Grant Likely
2015-01-16 14:37 ` Marc Zyngier
2015-01-22 12:46 ` Hanjun Guo
2015-01-22 14:46 ` Marc Zyngier
2015-01-23 9:38 ` Hanjun Guo
2015-01-27 16:12 ` Grant Likely
2015-01-29 15:29 ` Catalin Marinas
2015-01-29 16:06 ` Tomasz Nowicki
2015-01-20 10:40 ` Tomasz Nowicki
2015-01-20 13:05 ` Jon Masters
2015-01-14 15:05 ` [PATCH v7 14/17] ARM64 / ACPI: Parse GTDT to initialize arch timer Hanjun Guo
2015-01-15 18:50 ` Mark Langsdorf
2015-01-14 15:05 ` [PATCH v7 15/17] ARM64 / ACPI: Select ACPI_REDUCED_HARDWARE_ONLY if ACPI is enabled on ARM64 Hanjun Guo
2015-01-15 18:50 ` Mark Langsdorf
2015-01-14 15:05 ` [PATCH v7 16/17] ARM64 / ACPI: Enable ARM64 in Kconfig Hanjun Guo
2015-01-15 18:50 ` Mark Langsdorf
2015-01-14 15:05 ` [PATCH v7 17/17] Documentation: ACPI for ARM64 Hanjun Guo
2015-01-15 18:54 ` Mark Langsdorf
2015-01-15 16:26 ` [PATCH v7 00/17] Introduce ACPI for ARM64 based on ACPI 5.1 Grant Likely
2015-01-15 18:23 ` Catalin Marinas
2015-01-15 19:02 ` Mark Brown
2015-01-15 20:04 ` Jason Cooper
2015-01-15 20:31 ` Mark Brown
2015-01-15 20:51 ` Jason Cooper
2015-01-16 11:49 ` Mark Brown
2015-01-16 7:24 ` Hanjun Guo
2015-01-16 10:10 ` Catalin Marinas
2015-01-16 12:05 ` Mark Brown
2015-01-16 12:29 ` Will Deacon
2015-01-16 16:54 ` Mark Brown
2015-01-18 6:36 ` Hanjun Guo
2015-01-15 21:31 ` Al Stone
2015-01-15 21:38 ` Jon Masters
2015-01-16 10:20 ` Catalin Marinas
2015-01-16 15:17 ` [Linaro-acpi] " Al Stone
2015-01-16 15:23 ` Al Stone
2015-01-16 15:44 ` Suravee Suthikulpanit
2015-01-16 7:17 ` Hanjun Guo
2015-01-16 10:04 ` Catalin Marinas
2015-01-16 14:45 ` Tom Lendacky
2015-01-16 14:55 ` Will Deacon
2015-01-16 15:14 ` Arnd Bergmann
2015-01-16 15:25 ` Catalin Marinas
2015-01-16 15:33 ` Will Deacon
2015-01-16 15:40 ` Arnd Bergmann
2015-01-16 15:43 ` [Linaro-acpi] " Arnd Bergmann
2015-01-16 15:49 ` Will Deacon
2015-01-16 15:53 ` [Linaro-acpi] " Arnd Bergmann
2015-01-17 17:53 ` Rob Herring
2015-01-16 17:12 ` Tom Lendacky
2015-01-16 15:16 ` Tom Lendacky
2015-01-16 16:29 ` Grant Likely
2015-01-16 17:20 ` [Linaro-acpi] " Arnd Bergmann
2015-01-17 11:52 ` Catalin Marinas
2015-01-15 18:58 ` Jon Masters
2015-01-15 19:49 ` Mark Langsdorf
2015-01-16 8:37 ` Hanjun Guo
2015-01-15 21:33 ` Suravee Suthikulanit
2015-01-27 17:46 ` Timur Tabi
2015-01-28 13:53 ` Hanjun Guo
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=20150129231120.GA23786@e104818-lin.cambridge.arm.com \
--to=catalin.marinas@arm.com \
--cc=linux-arm-kernel@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 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).