linux-kernel - Re: warning in scheduler

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 29 Jul 2008 07:19:16 +1000
From:	Benjamin Herrenschmidt <benh@...nel.crashing.org>
To:	Matthew Wilcox <matthew@....cx>
Cc:	Ingo Molnar <mingo@...e.hu>, Peter Zijlstra <peterz@...radead.org>,
	"H. Peter Anvin" <hpa@...or.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, stephen@...cksapphire.com,
	benm@...metric.co.nz, jkosina@...e.cz
Subject: Re: warning in scheduler

On Mon, 2008-07-28 at 12:39 -0600, Matthew Wilcox wrote:
> On Mon, Jul 28, 2008 at 08:40:22PM +1000, Benjamin Herrenschmidt wrote:
> > I tend to agree. I might look into doing it for powerpc around rc3 or
> > so, when things have settled a bit.
> 
> It'd be nice if allmodconfig even compiled:
> 
> drivers/char/pcmcia/ipwireless/hardware.c:571: error: invalid use of undefined type 'struct ipw_network'
> 
> Not to mention the qla_attr patch:
> 
> +#include <linux/delay.h>
> 
> There's probably others, I just haven't got far enough in the
> compilation process yet.

Yup, it's been a pretty bad merge window (at least it's my impression as
my first one as maintainer), mostly fixing compile bugs all the time.

> But I am seeing lots of warnings _without_ the patch applied:
> 
> fs/omfs/inode.c: In function 'omfs_fill_super':
> fs/omfs/inode.c:495: warning: format '%llx' expects type 'long long
> unsigned int', but argument 2 has type 'u64'
> fs/proc/base.c: In function 'do_io_accounting':
> fs/proc/base.c:2429: warning: format '%llu' expects type 'long long
> unsigned int', but argument 3 has type 'u64'
> fs/ubifs/dir.c: In function 'ubifs_readdir':
> fs/ubifs/dir.c:437: warning: format '%llu' expects type 'long long
> unsigned int', but argument 5 has type '__le64'
> 
> (other warnings snipped for brevity).

I have a handful on powerpc too.

BTW. I'm looking at adding a %pR to print a struct resource (or maybe
a %pr for a resource_size_t and %pR for the whole resource). That would
help with the whole business of the thing being either 32 or 64 bits.

Ben.


--
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/