lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 07 Dec 2016 12:41:29 +0100 From: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com> To: Olof Johansson <olof@...om.net> Cc: "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Arnd Bergmann <arnd@...db.de>, Russell King <linux@....linux.org.uk> Subject: Re: [RFC PATCH 00/23] arm: defconfigs: use kconfig fragments Hi, On Tuesday, December 06, 2016 11:03:34 AM Olof Johansson wrote: > On Tue, Dec 6, 2016 at 4:38 AM, Bartlomiej Zolnierkiewicz > <b.zolnierkie@...sung.com> wrote: > > Hi, > > > > This RFC patchset starts convertion of ARM defconfigs to use kconfig > > fragments and dynamically generate defconfigs. The goals of this > > work are to: > > You don't provide any motivation as to why this is better. As far as I Benefits are: - no code duplication (this initial patchset alone removes ~1700 lines from defconfigs without any change in functionality) - prevention of "multi" defconfigs (i.e. multi_v7_defconfig) going out of sync with "SoC-family" ones (i.e. exynos_defconfig) - there will be just one place to update when changing things - possibility to add support for more optimized defconfigs (i.e. board specific ones) in the future without duplicating the code - making it easier to define arch specific parts of defconfigs in the future if we decide on doing it (i.e. we may want to enable things like CONFIG_SYSVIPC for all defconfigs) > am concerned it'll just be a mess. > > So: > > Nack. So much nack. I really don't want to see a proliferation of > config fragments like this. > > I had a feeling it was a bad idea to pick up that one line config > fragment before, since it opened the door for this kind of mess. :( Like I said in the cover-letter I'm not satisfied with the current patches and they have much room for improvement. However I see that you don't like the idea itself... :( Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics
Powered by blists - more mailing lists