public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: nunojsilva@ist.utl.pt (Nuno J. Silva)
To: gentoo-dev@lists.gentoo.org
Subject: [gentoo-dev] Re: RFC: CONFIG_CHECK_FATAL, making CONFIG_CHECKS fatal by default
Date: Fri, 25 Jan 2013 22:06:35 +0200	[thread overview]
Message-ID: <87vcalm3hw.fsf@ist.utl.pt> (raw)
In-Reply-To: CAGfcS_mvYHDch+i1sNJzdjqtdwazMvi=DTvn1Q6V+5Gyub5mDw@mail.gmail.com

On 2013-01-25, Rich Freeman wrote:

> On Fri, Jan 25, 2013 at 2:47 PM, Nuno J. Silva <nunojsilva@ist.utl.pt> wrote:
>>
>> Sorry, what's the difference between cheching =y and =m? I thought those
>> were both part of the kernel config...
>
> I'm talking about /proc/config.gz, which only reflects .config at the
> time that the kernel was built.  So, build with config=n, then set
> config=m and install the modules but don't replace the kernel.  Now
> /proc/config.gz still says n, but the module is there and works fine.
> And this is in fact the easiest way to add a module for something that
> you didn't realize you needed at kernel build time - you can do this
> on a running system.
>
>>
>>> You can also check /usr/src/linux/.config, but the sources might not
>>> correspond to the running kernel, or the kernel on the next reboot, or
>>> whatever.
>>
>> Ok, what do these checks do right now? I thought that they were checking
>> .config...
>
> I dunno.  I wasn't talking about how the current config checks work.
> The question was whether it was possible to determine how the kernel
> was configured - I was answering in general.

I am almost sure the current check does *not* use config.gz but
/usr/src/linux/.config. At least, I've not had config.gz enabled for a
long time, and I've always seen the checks working. In fact, I think the
checks print something along the lines of "looking for kernel source
under /usr/src/linux... found config for linux-...".

>> So you're saying that it's perfectly OK to check for =y or =n, but that
>> it's somehow more difficult to check for =m?
>
> My previous paragraph was referring to checking config.gz - and that
> is unreliable for modules.  /usr/src/linux/.config is unreliable for
> the reason I stated in the next paragraph - it doesn't necessarily
> reflect the running kernel.

Not only for modules, config.gz may not even be available, while
/usr/src/linux/.config is reliable, as it should exist, and, when you're
compiling, you're typically compiling for what is going to be your main
kernel, the one under /usr/src/linux, which, to have been built, must
have a .config, an.

And you do not want to check against the running kernel, you want to
check against the currently selected kernel. Checking /proc/config.gz
would lead to issues when you're in middle of a kernel update and
rebuilding modules before reboot.

>> This won't even solve the issue, even if some people may actually prefer
>> a pre-built kernel.
>
> Depends on the issue.  There isn't just one issue under discussion in
> this thread.  A fairly bulletproof kernel solves a lot of issues in
> general as it can have newbie-safe defaults (like just about anything
> in any config check).  There is a reason that most distros don't need
> config checks.

Well, we could also get rid of issues with clashing USE flags by getting
rid of USE flags and offering monolithic binary packages with almost
every compatible feature enabled by default.

And newbie-safe, I've had far too many systems where your usual binary
distro "newbie-safe" kernel fails and the first thing I need to do even
with binary distros is to build my own kernel to get something that
actually boots with no issues.

>> But, definitely, fatal checks should not be a default, there are way too
>> many scenarios.
>
> Yup - just trying to point out some of the perils.  As I said there
> are lots of 80% solutions.


-- 
Nuno Silva (aka njsg)
http://njsg.sdf-eu.org/



  reply	other threads:[~2013-01-25 20:07 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-22  3:38 [gentoo-dev] RFC: CONFIG_CHECK_FATAL, making CONFIG_CHECKS fatal by default Robin H. Johnson
2013-01-22  3:45 ` Mike Gilbert
2013-01-22  3:56   ` Zac Medico
2013-01-22  4:10     ` Rick "Zero_Chaos" Farina
2013-01-22  4:14       ` Zac Medico
2013-01-22  9:22     ` Markos Chandras
2013-01-22 14:49       ` Zac Medico
2013-01-22  4:23 ` Mike Gilbert
2013-01-22  6:22   ` Sergey Popov
2013-01-22  6:44     ` [gentoo-dev] " Duncan
2013-01-22 14:51     ` [gentoo-dev] " Zac Medico
2013-01-22 18:40       ` Robin H. Johnson
2013-01-23 12:32         ` Fabio Erculiani
2013-01-23 12:50           ` Rich Freeman
2013-01-23 21:27             ` Fabio Erculiani
2013-01-23 23:17               ` Francesco Riosa
2013-01-24  1:10           ` Robin H. Johnson
2013-01-24 19:46             ` Fabio Erculiani
2013-01-22 11:11 ` vivo75
2013-01-22 11:56   ` Rich Freeman
2013-01-24 17:04     ` Dustin C. Hatch
2013-01-24 17:49     ` [gentoo-dev] " Duncan
2013-01-24 18:09       ` Rich Freeman
2013-01-24 18:18         ` Ian Stakenvicius
2013-01-24 18:24           ` Dustin C. Hatch
2013-01-24 18:25           ` Rich Freeman
2013-01-24 18:55             ` Michael Orlitzky
2013-01-24 18:58               ` Ian Stakenvicius
2013-01-24 19:05                 ` Rich Freeman
2013-01-24 19:21                 ` Michael Orlitzky
2013-01-24 20:26                   ` vivo75
2013-01-24 20:39                     ` Michael Orlitzky
2013-01-24 20:45                       ` Michael Orlitzky
2013-01-25  0:29                         ` vivo75
2013-01-25  0:37                           ` Michael Orlitzky
2013-01-24 21:30                   ` Rich Freeman
2013-01-26 10:34                     ` Fabio Erculiani
2013-01-26 22:25                       ` Duncan
2013-01-26 22:30                         ` Fabio Erculiani
2013-01-26 22:54                           ` Rich Freeman
2013-01-27  0:31                             ` Peter Stuge
2013-01-27  1:06                               ` Rich Freeman
2013-01-25  1:39               ` Duncan
2013-01-25  1:53                 ` Robin H. Johnson
2013-01-25  2:22                 ` Michael Orlitzky
2013-01-25  3:12                   ` Rich Freeman
2013-01-25  3:27                     ` Michael Orlitzky
2013-01-25  6:15                   ` Duncan
2013-01-25 18:24 ` Nuno J. Silva
2013-01-25 18:47   ` Rich Freeman
2013-01-25 19:19     ` Christopher Head
2013-01-25 19:26       ` Rich Freeman
2013-01-25 19:47     ` Nuno J. Silva
2013-01-25 19:57       ` Rich Freeman
2013-01-25 20:06         ` Nuno J. Silva [this message]
2013-01-25 20:31           ` Rich Freeman
2013-01-26  1:51           ` Duncan

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=87vcalm3hw.fsf@ist.utl.pt \
    --to=nunojsilva@ist.utl.pt \
    --cc=gentoo-dev@lists.gentoo.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