All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Pitre <nicolas.pitre@linaro.org>
To: Chris Brandt <Chris.Brandt@renesas.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	Russell King <linux@armlinux.org.uk>,
	Greg Ungerer <gerg@linux-m68k.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	Vladimir Murzin <vladimir.murzin@arm.com>,
	Arnd Bergmann <arnd@arndb.de>,
	linux-kbuild <linux-kbuild@vger.kernel.org>
Subject: RE: [PATCHv4 4/4] ARM: versatile: support configuring versatile machine for no-MMU
Date: Fri, 22 Jun 2018 13:05:25 -0400 (EDT)	[thread overview]
Message-ID: <nycvar.YSQ.7.76.1806221244490.16670@knanqh.ubzr> (raw)
In-Reply-To: <TY1PR01MB15624342267F973525BF45698A750@TY1PR01MB1562.jpnprd01.prod.outlook.com>

On Fri, 22 Jun 2018, Chris Brandt wrote:

> On Friday, June 22, 2018 1, Nicolas Pitre wrote:
> > All that is possible already. But config symbol mutual exclusion is not.
> 
> I still don't understand why we need some false sense of security for 
> only selecting 1 platform.

Because that's the right thing to do. Small hacks are nice, but when 
they accumulate (and they always do over time) then maintainability 
suffers. This is why you may do whatever small hacks in your own copy of 
the kernel code and that might be good enough, but the mainline source 
tree should be held to higher standards.

> There are probably plenty of kernel config options that you could enable
> that would break any number of systems.

That's no excuse to add more of them. Many people are constantly working 
to move things in the other direction.

> So, why do we feel that XIP_KERNEL needs a warm safety blanket around 
> it?

Because we simply try not to create invalid kernel configurations. 
XIP_KERNEL is not more special than other symbols in that respect.

And if the kconfig language has to be extended for proper configurations 
to be produced then that's what it is, and trying to cheat around that 
fact won't produce any good.


Nicolas

WARNING: multiple messages have this Message-ID (diff)
From: nicolas.pitre@linaro.org (Nicolas Pitre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv4 4/4] ARM: versatile: support configuring versatile machine for no-MMU
Date: Fri, 22 Jun 2018 13:05:25 -0400 (EDT)	[thread overview]
Message-ID: <nycvar.YSQ.7.76.1806221244490.16670@knanqh.ubzr> (raw)
In-Reply-To: <TY1PR01MB15624342267F973525BF45698A750@TY1PR01MB1562.jpnprd01.prod.outlook.com>

On Fri, 22 Jun 2018, Chris Brandt wrote:

> On Friday, June 22, 2018 1, Nicolas Pitre wrote:
> > All that is possible already. But config symbol mutual exclusion is not.
> 
> I still don't understand why we need some false sense of security for 
> only selecting 1 platform.

Because that's the right thing to do. Small hacks are nice, but when 
they accumulate (and they always do over time) then maintainability 
suffers. This is why you may do whatever small hacks in your own copy of 
the kernel code and that might be good enough, but the mainline source 
tree should be held to higher standards.

> There are probably plenty of kernel config options that you could enable
> that would break any number of systems.

That's no excuse to add more of them. Many people are constantly working 
to move things in the other direction.

> So, why do we feel that XIP_KERNEL needs a warm safety blanket around 
> it?

Because we simply try not to create invalid kernel configurations. 
XIP_KERNEL is not more special than other symbols in that respect.

And if the kconfig language has to be extended for proper configurations 
to be produced then that's what it is, and trying to cheat around that 
fact won't produce any good.


Nicolas

  reply	other threads:[~2018-06-22 17:05 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-18 14:19 [PATCHv4 0/4] arm/versatile: no-MMU support Greg Ungerer
2018-06-18 14:19 ` [PATCHv4 1/4] ARM: versatile: support no-MMU mode addressing Greg Ungerer
2018-06-18 14:19 ` [PATCHv4 2/4] ARM: versatile: define empty debug_ll_io_init() for no-MMU Greg Ungerer
2018-06-18 14:19 ` [PATCHv4 3/4] ARM: versatile: empty Makefile.boot needed for no-MMU compile Greg Ungerer
2018-06-18 14:19 ` [PATCHv4 4/4] ARM: versatile: support configuring versatile machine for no-MMU Greg Ungerer
2018-06-21 15:59   ` Geert Uytterhoeven
2018-06-21 16:24     ` Nicolas Pitre
2018-06-21 16:24       ` Nicolas Pitre
2018-06-21 16:45       ` Chris Brandt
2018-06-21 16:45         ` Chris Brandt
2018-06-22  6:27         ` Geert Uytterhoeven
2018-06-22  6:27           ` Geert Uytterhoeven
2018-06-22 13:26           ` Chris Brandt
2018-06-22 13:26             ` Chris Brandt
2018-06-22 11:01       ` Russell King - ARM Linux
2018-06-22 11:01         ` Russell King - ARM Linux
2018-06-22 15:25         ` Nicolas Pitre
2018-06-22 15:25           ` Nicolas Pitre
2018-06-22 15:33           ` Geert Uytterhoeven
2018-06-22 15:33             ` Geert Uytterhoeven
2018-06-22 15:57             ` Nicolas Pitre
2018-06-22 15:57               ` Nicolas Pitre
2018-06-22 16:11               ` Russell King - ARM Linux
2018-06-22 16:11                 ` Russell King - ARM Linux
2018-06-22 16:21                 ` Nicolas Pitre
2018-06-22 16:21                   ` Nicolas Pitre
2018-06-22 16:40                   ` Russell King - ARM Linux
2018-06-22 16:40                     ` Russell King - ARM Linux
2018-06-22 16:54                     ` Nicolas Pitre
2018-06-22 16:54                       ` Nicolas Pitre
2018-06-22 17:09                       ` Russell King - ARM Linux
2018-06-22 17:09                         ` Russell King - ARM Linux
2018-06-22 17:25                         ` Nicolas Pitre
2018-06-22 17:25                           ` Nicolas Pitre
2018-06-22 16:40                 ` Chris Brandt
2018-06-22 16:40                   ` Chris Brandt
2018-06-22 16:44               ` Chris Brandt
2018-06-22 16:44                 ` Chris Brandt
2018-06-22 17:05                 ` Nicolas Pitre [this message]
2018-06-22 17:05                   ` Nicolas Pitre
2018-06-22 17:23                   ` Chris Brandt
2018-06-22 17:23                     ` Chris Brandt
2018-06-22 17:47                     ` Nicolas Pitre
2018-06-22 17:47                       ` Nicolas Pitre
2018-06-22 18:38                     ` Russell King - ARM Linux
2018-06-22 18:38                       ` Russell King - ARM Linux
2018-06-22 20:25                       ` Chris Brandt
2018-06-22 20:25                         ` Chris Brandt
2018-06-22 20:28                         ` Geert Uytterhoeven
2018-06-22 20:28                           ` Geert Uytterhoeven
2018-06-22 20:33                         ` Russell King - ARM Linux
2018-06-22 20:33                           ` Russell King - ARM Linux
2018-06-22  0:58     ` Greg Ungerer
2018-06-22  9:26     ` Russell King - ARM Linux

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=nycvar.YSQ.7.76.1806221244490.16670@knanqh.ubzr \
    --to=nicolas.pitre@linaro.org \
    --cc=Chris.Brandt@renesas.com \
    --cc=arnd@arndb.de \
    --cc=geert@linux-m68k.org \
    --cc=gerg@linux-m68k.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=vladimir.murzin@arm.com \
    /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.