All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Marek <mmarek@suse.cz>
To: Valdis.Kletnieks@vt.edu
Cc: Borislav Petkov <bp@alien8.de>, Sam Ravnborg <sam@ravnborg.org>,
	linux-kbuild <linux-kbuild@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Dave Jones <davej@redhat.com>
Subject: Re: [PATCH v3.1] kbuild: implement several W= levels
Date: Thu, 28 Apr 2011 18:24:46 +0200	[thread overview]
Message-ID: <4DB994CE.40403@suse.cz> (raw)
In-Reply-To: <11802.1303950320@localhost>

On 28.4.2011 02:25, Valdis.Kletnieks@vt.edu wrote:
> On Wed, 27 Apr 2011 13:35:13 +0200, Borislav Petkov said:
>
>> If we do this inclusive, then W=2 dumps the, let's call it, level 1
>> _plus_ the new level 2 warnings, polluting the output with something
>> I've already seen, but only partially. And then I start to think, did
>> I see this one already, didn't I, which was it? By the time you enable
>> W=3, the output becomes pretty useless. For example, W=3 generates 190+
>> MB logfile here only with level 3 warnings. Now imagine all 3 levels
>> combined.
>
> If each level is averaging 10x the previous level, then all 3 levels will only be 11%
> bigger, or 211MB.
>
> You *really* want to get *all* the warnings - quite often, you'll be looking
> at a set of 15 or 20 level-3 warnings.  And if you had the Level-2's in there as
> well, you'd immediately realize that the single level-2 was the real root-cause
> of all the cascating warnings.

How about W=12 for level 1 and 2 warnings and W=123 for all levels?

Michal

  reply	other threads:[~2011-04-28 16:24 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-21 21:39 [RFC PATCH] kbuild: implement several W= levels Sam Ravnborg
2011-04-21 21:58 ` Joe Perches
2011-04-21 22:56   ` Stratos Psomadakis
2011-04-21 22:06 ` Stratos Psomadakis
2011-04-22  1:28   ` Sam Ravnborg
2011-04-22  7:38 ` [RFC PATCH v2] " Sam Ravnborg
2011-04-22  7:38   ` Sam Ravnborg
2011-04-22  8:19 ` [RFC PATCH] " Borislav Petkov
2011-04-22 10:15   ` Sam Ravnborg
2011-04-22 10:20     ` Borislav Petkov
2011-04-22 10:46       ` [PATCH v3] " Borislav Petkov
2011-04-22 11:09         ` Sam Ravnborg
2011-04-22 17:50           ` [PATCH v3.1] " Borislav Petkov
2011-04-26 19:52             ` Michal Marek
2011-04-26 20:43               ` Borislav Petkov
2011-04-27  8:22                 ` Sam Ravnborg
2011-04-27  8:25             ` Sam Ravnborg
2011-04-27 11:35               ` Borislav Petkov
2011-04-27 20:15                 ` [PATCH v3.2] " Borislav Petkov
2011-04-27 20:21                   ` Joe Perches
2011-04-27 20:46                     ` Sam Ravnborg
2011-04-27 20:46                       ` Sam Ravnborg
2011-04-27 20:46                   ` Sam Ravnborg
2011-04-27 20:46                     ` Sam Ravnborg
2011-04-28 16:18                   ` Michal Marek
2011-04-28  0:25                 ` [PATCH v3.1] " Valdis.Kletnieks
2011-04-28 16:24                   ` Michal Marek [this message]
2011-04-28 17:59                     ` Valdis.Kletnieks
2011-04-29 13:31                       ` [PATCH] kbuild: Allow to combine multiple " Michal Marek
2011-04-29 17:43                         ` Sam Ravnborg
2011-04-29 18:13                         ` Arnaud Lacombe
2011-04-29 18:27                           ` Valdis.Kletnieks
2011-04-29 18:29                         ` Valdis.Kletnieks
2011-05-02 15:38                         ` Michal Marek
2011-05-02 15:53                           ` Arnaud Lacombe
2011-05-02 16:05                             ` Michal Marek
2011-05-02 16:17                               ` Arnaud Lacombe
2011-05-02 16:16                         ` Américo Wang
2011-05-02 17:07                           ` Sam Ravnborg
2011-05-02 17:34                             ` Arnaud Lacombe
2011-05-02 18:03                               ` boris
2011-05-02 18:45                                 ` Arnaud Lacombe
2011-05-02 18:51                                   ` boris
2011-05-02 20:35                                   ` Michal Marek
2011-04-27  8:27         ` [PATCH v3] kbuild: implement several " Geert Uytterhoeven

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=4DB994CE.40403@suse.cz \
    --to=mmarek@suse.cz \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=bp@alien8.de \
    --cc=davej@redhat.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.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.