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

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

[Arnd Bergmann]

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

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

[Olof Johansson]

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

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

[Arnd Bergmann]

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