Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-08 Thread Thomas Gleixner
On Thu, 8 Dec 2016, Pavel Machek wrote: > On Thu 2016-12-08 10:18:13, Thomas Gleixner wrote: > > 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 p

Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-08 Thread Pavel Machek
On Thu 2016-12-08 10:18:13, Thomas Gleixner wrote: > 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 > > > > > > > > > > Upon removal of the i7300_idle driver, the idle_notifer i

Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-08 Thread Thomas Gleixner
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 > > > > > > > > Upon removal of the i7300_idle driver, the idle_notifer is unused. > > > > > > Actually... do we want to keep this this

Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-07 Thread Pavel Machek
On Wed 2016-12-07 12:46:12, Thomas Gleixner wrote: > On Tue, 6 Dec 2016, Pavel Machek wrote: > > > From: Len Brown > > > > > > 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 i

Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-07 Thread Thomas Gleixner
On Tue, 6 Dec 2016, Pavel Machek wrote: > > From: Len Brown > > > > 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 proces

Re: [PATCH 2/5] x86: remove idle_notifier

2016-12-06 Thread Pavel Machek
Hi! > From: Len Brown > > 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

[PATCH 2/5] x86: remove idle_notifier

2016-11-17 Thread Len Brown
From: Len Brown Upon removal of the i7300_idle driver, the idle_notifer is unused. Signed-off-by: Len Brown Acked-by: Peter Zijlstra (Intel) --- arch/x86/include/asm/idle.h | 7 --- arch/x86/kernel/process.c | 15 --- 2 files changed, 22 deletions(-) diff --git a/arch/x86/