Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1537759

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

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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.

Best regards,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/5] x86: remove idle_notifier Pavel Machek <pavel@ucw.cz> - 2016-12-06 18:00 +0100
  Re: [PATCH 2/5] x86: remove idle_notifier Thomas Gleixner <tglx@linutronix.de> - 2016-12-07 12:50 +0100
    Re: [PATCH 2/5] x86: remove idle_notifier Pavel Machek <pavel@ucw.cz> - 2016-12-07 14:50 +0100
      Re: [PATCH 2/5] x86: remove idle_notifier Thomas Gleixner <tglx@linutronix.de> - 2016-12-08 10:30 +0100
        Re: [PATCH 2/5] x86: remove idle_notifier Pavel Machek <pavel@ucw.cz> - 2016-12-08 23:10 +0100
          Re: [PATCH 2/5] x86: remove idle_notifier Thomas Gleixner <tglx@linutronix.de> - 2016-12-09 00:00 +0100

csiph-web