From: ryan@bluewatersys.com (Ryan Mallon)
To: linux-arm-kernel@lists.infradead.org
Subject: Heads up: Linus plans to kill ARM defconfigs
Date: Wed, 09 Jun 2010 08:51:07 +1200 [thread overview]
Message-ID: <4C0EAD3B.8010404@bluewatersys.com> (raw)
In-Reply-To: <20100608131007.GE25370@n2100.arm.linux.org.uk>
Russell King - ARM Linux wrote:
> On Tue, Jun 08, 2010 at 02:50:55PM +0200, Christer Weinigel wrote:
>> Linus isn't stupid. If you explain what you are doing and that the goal
>> is to reduce the set of defconfigs to just one per processor family, and
>> that it will reduce the churn in the end, I think Linus will listen.
>
> It's also not just about the number of defconfigs. Linus had other
> valid points too - for example, you can't review the changes to them
> properly, because any change to them is far too noisy. That point
> doesn't go away by reducing the number of defconfigs.
Part of the problem with the defconfigs is that they are auto-generated
and contain a lot of useless information. As you point out, diffs on
defconfigs tend to result in a lot of noise which makes viewing changes
very difficult.
Striping the spitz defconfig back for example:
ryan at okiwi:configs$ wc -l spitz_defconfig
1820 spitz_defconfig
ryan at okiwi:configs$ grep -v "is not set" spitz_defconfig | grep -v
"^#" | wc -l
641
So removing all the comments and non-set options makes the defconfig
about 1/3 the size. If the defconfigs were generated by hand, or a
proper set of tools, then they could be much less verbose and diffs for
things like adding or removing a single config option would actually be
readable.
If we want to have individual board configurations in the kernel, then
the information has to live somewhere. Whether it is defconfig, KConfig,
Documentation, whatever, the information will still take up a similar
amount of space, and the process of moving the information will generate
a load of diffstat noise.
I think deleting defconfigs is a good first step. Although there will be
diffstat noise, it will also be clear that the net result is that
thousands of lines are being removed, which is a good thing. Having a
better way of generating defconfigs (or however the information is
stored) would fix the general problem of diffstat noise when changes are
made.
~Ryan
--
Bluewater Systems Ltd - ARM Technology Solution Centre
Ryan Mallon 5 Amuri Park, 404 Barbadoes St
ryan at bluewatersys.com PO Box 13 889, Christchurch 8013
http://www.bluewatersys.com New Zealand
Phone: +64 3 3779127 Freecall: Australia 1800 148 751
Fax: +64 3 3779135 USA 1800 261 2934
next prev parent reply other threads:[~2010-06-08 20:51 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-03 19:24 Heads up: Linus plans to kill ARM defconfigs Russell King - ARM Linux
2010-06-03 19:55 ` Marek Vasut
2010-06-03 20:01 ` Russell King - ARM Linux
2010-06-03 20:27 ` Marek Vasut
2010-06-03 20:38 ` Nicolas Pitre
2010-06-08 14:41 ` Russell King - ARM Linux
2010-06-03 23:46 ` Daniel Walker
2010-06-04 11:06 ` Uwe Kleine-König
2010-06-03 19:59 ` Nicolas Pitre
2010-06-03 20:03 ` Russell King - ARM Linux
2010-06-03 20:36 ` Nicolas Pitre
2010-06-03 21:04 ` Ryan Mallon
2010-06-03 22:31 ` Nicolas Pitre
2010-06-03 23:33 ` Ryan Mallon
2010-06-03 23:45 ` Kyungmin Park
2010-06-04 0:13 ` Nicolas Pitre
2010-06-04 1:10 ` Marek Vasut
2010-06-04 1:16 ` Ryan Mallon
2010-06-04 1:35 ` Eric Miao
2010-06-04 1:37 ` Marek Vasut
2010-06-04 1:50 ` Marek Vasut
2010-06-04 1:53 ` Marek Vasut
2010-06-04 6:03 ` Tony Lindgren
2010-06-04 14:59 ` Cory Maccarrone
2010-06-07 7:41 ` Tony Lindgren
2010-06-04 1:57 ` Eric Miao
2010-06-04 6:10 ` Tony Lindgren
2010-06-04 7:12 ` Eric Miao
2010-06-04 8:40 ` Martin Guy
2010-06-04 20:51 ` Nicolas Pitre
2010-06-04 22:08 ` Krzysztof Halasa
2010-06-08 11:58 ` Russell King - ARM Linux
2010-06-08 12:31 ` Tony Lindgren
2010-06-08 12:43 ` Eric Miao
2010-06-08 12:49 ` Russell King - ARM Linux
2010-06-08 13:00 ` Eric Miao
2010-06-08 12:43 ` David John
2010-06-08 12:44 ` Eric Miao
2010-06-08 12:50 ` Nicolas Pitre
2010-06-08 12:44 ` Nicolas Pitre
2010-06-08 12:50 ` Russell King - ARM Linux
2010-06-08 13:01 ` Nicolas Pitre
2010-06-08 13:13 ` Russell King - ARM Linux
2010-06-08 14:51 ` Eric Miao
2010-06-08 16:55 ` Nicolas Pitre
2010-06-08 23:23 ` Daniel Walker
2010-06-08 12:50 ` Christer Weinigel
2010-06-08 13:10 ` Russell King - ARM Linux
2010-06-08 20:51 ` Ryan Mallon [this message]
2010-06-08 21:22 ` Russell King - ARM Linux
2010-06-08 21:32 ` Ryan Mallon
2010-06-08 23:02 ` Nicolas Pitre
2010-06-08 23:21 ` Ryan Mallon
2010-06-08 23:26 ` Daniel Walker
2010-06-08 23:31 ` Nicolas Pitre
2010-06-08 23:52 ` Ryan Mallon
2010-06-09 0:14 ` Nicolas Pitre
2010-06-09 6:07 ` Hendrik Sattler
2010-06-09 13:32 ` Daniel Walker
2010-06-10 6:32 ` Uwe Kleine-König
2010-06-10 19:18 ` Nicolas Pitre
2010-06-09 21:56 ` Ryan Mallon
2010-06-25 12:36 ` Catalin Marinas
2010-06-07 21:09 ` Ryan Mallon
2010-06-04 10:42 ` Tony Lindgren
2010-06-04 8:36 ` pieterg
[not found] ` <AANLkTilyIb8WDAanNHlQKHco6rSjzNjNS9Q3TpWQqt8o@mail.gmail.com>
2010-06-04 8:42 ` Eric Miao
2010-06-04 8:56 ` Daniel Mack
2010-06-04 9:37 ` pieterg
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=4C0EAD3B.8010404@bluewatersys.com \
--to=ryan@bluewatersys.com \
--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.