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]
Date:	Mon, 11 Jul 2011 10:34:05 +0300
From:	Pekka Enberg <penberg@...nel.org>
To:	Steven Rostedt <rostedt@...dmis.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>,
	Arjan van de Ven <arjan@...radead.org>
Subject: Re: [Bug #38132] [Warning] 2.6.39.x latencytop

On Mon, Jul 11, 2011 at 12:25 AM, Steven Rostedt <rostedt@...dmis.org> wrote:
> 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.

AFAICT, if you go ahead with the deprecation and actually remove the
file, you'll break latencytop fsync tracing.

On Mon, Jul 11, 2011 at 12:25 AM, Steven Rostedt <rostedt@...dmis.org> wrote:
> Question: Why is latency top even touching this file?

Looking at the sources, it seems to use it to enable fsync tracing. I
don't know the details so lets CC Arjan.
--
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