Re: [RFC][PATCH] tracing: Remove useless 4 bytes of padding from every event

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[ Fixes David Sharps email. I have two versions, and picked the broken 
  one :-p ]


On Wed, 2012-05-16 at 12:36 -0700, Arjan van de Ven wrote:
> On 5/16/2012 12:33 PM, Linus Torvalds wrote:
> > On Wed, May 16, 2012 at 6:44 AM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> >>
> >> But this change will break PowerTop v1. Thus my question is, how long do
> >> we need to keep this wasted space in the ring buffers to satisfy an out
> >> of date tool?
> > 
> > The wasted space seems of limited importance.

Depends on who you ask. For you, it's probably of no importance. For
someone trying to get as many events as possible, it's of much greater
importance. Lack of events in a report is one of the biggest issues we
have. Especially for those that are trying to debug an issue and needs
as much info as possible. (function tracing sucks up a hell of a lot)

But that said, it doesn't go against what you stated. "limited
importance". Its importance is limited to a group of people.

> > 
> > More important is to check which distros have the new powertop.
> > 
> > F16 and F17 seem to have powertop-1.98, which I assume is the new
> > world order already. But maybe I assume incorrectly.
> 
> this is the one that will break

Shh ;-)

> 
> > 
> > F14 (which I personally still use, since it doesn't have gnome3) is 1.13.
> 
> this one is fine, it does not use perf events at all.

Oo oo! We can change it and Linus won't know.

-- Steve


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[Index of Archives]

  Powered by Linux

[Older Kernel Discussion]     [Yosemite National Park Forum]     [Large Format Photos]     [Gimp]     [Yosemite Photos]     [Stuff]     [Index of Other Archives]