All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: XIP_KERNEL and !ARCH_MULTIPLATFORM
Date: Sun, 22 Mar 2015 19:40:55 +0000	[thread overview]
Message-ID: <201503222040.56105.arnd@arndb.de> (raw)
In-Reply-To: <HK2PR06MB05611E6828E494FB938179658A0F0@HK2PR06MB0561.apcprd06.prod.outlook.com>

On Saturday 21 March 2015, Chris Brandt wrote:
> > At first glance, HIGHMEM shouldn't be affected by XIP_KERNEL at all.
> 
> I really didn't find the root cause of this. I just remember looking at the crashed log and it seemed like removing HIGHMEM would fix it....and it did (can't remember the details).
> 
> > However, SMP_ON_UP is incompatible with XIP_KERNEL (as already listed in its dependencies) and building with SMP=y without SMP_ON_UP, and trying to run on UP hardware will definitely fail.
> 
> So, it sounds like you are saying XIP_KERNEL and SMP is not going to happen.
> 
> However, you said:
> > To put that another way:
> > 1. SMP_ON_UP relies on being able to change the kernel text, which it
> >    can't do with an XIP kernel.
> 
> But, what kernel text are we talking about? All of it, or just certain parts?
> What if I changed the linker to put all those files in a specific area and
> then remap that one area to RAM (of course and copy the binary too). Yes,
> sounds pretty hack-ish, but if XIP can't go past single core, it might not
> have a long term future anyway.

XIP should be fine on ARMv7 SMP, which does not require SMP_ON_UP.

Maybe it's enough to describe the dependency in the opposite way and make
XIP_KERNEL depend on !SMP_ON_UP. You should never really need the two
together anyway, since SMP_ON_UP is only needed for kernels that are
meant to run on SMP machines with full SMP support as well as legacy
ARMv6 machines without SMP instructions.
 

	Arnd

WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: XIP_KERNEL and !ARCH_MULTIPLATFORM
Date: Sun, 22 Mar 2015 20:40:55 +0100	[thread overview]
Message-ID: <201503222040.56105.arnd@arndb.de> (raw)
In-Reply-To: <HK2PR06MB05611E6828E494FB938179658A0F0@HK2PR06MB0561.apcprd06.prod.outlook.com>

On Saturday 21 March 2015, Chris Brandt wrote:
> > At first glance, HIGHMEM shouldn't be affected by XIP_KERNEL at all.
> 
> I really didn't find the root cause of this. I just remember looking at the crashed log and it seemed like removing HIGHMEM would fix it....and it did (can't remember the details).
> 
> > However, SMP_ON_UP is incompatible with XIP_KERNEL (as already listed in its dependencies) and building with SMP=y without SMP_ON_UP, and trying to run on UP hardware will definitely fail.
> 
> So, it sounds like you are saying XIP_KERNEL and SMP is not going to happen.
> 
> However, you said:
> > To put that another way:
> > 1. SMP_ON_UP relies on being able to change the kernel text, which it
> >    can't do with an XIP kernel.
> 
> But, what kernel text are we talking about? All of it, or just certain parts?
> What if I changed the linker to put all those files in a specific area and
> then remap that one area to RAM (of course and copy the binary too). Yes,
> sounds pretty hack-ish, but if XIP can't go past single core, it might not
> have a long term future anyway.

XIP should be fine on ARMv7 SMP, which does not require SMP_ON_UP.

Maybe it's enough to describe the dependency in the opposite way and make
XIP_KERNEL depend on !SMP_ON_UP. You should never really need the two
together anyway, since SMP_ON_UP is only needed for kernels that are
meant to run on SMP machines with full SMP support as well as legacy
ARMv6 machines without SMP instructions.
 

	Arnd

  parent reply	other threads:[~2015-03-22 19:40 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-17  3:48 XIP_KERNEL and !ARCH_MULTIPLATFORM Chris Brandt
2015-03-17  3:48 ` Chris Brandt
2015-03-17 10:26 ` Russell King - ARM Linux
2015-03-17 10:26   ` Russell King - ARM Linux
2015-03-17 13:29   ` Chris Brandt
2015-03-17 13:29     ` Chris Brandt
2015-03-17 10:50 ` Geert Uytterhoeven
2015-03-17 10:50   ` Geert Uytterhoeven
2015-03-17 13:46   ` Chris Brandt
2015-03-17 13:46     ` Chris Brandt
2015-03-17 13:57     ` Geert Uytterhoeven
2015-03-17 13:57       ` Geert Uytterhoeven
2015-03-17 16:08       ` Chris Brandt
2015-03-17 16:08         ` Chris Brandt
2015-03-17 16:20         ` Geert Uytterhoeven
2015-03-17 16:20           ` Geert Uytterhoeven
2015-03-17 16:28           ` Russell King - ARM Linux
2015-03-17 16:28             ` Russell King - ARM Linux
2015-03-17 16:37             ` Chris Brandt
2015-03-17 16:37               ` Chris Brandt
2015-03-17 16:34           ` Chris Brandt
2015-03-17 16:34             ` Chris Brandt
2015-03-19 20:57           ` Chris Brandt
2015-03-19 20:57             ` Chris Brandt
2015-03-20  8:19             ` Uwe Kleine-König
2015-03-20  8:19               ` Uwe Kleine-König
2015-03-20 20:18               ` Chris Brandt
2015-03-20 20:18                 ` Chris Brandt
2015-03-22  9:13                 ` Uwe Kleine-König
2015-03-22  9:13                   ` Uwe Kleine-König
2015-03-20 22:23             ` Russell King - ARM Linux
2015-03-20 22:23               ` Russell King - ARM Linux
2015-03-21 15:39               ` Chris Brandt
2015-03-21 15:39                 ` Chris Brandt
2015-03-22 16:57                 ` Russell King - ARM Linux
2015-03-22 16:57                   ` Russell King - ARM Linux
2015-03-22 19:48                   ` Arnd Bergmann
2015-03-22 19:48                     ` Arnd Bergmann
2015-03-23  1:41                     ` Chris Brandt
2015-03-23  1:41                       ` Chris Brandt
2015-03-23  1:24                   ` Chris Brandt
2015-03-23  1:24                     ` Chris Brandt
2015-03-23  5:54                     ` Arnd Bergmann
2015-03-23  5:54                       ` Arnd Bergmann
2015-03-23 13:54                       ` Chris Brandt
2015-03-23 13:54                         ` Chris Brandt
2015-03-22 19:40                 ` Arnd Bergmann [this message]
2015-03-22 19:40                   ` Arnd Bergmann
2015-03-23  1:37                 ` Geert Uytterhoeven
2015-03-23  1:37                   ` Geert Uytterhoeven
2015-03-23  1:49                   ` Chris Brandt
2015-03-23  1:49                     ` Chris Brandt
2015-03-17 16:36 ` Uwe Kleine-König
2015-03-17 16:36   ` Uwe Kleine-König
2015-04-05 16:02 ` Russell King - ARM Linux
2015-04-05 16:02   ` Russell King - ARM Linux
2015-04-07  3:31   ` Chris Brandt
2015-04-07  3:31     ` Chris Brandt

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=201503222040.56105.arnd@arndb.de \
    --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 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.