From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752753AbcLHJVC (ORCPT <rfc822;w@1wt.eu>);
        Thu, 8 Dec 2016 04:21:02 -0500
Received: from Galois.linutronix.de ([146.0.238.70]:45497 "EHLO
        Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751764AbcLHJVA (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 8 Dec 2016 04:21:00 -0500
Date: Thu, 8 Dec 2016 10:18:13 +0100 (CET)
From: Thomas Gleixner <tglx@linutronix.de>
To: Pavel Machek <pavel@ucw.cz>
cc: Len Brown <lenb@kernel.org>, x86@kernel.org,
        linux-kernel@vger.kernel.org, Len Brown <len.brown@intel.com>
Subject: Re: [PATCH 2/5] x86: remove idle_notifier
In-Reply-To: <20161207134033.GA22493@amd>
Message-ID: <alpine.DEB.2.20.1612081014280.3400@nanos>
References: <ad6a044e57cc75f44cc8621abe846e58f7882243.1479449716.git.len.brown@intel.com> <f15385a82ec4bf51f4f06777193d83f03b28cfdd.1479449716.git.len.brown@intel.com> <20161206165420.GA4406@amd> <alpine.DEB.2.20.1612071243410.3448@nanos>
 <20161207134033.GA22493@amd>
User-Agent: Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, 7 Dec 2016, Pavel Machek wrote:
> On Wed 2016-12-07 12:46:12, Thomas Gleixner wrote:
> > On Tue, 6 Dec 2016, Pavel Machek wrote:
> > > > From: Len Brown <len.brown@intel.com>
> > > > 
> > > > Upon removal of the i7300_idle driver, the idle_notifer is unused.
> > > 
> > > Actually... do we want to keep this this one? There's "cpu is loaded"
> > > led trigger, which is implemented on arm but not on x86.
> > > 
> > > I tried to patch process.c, but this would be easier (and does not
> > > require new hooks).
> > > 
> > > Does that make sense?
> > 
> > No. Notifiers are crap.
> > 
> > I assume this LED thing is just for entertainment purposes as I cannot
> > figure out why it would be useful. ARM has it does not qualify as useful.
> 
> Well, on low-speed systems, that led is actually quite important. You
> know if your keypress was registered, and CPU is just slow, or if you
> need to press it again.

I'm dealing with low-speed systems for 20 years now and that LED was never
important for me, quite the contrary, it's annoying to have the extra work
in the idle wakeup path which causes extra pointless latency. If you can't
figure out your keypress lag without that LED then feel free to patch your
own kernel, but stop trying to impose that nonsense on everyone.

Thanks,

	tglx