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
next prev parent 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: linkBe 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.