From: Ingo Molnar <mingo@elte.hu>
To: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Steven Rostedt <srostedt@redhat.com>,
Stephen Rothwell <sfr@canb.auug.org.au>,
linux-next@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
Frederic Weisbecker <fweisbec@gmail.com>
Subject: Re: linux-next: Tree for March 11 (tracing)
Date: Wed, 18 Mar 2009 17:49:26 +0100 [thread overview]
Message-ID: <20090318164926.GC31315@elte.hu> (raw)
In-Reply-To: <49C124DE.9090001@oracle.com>
* Randy Dunlap <randy.dunlap@oracle.com> wrote:
> Ingo Molnar wrote:
> > * Randy Dunlap <randy.dunlap@oracle.com> wrote:
> >
> >> Ingo Molnar wrote:
> >>> * Randy Dunlap <randy.dunlap@oracle.com> wrote:
> >>>
> >>>> On Thu, 12 Mar 2009 12:26:21 -0400 Steven Rostedt wrote:
> >>>>
> >>>>> On Thu, 2009-03-12 at 09:17 -0700, Randy Dunlap wrote:
> >>>>>> [adding cc:s]
> >>>>>>
> >>>>>> [same report for March 12]
> >>>>>>
> >>>>>> Randy Dunlap wrote:
> >>>>>>> Stephen Rothwell wrote:
> >>>>>>>> Hi all,
> >>>>>>>>
> >>>>>>>> Changes since 20090310:
> >>>>>>> Building on i386 generates a ton of printk format warnings:
> >>>>>>>
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 5 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 6 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 9 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 10 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 13 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 14 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 17 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 18 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 21 has type 'unsigned int'
> >>>>>>> kernel/trace/trace_events.c:470: warning: format '%lu' expects type 'long unsigned int', but argument 22 has type 'unsigned int'
> >>>>>>>
> >>>>> I believe this is corrected in Ingo's tip tree. I changed %lu to %zu to
> >>>>> handle the "sizeof()" case. The fix was suggested by Andrew Morton.
> >>>> This build warning is still around (20090318).
> >>>> Is the fix not in some branch that is imported into linux-next or what?
> >>> be patient.
> >>>
> >>> Ingo
> >> I think that 7 days is being patient for a simple build fix.
> >
> > s/build fix/harmless build warning fix
>
> 180+ lines of noise in a build log.
>
> > If you are interested in having a resolution you can git-merge the
> > latest development tree yourself and you can get rid of that
> > warning.
> >
> > Of course that way you'd expose yourself to even fresher code,
> > potentially with much more serious breakages.
> >
> > It's a balance of freshness versus stability, and that balance is
> > kept by maintainers.
> >
> > If you want the latest development code - go engage with the
> > development trees directly.
> >
> > If you want something that is relatively new (i.e. 1-2 weeks fresh)
> > but works on the range of systems we test, use what you get in
> > linux-next.
> >
> > It's your choice which one you pick.
> >
> > But you cannot have both.
> >
> > If you genuinely think you can have it both, by all means i
> > encourage you to try it - it's all open source so you can run your
> > own tree. Just please dont feel entitled to demand it from others.
>
> Thanks for the explanation. That's what I tried to ask for
> to begin with. I guess that I have a language problem.
Note, six hours ago i reintegrated all the auto-next branches so
this warning should be gone in the next linux-next iteration - they
are now traded for brand new bugs ;-)
Ingo
prev parent reply other threads:[~2009-03-18 16:49 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-11 11:59 linux-next: Tree for March 11 Stephen Rothwell
2009-03-11 17:12 ` linux-next: Tree for March 11 (staging/multimedia) Randy Dunlap
2009-03-11 18:11 ` Greg KH
2009-03-11 18:21 ` Randy Dunlap
2009-03-11 18:24 ` Greg KH
2009-03-11 18:27 ` Hans Verkuil
2009-03-11 17:36 ` linux-next: Tree for March 11 (tracing) Randy Dunlap
2009-03-12 16:17 ` Randy Dunlap
2009-03-12 16:26 ` Steven Rostedt
2009-03-12 16:32 ` Randy Dunlap
2009-03-18 15:46 ` Randy Dunlap
2009-03-18 16:06 ` Ingo Molnar
2009-03-18 16:13 ` Randy Dunlap
2009-03-18 16:32 ` Ingo Molnar
2009-03-18 16:44 ` Randy Dunlap
2009-03-18 16:49 ` Ingo Molnar [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090318164926.GC31315@elte.hu \
--to=mingo@elte.hu \
--cc=fweisbec@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=randy.dunlap@oracle.com \
--cc=sfr@canb.auug.org.au \
--cc=srostedt@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).