[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1292513415.6803.4604.camel@twins>
Date: Thu, 16 Dec 2010 16:30:15 +0100
From: Peter Zijlstra <peterz@...radead.org>
To: Frederic Weisbecker <fweisbec@...il.com>
Cc: Chris Mason <chris.mason@...cle.com>,
Frank Rowand <frank.rowand@...sony.com>,
Ingo Molnar <mingo@...e.hu>,
Thomas Gleixner <tglx@...utronix.de>,
Mike Galbraith <efault@....de>,
Oleg Nesterov <oleg@...hat.com>, Paul Turner <pjt@...gle.com>,
Jens Axboe <axboe@...nel.dk>, linux-kernel@...r.kernel.org
Subject: Re: [RFC][PATCH 3/5] sched: Change the ttwu success details
On Thu, 2010-12-16 at 16:27 +0100, Peter Zijlstra wrote:
> On Thu, 2010-12-16 at 16:23 +0100, Frederic Weisbecker wrote:
>
> > > - TP_printk("comm=%s pid=%d prio=%d success=%d target_cpu=%03d",
> > > + TP_printk("comm=%s pid=%d prio=%d target_cpu=%03d",
> > > __entry->comm, __entry->pid, __entry->prio,
> > > - __entry->success, __entry->target_cpu)
> > > + __entry->target_cpu)
> >
> > Note we'll need to fix some perf scripts after that. And also perf sched,
> > probably perf timechart and so on...
>
> Do any of those actually use the success parameter? If not, then me
> removing it shouldn't break those tools since they're supposed to parse
> the format stuff and not notice it missing ;-)
Alternatively we could always call trace_sched_wakeup() and make success
reflect actual wakeup success.
--
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