linux-kernel - Re: [Bug #38132] [Warning] 2.6.39.x latencytop

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1310333145.2269.6.camel@gandalf.stny.rr.com>
Date:	Sun, 10 Jul 2011 17:25:45 -0400
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Pekka Enberg <penberg@...nel.org>
Cc:	"Rafael J. Wysocki" <rjw@...k.pl>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Kernel Testers List <kernel-testers@...r.kernel.org>,
	Maciej Rutecki <maciej.rutecki@...il.com>,
	Florian Mickler <florian@...kler.org>,
	Andrew Watts <akwatts@...il.com>, Ingo Molnar <mingo@...e.hu>,
	Frederic Weisbecker <fweisbec@...il.com>
Subject: Re: [Bug #38132] [Warning] 2.6.39.x latencytop

On Sun, 2011-07-10 at 14:53 +0300, Pekka Enberg wrote:
> On Sun, Jul 10, 2011 at 1:58 PM, Rafael J. Wysocki <rjw@...k.pl> wrote:
> > This message has been generated automatically as a part of a report
> > of regressions introduced between 2.6.38 and 2.6.39.
> >
> > The following bug entry is on the current list of known regressions
> > introduced between 2.6.38 and 2.6.39.  Please verify if it still should
> > be listed and let the tracking team know (either way).
> >
> > Bug-Entry       : http://bugzilla.kernel.org/show_bug.cgi?id=38132
> > Subject         : [Warning] 2.6.39.x latencytop
> > Submitter       : Andrew Watts <akwatts@...il.com>
> > Date            : 2011-06-14 17:07 (27 days old)
> > Message-ID      : <80098.5633.qm@...111013.mail.gq1.yahoo.com>
> > References      : http://marc.info/?l=linux-kernel&m=130807128506490&w=2
> 
> This is a WARN_ON added in commit 6752ab4a ("tracing: Deprecate
> tracing_enabled for tracing_on") that lets the user know about
> deprecated ABI. It's somewhat unfortunate that there's no mention in
> the changelog if everyone agreed on the deprecation or not...
> 
> That said, assuming the deprecation is OK (hi Steven, Frederic, Ingo!)
> I think this issue can be closed.

I deprecated it because I said I would for the last year ;)

The tracing_enabled never did what it was suppose to do. It was suppose
to be a "quick" way to disable tracing without fully disabling it. It
was suppose to try to turn things off to a low overhead but not fully
off without any overhead. The problem is that it never really worked
well. The only tracer that does anything different with it as suppose to
tracing_on, is the irqsoff tracers. They stop looking at max latencies
when you echo 0 into tracing_enabled. But that's it. Everything else,
it's just the same as echoing 0 into tracing_on. I think the irqsoff
tracers now even look at tracing_on if it should trace a max latency or
not, so tracing_enabled doesn't even help with that.

We can keep it around but I don't see any good reason for it besides not
having latency top give a warning.

Question: Why is latency top even touching this file?

-- Steve


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