linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

      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).