From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] ARM: realview: basic device tree implementation
Date: Thu, 22 May 2014 22:25:23 +0200 [thread overview]
Message-ID: <7651671.1Wa4i4TJsu@wuerfel> (raw)
In-Reply-To: <alpine.LFD.2.11.1405221431370.17310@knanqh.ubzr>
On Thursday 22 May 2014 14:39:20 Nicolas Pitre wrote:
> On Thu, 22 May 2014, Arnd Bergmann wrote:
>
> > On Thursday 22 May 2014 14:09:52 Nicolas Pitre wrote:
> > > On Thu, 22 May 2014, Arnd Bergmann wrote:
> > >
> > > > On Thursday 22 May 2014 15:45:03 Catalin Marinas wrote:
> > > > > By making the DMA memory Strongly Ordered, we avoid the
> > > > > L220 write buffer. In addition to the above commit, we also had
> > > > > 2503a5ecd86c (ARM: 6201/1: RealView: Do not use outer_sync() on
> > > > > ARM11MPCore boards with L220).
> > > >
> > > > Ah, so this also stands in the way of multiplatform realview
> > >
> > > Isn't the real value of a multiplatform realview in the ability to use
> > > it with Qemu? Did Qemu go to the extreme of emulating all those cache
> > > issues?
> >
> > The qemu developers have in the past always argued strongly that they
> > want to run the same kernel that runs on real hardware. Having a kernel
> > that runs only on the emulated version but not on actual hardware is
> > clearly against that goal, and we should use CONFIG_ARCH_VIRT for
> > that purpose.
>
> But the kernel that actually runs on the real hardware also runs on Qemu
> already. It is just that, by some coincidence, a multi-platform kernel
> that pretends to have the same peripherals as a Realview board could
> also be used on Qemu.
>
> Otherwise I really don't see the point in making the code more complex
> just to make a multi-platform kernel run on actual Realview hardware
> that is quite obsolete now and that very few people have access to.
The main reason is that out of principle I want to get to the point where
every ARMv6 and ARMv7 machine is part of CONFIG_ARCH_MULTIPLATFORM.
Then again, once we introduce CONFIG_BROKEN_MULTIPLATFORM, we could
have an option that depends on that, and turns on both
ARM_DMA_MEM_BUFFERABLE as well as the hack to remove the outer_sync
from mb() and wmb() for CONFIG_SMP.
Arnd
next prev parent reply other threads:[~2014-05-22 20:25 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-08 22:08 [PATCH v2] ARM: realview: basic device tree implementation Linus Walleij
2014-05-09 10:44 ` Arnd Bergmann
2014-05-22 7:41 ` Linus Walleij
2014-05-22 8:31 ` Arnd Bergmann
2014-05-22 13:04 ` Linus Walleij
2014-05-22 13:38 ` Arnd Bergmann
2014-05-22 14:45 ` Catalin Marinas
2014-05-22 15:49 ` Arnd Bergmann
2014-05-22 16:09 ` Catalin Marinas
2014-05-22 18:09 ` Nicolas Pitre
2014-05-22 18:26 ` Arnd Bergmann
2014-05-22 18:39 ` Nicolas Pitre
2014-05-22 20:25 ` Arnd Bergmann [this message]
2014-05-22 18:52 ` Linus Walleij
2014-05-22 20:35 ` Arnd Bergmann
2014-05-29 9:38 ` Linus Walleij
2014-05-22 22:34 ` Sebastian Hesselbarth
2014-05-29 9:44 ` Linus Walleij
2014-05-29 10:10 ` Sebastian Hesselbarth
2014-05-29 14:13 ` Linus Walleij
2014-05-29 14:47 ` Sebastian Hesselbarth
2014-05-22 13:17 ` Rob Herring
2014-05-22 13:30 ` Linus Walleij
2014-05-22 13:36 ` Arnd Bergmann
2014-05-22 13:41 ` Jason Cooper
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=7651671.1Wa4i4TJsu@wuerfel \
--to=arnd@arndb.de \
--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