2003-09-04 06:48:58

by John Cherry

[permalink] [raw]
Subject: IA32 - 4 New warnings

On the nightly IA32 compiles, I am now flagging NEW warnings that were
introduced into the build in the last 24 hours. I'll send these out
automatically for awhile with the builds as it has some value at this
stage.

drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
(declared at include/linux/module.h:482)
drivers/net/wan/cosa.c:669: warning: `MOD_DEC_USE_COUNT' is deprecated
(declared at include/linux/module.h:494)
drivers/net/wan/cosa.c:729: warning: `MOD_DEC_USE_COUNT' is deprecated
(declared at include/linux/module.h:494)

John



2003-09-04 07:29:33

by David Miller

[permalink] [raw]
Subject: Re: IA32 - 4 New warnings

On Wed, 3 Sep 2003 23:48:54 -0700 (PDT)
John Cherry <[email protected]> wrote:

> drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
> drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
> (declared at include/linux/module.h:482)

There is no way these warnings were added in the past
24 hours, they've been there nearly the entire 2.5.x
series.

2003-09-04 16:32:11

by John Cherry

[permalink] [raw]
Subject: Re: IA32 - 4 New warnings

David,

You are absolutely correct that these are not new warnings. However,
they are new with respect to CLEAN_COMPILE being enabled by default. On
Sept 2, Linus add the following patch...

-config BROKEN
- bool "Prompt for old and known-broken drivers"
- depends on EXPERIMENTAL
- default n
+config CLEAN_COMPILE
+ bool "Don't select drivers known to be broken" if EXPERIMENTAL
+ default y help
- This option allows you to choose whether you want to try to
- compile (and fix) old drivers that haven't been updated to
- new infrastructure.
+ Select this option if you don't even want to see the option
+ to configure known-broken drivers.
+
+ If unsure, say Y
- If unsure, say N.
+config BROKEN
+ bool
+ depends on !CLEAN_COMPILE
+ default y

By default, the builds did not include known broken drivers. The build
yesterday picked up bunk's patch for COSA...

- depends on WAN && ISA && m && BROKEN
+ depends on WAN && ISA && m

This allowed the driver to be built again, and the warnings reappeared.
So these ARE new warnings in the sense that bunk specifically declared
this driver NOT to be broken. In fact, the changeset comment was "COSA
is no longer BROKEN".

Actually, to remain consistent with warning and error statistics, I will
need to over-ride the default CLEAN_COMPILE config option in the
automated builds and continue to build broken drivers. Perhaps I will
build both. Hey, it is just CPU time. :)

John


On Thu, 2003-09-04 at 00:24, David S. Miller wrote:
> On Wed, 3 Sep 2003 23:48:54 -0700 (PDT)
> John Cherry <[email protected]> wrote:
>
> > drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
> > drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
> > (declared at include/linux/module.h:482)
>
> There is no way these warnings were added in the past
> 24 hours, they've been there nearly the entire 2.5.x
> series.
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/