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

Groups > linux.kernel > #1304102

Re: Build failure: -Wno-unused-const-variable DNE on old GCC

Show all headers | View raw

Excerpts from Brian Norris's message of 2016-01-08 10:02:25 +1100:
> >  - It forces cxl developers to a higher standard. cxl has already had
> >    more than it's fair share of incredibly difficult to debug issues,
> >    so any way we can reduce the risk of errors going in makes our lives
> >    (and our end-users lives) better.
> 
> One problem with this point: not all warnings are under the purview of
> cxl developers. For instance, if I turn up warning verbosity (W=1), then
> the *header* files start producing plenty of warnings. Should this break
> the build? Your code didn't change, and you can't fix those errors.

That's a good point, but the specific warnings that we suppressed in the
new compiler are in drivers/misc/cxl/cxl.h, which is an internal header
that should only ever be included by the cxl driver. We do have some
headers elsewhere which are included by other drivers, the generic ppc
architecture code and userspace, but these are all warning free and
won't be affected by the -Werror when included from elsewhere.

> That is a real use case for me daily: I turn the warning verbosity up on
> my compile tests, then (smart)diff the build logs before and after
> new patches. That way, I can see what new warnings (even potentially
> false positive ones) are introduced. I can't do that if every random
> developer wants to stick -Werror in their Makefile.

Makes sense.

> I think there are plenty of reasons to either remove -Werror, or make it
> configurable. Some of them are detailed above.
>
> Maybe you can gate the -Werror on CONFIG_PPC_WERROR, just like the rest
> of PowerPC?

Agreed.

@Daniel - since you added the -Werror do you want to do this, or shall
I?

Cheers,
-Ian

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Build failure: -Wno-unused-const-variable DNE on old GCC Brian Norris <computersforpeace@gmail.com> - 2016-01-07 20:00 +0100
  Re: Build failure: -Wno-unused-const-variable DNE on old GCC Joe Perches <joe@perches.com> - 2016-01-07 20:40 +0100
    Re: Build failure: -Wno-unused-const-variable DNE on old GCC Michal Marek <mmarek@suse.com> - 2016-01-07 20:50 +0100
      Re: Build failure: -Wno-unused-const-variable DNE on old GCC Joe Perches <joe@perches.com> - 2016-01-07 21:00 +0100
        Re: Build failure: -Wno-unused-const-variable DNE on old GCC Brian Norris <computersforpeace@gmail.com> - 2016-01-07 21:20 +0100
          [PATCH] misc: cxl: fix build for GCC 4.6.x Brian Norris <computersforpeace@gmail.com> - 2016-01-07 21:40 +0100
            Re: [PATCH] misc: cxl: fix build for GCC 4.6.x Michael Ellerman <mpe@ellerman.id.au> - 2016-01-08 03:20 +0100
  Re: Build failure: -Wno-unused-const-variable DNE on old GCC Arnd Bergmann <arnd@arndb.de> - 2016-01-07 21:30 +0100
    Re: Build failure: -Wno-unused-const-variable DNE on old GCC Daniel Axtens <dja@axtens.net> - 2016-01-08 00:00 +0100
      Re: Build failure: -Wno-unused-const-variable DNE on old GCC Brian Norris <computersforpeace@gmail.com> - 2016-01-08 00:10 +0100
        Re: Build failure: -Wno-unused-const-variable DNE on old GCC Ian Munsie <imunsie@au1.ibm.com> - 2016-01-08 02:40 +0100
          Re: Build failure: -Wno-unused-const-variable DNE on old GCC Brian Norris <computersforpeace@gmail.com> - 2016-01-08 03:10 +0100
            Re: Build failure: -Wno-unused-const-variable DNE on old GCC Michael Ellerman <mpe@ellerman.id.au> - 2016-01-08 03:20 +0100
              RE: Build failure: -Wno-unused-const-variable DNE on old GCC David Laight <David.Laight@ACULAB.COM> - 2016-01-08 11:20 +0100
    Re: Build failure: -Wno-unused-const-variable DNE on old GCC Ian Munsie <imunsie@au1.ibm.com> - 2016-01-08 02:40 +0100

csiph-web