Messages in this thread | | | Date | Thu, 10 Jun 2010 12:30:33 +1000 | From | Anton Blanchard <> | Subject | Re: [PATCH v2] panic: keep blinking in spite of long spin timer mode |
| |
Hi,
> Morse code? Kidding? > > Sorry, no. Nobody who uses this feature will know what what > "words per minute" means. It's nutty! > > Please remind me why we're making this configurable at all. Can't we > just hardwire the thing to 1Hz or something? Add an > im_using_a_hypervisor boot option or something, if necessary?
I agree. The panic_blink() interface is quite painful and I have no idea why someone would want to configure the blink frequency of their keyboard LED when panicing. Maybe they want to match it to the beat of their techno music.
Since the keyboard LED default is a transition every 0.5s, why don't we just remove i8042.panicblink and change all users (all 2 of them) to expect a 2 HZ call rate? The hypervisor case should be fine with 0.5s mdelays, so we end up removing that special case.
I would have said 1 HZ, but it seems like the default was chosen to be different to kdb:
/* * We expect frequency to be about 1/2s. KDB uses about 1s. * Make sure they are different. */
No idea if that comment is still valid.
Anton
| |