mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Dave Jones <davej@codemonkey.org.uk>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Kees Cook <keescook@chromium.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Xing Gao <xgao01@email.wm.edu>,
	Jessica Frazelle <me@jessfraz.com>,
	John Stultz <john.stultz@linaro.org>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Jonathan Corbet <corbet@lwn.net>, Tejun Heo <tj@kernel.org>,
	Lai Jiangshan <jiangshanlai@gmail.com>,
	Petr Mladek <pmladek@suse.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Oleg Nesterov <oleg@redhat.com>,
	Nicolas Iooss <nicolas.iooss_linux@m4x.org>,
	Nicolas Pitre <nicolas.pitre@linaro.org>,
	Richard Cochran <richardcochran@gmail.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	Michal Marek <mmarek@suse.com>,
	Josh Poimboeuf <jpoimboe@redhat.com>,
	Dmitry Vyukov <dvyukov@google.com>,
	Olof Johansson <olof@lixom.net>, Shuah Khan <shuah@kernel.org>,
	"open list:DOCUMENTATION" <linux-doc@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	"kernel-hardening@lists.openwall.com" 
	<kernel-hardening@lists.openwall.com>
Subject: Re: [PATCH v2] time: Remove CONFIG_TIMER_STATS
Date: Wed, 8 Feb 2017 15:30:27 -0500	[thread overview]
Message-ID: <20170208203027.r6v4apovtyi6nuu3@codemonkey.org.uk> (raw)
In-Reply-To: <CA+55aFzz25AxbG7vJNNKAOEpAJFYnd=D0wJrL-tEN-SREk6kSQ@mail.gmail.com>

On Wed, Feb 08, 2017 at 11:54:30AM -0800, Linus Torvalds wrote:
 > On Wed, Feb 8, 2017 at 11:26 AM, Kees Cook <keescook@chromium.org> wrote:
 > >
 > > Given that the tracer can give the same information, this patch entirely
 > > removes CONFIG_TIMER_STATS.
 > >
 > > Suggested-by: Thomas Gleixner <tglx@linutronix.de>
 > > Signed-off-by: Kees Cook <keescook@chromium.org>
 > > Acked-by: John Stultz <john.stultz@linaro.org>
 > 
 > Looks good to me. Let's wait for the 4.11 merge window though. I'm
 > assuming I'll get this through the tip timer tree..

>From a quick look at the source, powertop uses this file, and appears to
handle it gracefully if it fails to open, but it will lose functionality
to determine if a timer is deferred.

	Dave

  reply	other threads:[~2017-02-08 20:32 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-08 19:26 Kees Cook
2017-02-08 19:54 ` Linus Torvalds
2017-02-08 20:30   ` Dave Jones [this message]
2017-02-09 13:41     ` Thomas Gleixner
2017-02-09 13:32 ` [kernel-hardening] " Yann Droneaud
2017-02-10 10:18 ` [tip:timers/core] " tip-bot for Kees Cook

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=20170208203027.r6v4apovtyi6nuu3@codemonkey.org.uk \
    --to=davej@codemonkey.org.uk \
    --cc=akpm@linux-foundation.org \
    --cc=corbet@lwn.net \
    --cc=dvyukov@google.com \
    --cc=ebiederm@xmission.com \
    --cc=jiangshanlai@gmail.com \
    --cc=john.stultz@linaro.org \
    --cc=jpoimboe@redhat.com \
    --cc=keescook@chromium.org \
    --cc=kernel-hardening@lists.openwall.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=me@jessfraz.com \
    --cc=mmarek@suse.com \
    --cc=nicolas.iooss_linux@m4x.org \
    --cc=nicolas.pitre@linaro.org \
    --cc=oleg@redhat.com \
    --cc=olof@lixom.net \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=pmladek@suse.com \
    --cc=richardcochran@gmail.com \
    --cc=shuah@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=tj@kernel.org \
    --cc=torvalds@linux-foundation.org \
    --cc=xgao01@email.wm.edu \
    /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

all inboxes | Powered by JetHome®