public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: ijc@hellion.org.uk (Ian Campbell)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/4] ARM: mvebu: minor Armada 370 RD improvements
Date: Thu, 11 Sep 2014 16:24:32 +0100	[thread overview]
Message-ID: <1410449072.25388.35.camel@kazak.uk.xensource.com> (raw)
In-Reply-To: <20140911142655.GB30828@titan.lakedaemon.net>

On Thu, 2014-09-11 at 10:26 -0400, Jason Cooper wrote:
> merf.  It helps if I actually _add_ Ian to the Cc. :)
> 
> On Thu, Sep 11, 2014 at 10:26:01AM -0400, Jason Cooper wrote:
> > +Ian
> > 
> > On Thu, Sep 11, 2014 at 04:00:00PM +0200, Thomas Petazzoni wrote:
> > > On Thu, 11 Sep 2014 15:43:23 +0200, Andrew Lunn wrote:
> > > > A point for discussion, triggered by 3/4. We have repeatedly seen
> > > > issues with modular builds, which we don't see with everything built
> > > > in. Can we improve our testing? Is there is way to take for example
> > > > mvebu_v7_defconfig and turn any y tristate options into m? Build and
> > > > boot? Get this added to the boot test farms?
> > > 
> > > I indeed agree that we are not testing much the modular case, but:
> > > 
> > >  1/ In my case, a mvebu_v7_defconfig where most the stuff would become
> > >     enabled as a module would make mvebu_v7_defconfig pretty much
> > >     useless for my daily work. When I'm doing active development, I
> > >     typically have everything built-in including the initramfs itself,
> > >     so that I don't rely on any storage/network to get the rootfs and
> > >     additional kernel drivers.
> > > 
> > >  2/ I don't think it would change much in terms of runtime coverage:
> > >     the tests done on the boot farm are just boot tests, I don't think
> > >     they load any kernel module. We could ask Kevin and Olof what
> > >     rootfs they use, but most likely it's some very small rootfs that
> > >     doesn't even have udev/mdev for automatic module loading. So no
> > >     module would be loaded, which would mean that a mvebu_v7_defconfig
> > >     with everything as a module would actually test *less* thing than a
> > >     configuration that has everything built in.
> > > 
> > > So, I agree on the observation, but I'm unsure the proposed solution
> > > will really solve or even improve the situation :/
> > 
> > Since debian does modular builds on a regular basis, maybe they have
> > some recommendations.

Rather than modularising $socfamily_defconfig perhaps this is something
which can be achieved by also testing multiplatform defconfigs builds,
and perhaps making those more modular (if they aren't already)?

It seems like a multiplatform kernel is something which would be more
naturally quite modular anyway, and certainly distros are mostly
interested in moving to configurations which are both multiplatform and
modular (at least in the long run, Debian has some legacy v5 flavours
which may or may not get moved over).

That would avoid needing to mess with mvebu_v7_defconfig, since it seems
quite reasonable and natural for that to be pretty non-modular.

Ian.

  parent reply	other threads:[~2014-09-11 15:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-11  9:56 [PATCH 0/4] ARM: mvebu: minor Armada 370 RD improvements Thomas Petazzoni
2014-09-11  9:56 ` [PATCH 1/4] ARM: mvebu: add gpio fan support to Armada 370 RD Thomas Petazzoni
2014-09-11  9:56 ` [PATCH 2/4] ARM: mvebu: add user LED support of " Thomas Petazzoni
2014-09-11  9:56 ` [PATCH 3/4] ARM: mvebu: add LED class support built-in in mvebu_v7_defconfig Thomas Petazzoni
2014-09-11  9:56 ` [PATCH 4/4] ARM: mvebu: add gpio-fan to mvebu_v7_defconfig Thomas Petazzoni
2014-09-11 13:43 ` [PATCH 0/4] ARM: mvebu: minor Armada 370 RD improvements Andrew Lunn
2014-09-11 14:00   ` Thomas Petazzoni
2014-09-11 14:26     ` Jason Cooper
2014-09-11 14:26       ` Jason Cooper
2014-09-11 14:40         ` Andrew Lunn
2014-09-11 15:24         ` Ian Campbell [this message]
2014-09-13 21:27 ` Jason Cooper

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=1410449072.25388.35.camel@kazak.uk.xensource.com \
    --to=ijc@hellion.org.uk \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox