| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 25 Jul 2011 21:04:44 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Arnaud Lacombe <lacombar@...il.com>,
linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>,
Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [GIT PULL] core/locking changes for v3.1
* Linus Torvalds <torvalds@...ux-foundation.org> wrote:
> On Mon, Jul 25, 2011 at 3:09 AM, Peter Zijlstra <a.p.zijlstra@...llo.nl> wrote:
> >
> > Argh, I really need -Werror to even see warnings, compiler output just
> > flies by waaaaaay too fast.
>
> Do what I do:
>
> make -j32 > ../makes
>
> that way the only thing you see is the warnings (and the final link report).
>
> That pattern was one reason why I wanted me default compile to be
> warning-clean. Of course, then gcc started adding more warnings,
> and it's not clean any more ("warning: statement with no effect"
> from the fact that we just define many functions to '(0)' when they
> are disabled) but my current personal compile config only gets four
> warnings.
>
> I realize there are *many* more warnings once you start enabling
> more special code - as usual, the core code tends to be better
> quality, and the further away from that base you get, the worse
> things get.
>
> But it would definitely be good if most kernel developers did the
> above. Do it for just your personal config - at least you'll see
> the warnings in code you write (because presumably your personal
> config will have that code enabled ;)
Here is what i see on 'make defconfig' (which enables common core and
common drivers):
sound/pci/hda/patch_sigmatel.c: In function ‘stac92xx_init’:
sound/pci/hda/patch_sigmatel.c:4385:3: warning: statement with no
effect [-Wunused-value]
sound/pci/hda/patch_sigmatel.c: In function ‘stac92xx_resume’:
sound/pci/hda/patch_sigmatel.c:4925:3: warning: statement with no
effect [-Wunused-value]
sound/pci/hda/patch_realtek.c: In function ‘alc_init’:
sound/pci/hda/patch_realtek.c:4176:2: warning: statement with no
effect [-Wunused-value]
sound/pci/hda/patch_realtek.c: In function ‘alc_resume’:
sound/pci/hda/patch_realtek.c:4558:2: warning: statement with no
effect [-Wunused-value]
sound/pci/hda/patch_realtek.c: In function ‘alc269_resume’:
sound/pci/hda/patch_realtek.c:14878:2: warning: statement with no
effect [-Wunused-value]
net/sched/sch_generic.c: In function ‘dev_graft_qdisc’:
net/sched/sch_generic.c:678:2: warning: the comparison will always
evaluate as ‘true’ for the address of ‘noop_qdisc’ will never be NULL
[-Waddress]
net/netfilter/nf_conntrack_core.c: In function ‘nf_conntrack_init’:
net/netfilter/nf_conntrack_core.c:1579:3: warning: the comparison
will always evaluate as ‘true’ for the address of
‘nf_conntrack_attach’ will never be NULL [-Waddress]
net/netfilter/nf_conntrack_core.c:1580:3: warning: the comparison
will always evaluate as ‘true’ for the address of ‘destroy_conntrack’
will never be NULL [-Waddress]
sound/pci/hda/hda_codec.c: In function
‘snd_hda_mixer_amp_switch_put’:
sound/pci/hda/hda_codec.c:2320:2: warning: statement with no effect
[-Wunused-value]
net/netfilter/nf_conntrack_proto.c: In function
‘nf_conntrack_l3proto_unregister’:
net/netfilter/nf_conntrack_proto.c:210:2: warning: the comparison
will always evaluate as ‘true’ for the address of
‘nf_conntrack_l3proto_generic’ will never be NULL [-Waddress]
net/netfilter/nf_conntrack_proto.c: In function
‘nf_conntrack_l4proto_unregister’:
net/netfilter/nf_conntrack_proto.c:345:2: warning: the comparison
will always evaluate as ‘true’ for the address of
‘nf_conntrack_l4proto_generic’ will never be NULL [-Waddress]
net/netfilter/nf_conntrack_proto.c: In function
‘nf_conntrack_proto_init’:
net/netfilter/nf_conntrack_proto.c:370:3: warning: the comparison
will always evaluate as ‘true’ for the address of
‘nf_conntrack_l3proto_generic’ will never be NULL [-Waddress]
drivers/md/dm.c: In function ‘split_bvec’:
drivers/md/dm.c:1052:3: warning: statement with no effect
[-Wunused-value]
drivers/md/dm.c: In function ‘clone_bio’:
drivers/md/dm.c:1079:3: warning: statement with no effect
[-Wunused-value]
drivers/md/dm-table.c: In function ‘dm_table_set_integrity’:
drivers/md/dm-table.c:1193:2: warning: statement with no effect
[-Wunused-value]
net/ipv4/netfilter/nf_nat_standalone.c: In function
‘nf_nat_standalone_init’:
net/ipv4/netfilter/nf_nat_standalone.c:287:2: warning: the comparison
will always evaluate as ‘true’ for the address of
‘nat_decode_session’ will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c: In function
‘nf_nat_protocol_unregister’:
net/ipv4/netfilter/nf_nat_core.c:528:2: warning: the comparison will
always evaluate as ‘true’ for the address of
‘nf_nat_unknown_protocol’ will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c: In function ‘nf_nat_init’:
net/ipv4/netfilter/nf_nat_core.c:739:3: warning: the comparison will
always evaluate as ‘true’ for the address of
‘nf_nat_unknown_protocol’ will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:740:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_protocol_tcp’
will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:741:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_protocol_udp’
will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:742:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_protocol_icmp’
will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:751:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_seq_adjust’ will
never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:753:2: warning: the comparison will
always evaluate as ‘true’ for the address of
‘nfnetlink_parse_nat_setup’ will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_core.c:756:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_get_offset’ will
never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_ftp.c: In function ‘nf_nat_ftp_init’:
net/ipv4/netfilter/nf_nat_ftp.c:123:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘nf_nat_ftp’ will never
be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_irc.c: In function ‘nf_nat_irc_init’:
net/ipv4/netfilter/nf_nat_irc.c:85:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘help’ will never be
NULL [-Waddress]
In file included from drivers/usb/host/ohci-hcd.c:101:0:
drivers/usb/host/ohci-dbg.c: In function ‘fill_registers_buffer’:
drivers/usb/host/ohci-dbg.c:642:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘next’ will never be
NULL [-Waddress]
drivers/usb/host/ohci-dbg.c:661:3: warning: the comparison will
always evaluate as ‘true’ for the address of ‘next’ will never be
NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c: In function ‘nf_nat_sip_init’:
net/ipv4/netfilter/nf_nat_sip.c:550:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sip’ will never
be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:551:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sip_seq_adjust’
will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:552:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sip_expect’ will
never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:553:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sdp_addr’ will
never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:554:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sdp_port’ will
never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:555:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sdp_session’
will never be NULL [-Waddress]
net/ipv4/netfilter/nf_nat_sip.c:556:2: warning: the comparison will
always evaluate as ‘true’ for the address of ‘ip_nat_sdp_media’ will
never be NULL [-Waddress]
Setup is 15260 bytes (padded to 15360 bytes).
System is 4654 kB
CRC 43bdb86
Not pretty :-/
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists