Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1392033 > unrolled thread

Re: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d)

Started byArnd Bergmann <arnd@arndb.de>
First post2016-05-02 09:30 +0200
Last post2016-05-03 22:20 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d) Arnd Bergmann <arnd@arndb.de> - 2016-05-02 09:30 +0200
    Re: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d) Olof Johansson <olof@lixom.net> - 2016-05-03 04:20 +0200
      Re: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d) Arnd Bergmann <arnd@arndb.de> - 2016-05-03 22:20 +0200

#1392033 — Re: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d)

FromArnd Bergmann <arnd@arndb.de>
Date2016-05-02 09:30 +0200
SubjectRe: mainline build: 896 warnings 1 failures (mainline/v4.6-rc5-224-gda9373d)
Message-ID<rugsO-7Dm-11@gated-at.bofh.it>
On Sunday 01 May 2016 16:19:53 Olof Johansson wrote:
> On Sun, May 1, 2016 at 2:26 PM, Olof's autobuilder <build@lixom.net> wrote:
> > Here are the build results from automated periodic testing.
> >
> > The tree being built was mainline, found at:
> >
> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git (master branch)
> >
> > Topmost commit:
> >
> > da9373d Merge branch 'fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal
> >
> > Build logs (stderr only) can be found at the following link (experimental):
> >
> > http://arm-soc.lixom.net/buildlogs/mainline/v4.6-rc5-224-gda9373d/
> >
> >
> >         Runtime:                5m 0s
> >
> >         Passed:                 117
> >         Failed:                 1
> >
> >         Warnings:               896
> >
> >         Section mismatches:     0
> 
> So, yeah, I updated to GCC 6.1.0 and it looks noisy. If it doesn't get
> whittled down pretty quickly I might switch to an older compiler
> again.

Oops, it looks like I never submitted a patch to shut up
-Wunused-const-variable, though I've been building that way
myself here and fixing all /other/ warnings.

I think we want to have -Wunused-const-variable at the W=1 level, but
not by default.

	Arnd

[toc] | [next] | [standalone]


#1392989

FromOlof Johansson <olof@lixom.net>
Date2016-05-03 04:20 +0200
Message-ID<ruy6m-7Dk-5@gated-at.bofh.it>
In reply to#1392033
On Mon, May 2, 2016 at 12:24 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Sunday 01 May 2016 16:19:53 Olof Johansson wrote:
>> On Sun, May 1, 2016 at 2:26 PM, Olof's autobuilder <build@lixom.net> wrote:
>> > Here are the build results from automated periodic testing.
>> >
>> > The tree being built was mainline, found at:
>> >
>> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git (master branch)
>> >
>> > Topmost commit:
>> >
>> > da9373d Merge branch 'fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal
>> >
>> > Build logs (stderr only) can be found at the following link (experimental):
>> >
>> > http://arm-soc.lixom.net/buildlogs/mainline/v4.6-rc5-224-gda9373d/
>> >
>> >
>> >         Runtime:                5m 0s
>> >
>> >         Passed:                 117
>> >         Failed:                 1
>> >
>> >         Warnings:               896
>> >
>> >         Section mismatches:     0
>>
>> So, yeah, I updated to GCC 6.1.0 and it looks noisy. If it doesn't get
>> whittled down pretty quickly I might switch to an older compiler
>> again.
>
> Oops, it looks like I never submitted a patch to shut up
> -Wunused-const-variable, though I've been building that way
> myself here and fixing all /other/ warnings.
>
> I think we want to have -Wunused-const-variable at the W=1 level, but
> not by default.

Agreed, at least as long as it is as noisy as it is today.

Care to send it in so we can see it in 4.6?

-Olof

[toc] | [prev] | [next] | [standalone]


#1393779

FromArnd Bergmann <arnd@arndb.de>
Date2016-05-03 22:20 +0200
Message-ID<ruOXv-6su-1@gated-at.bofh.it>
In reply to#1392989
On Monday 02 May 2016 19:12:31 Olof Johansson wrote:
> On Mon, May 2, 2016 at 12:24 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> > On Sunday 01 May 2016 16:19:53 Olof Johansson wrote:
> >> On Sun, May 1, 2016 at 2:26 PM, Olof's autobuilder <build@lixom.net> wrote:
> >> > Here are the build results from automated periodic testing.
> >> >
> >> > The tree being built was mainline, found at:
> >> >
> >> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git (master branch)
> >> >
> >> > Topmost commit:
> >> >
> >> > da9373d Merge branch 'fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal
> >> >
> >> > Build logs (stderr only) can be found at the following link (experimental):
> >> >
> >> > http://arm-soc.lixom.net/buildlogs/mainline/v4.6-rc5-224-gda9373d/
> >> >
> >> >
> >> >         Runtime:                5m 0s
> >> >
> >> >         Passed:                 117
> >> >         Failed:                 1
> >> >
> >> >         Warnings:               896
> >> >
> >> >         Section mismatches:     0
> >>
> >> So, yeah, I updated to GCC 6.1.0 and it looks noisy. If it doesn't get
> >> whittled down pretty quickly I might switch to an older compiler
> >> again.
> >
> > Oops, it looks like I never submitted a patch to shut up
> > -Wunused-const-variable, though I've been building that way
> > myself here and fixing all /other/ warnings.
> >
> > I think we want to have -Wunused-const-variable at the W=1 level, but
> > not by default.
> 
> Agreed, at least as long as it is as noisy as it is today.
> 
> Care to send it in so we can see it in 4.6?

Sure, I'll try.

I seem to have a low success rate getting Makefile changes like this
merged: My best guess is that Michal Marek should pick them up, but
he ignored my last few submission for warning changes, so I don't
know if I should instead send them to Andrew Morton or someone else.

	Arnd

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web