All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Rostedt <srostedt@redhat.com>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: "Robert P. J. Day" <rpjday@crashcourse.ca>,
	Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>
Subject: Re: strange behaviour from "make localmodconfig" throws out ath9k stuff
Date: Mon, 29 Apr 2013 10:02:54 -0400	[thread overview]
Message-ID: <1367244174.28120.3.camel@fedora> (raw)
In-Reply-To: <20130428202854.GC4571@free.fr>

On Sun, 2013-04-28 at 22:28 +0200, Yann E. MORIN wrote:
> Robert, All,
> 
> On Sat, Apr 27, 2013 at 02:19:05PM -0400, Robert P. J. Day wrote:
> >   i'm going to take a wild, uneducated stab at this, but it matches
> > what i was starting to suspect, anyway. this has nothing to do with
> > config processing, it has to do specifically with how the Kconfig
> > entries related to atheros cards were changed.
> > 
> >   here's the important part:
> > 
> > $ git show 23c1d7f
> > ... snip ...
> >     So, this patch introduce new Kconfig variable ATH_CARDS for belonging
> >     to the "Atheros Wireless Cards" family; while ATH_COMMON becomes hidden
> >     variable to express dependency on common Atheros code in ath.ko. Modules
> >     that depend on this common code now express it by setting ATH_COMMON.
> > ... snip ...
> > -menuconfig ATH_COMMON
> > +config ATH_COMMON
> > +       tristate
> > +
> > +menuconfig ATH_CARDS
> > 
> >   in short, a new variable, ATH_CARDS, was introduced that doesn't
> > appear in the earlier .config so, unsurprisingly, when you run "make
> > oldconfig", in the midst of all of the other manual answers, you have
> > to specify what you want done, and look at the default:
> > 
> > $ make oldconfig
> > ... many manual choices ...
> > Atheros Wireless Cards (ATH_CARDS) [N/m/?] (NEW)   <-- there's the culprit
> > ... snip ...
> > 
> >   so running the standard "yes '' | make oldconfig" is going to
> > deselect what looks like almost all ath9k-related stuff, simply
> > because a new, low-level dependency variable was introduced.
> > 
> >   am i making sense here?
> 
> Not sure how streamline_config,pl should behave. Cc-ing Steven as the
> original author, maybe he has a better understanding on this situation.

Running an older config on a newer kernel can have strange effects,
although I do that all the time. I just expect the strange effects and
fix them when they occur.

Looks like the above is one of the strange effects that need a manual
fix. localmodconfig will not enable anything that wasn't enabled in the
original config. If a new dependency is added by a newer kernel then you
need to run an make oldconfig and make sure you have everything before
doing a localmodconfig. Otherwise, you may lose a module.

Hmm, I may be able to have localmodconfig warn if it can not satisfy a
module.

-- Steve




  parent reply	other threads:[~2013-04-29 14:03 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-26 19:05 strange behaviour from "make localmodconfig" throws out ath9k stuff Robert P. J. Day
2013-04-26 19:36 ` Yann E. MORIN
2013-04-27  0:47   ` Robert P. J. Day
2013-04-27 15:38 ` Yann E. MORIN
2013-04-27 17:30   ` Robert P. J. Day
2013-04-27 17:42     ` Yann E. MORIN
2013-04-27 17:48       ` Robert P. J. Day
2013-04-27 18:19       ` Robert P. J. Day
2013-04-28 20:28         ` Yann E. MORIN
2013-04-29 10:54           ` Robert P. J. Day
2013-04-29 14:11             ` Steven Rostedt
2013-04-29 14:28               ` Robert P. J. Day
2013-04-29 14:41                 ` Steven Rostedt
2013-04-29 15:01                   ` Robert P. J. Day
2013-04-29 15:13                     ` Steven Rostedt
2013-04-29 15:19                       ` Robert P. J. Day
     [not found]                       ` <alpine.DEB.2.02.1304291121180.26536@oneiric>
2013-04-29 16:15                         ` Steven Rostedt
2013-04-29 18:00                         ` Steven Rostedt
2013-04-29 18:26                           ` Robert P. J. Day
2013-04-29 18:41                           ` Robert P. J. Day
2013-04-29 19:15                           ` Robert P. J. Day
2013-04-29 20:43                             ` Steven Rostedt
2013-04-29 20:50                               ` Robert P. J. Day
2013-04-29 20:39                           ` Robert P. J. Day
2013-04-29 23:40                             ` Steven Rostedt
2013-04-29 14:02           ` Steven Rostedt [this message]
2013-04-27 17:47   ` Robert P. J. Day

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=1367244174.28120.3.camel@fedora \
    --to=srostedt@redhat.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=rpjday@crashcourse.ca \
    --cc=yann.morin.1998@free.fr \
    /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.