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

Groups > linux.kernel > #1387182

Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to "panic" on a kernel panic

Show all headers | View raw

Hi Ezequiel,

On 04/25/2016 06:27 PM, Ezequiel Garcia wrote:
> On 25 April 2016 at 03:56, Jacek Anaszewski <j.anaszewski@samsung.com> wrote:
>> On 04/24/2016 11:29 AM, Pavel Machek wrote:
>>>
>>> On Sun 2016-04-24 11:25:51, Pavel Machek wrote:
>>>>
>>>> On Mon 2016-04-04 17:22:02, Ezequiel Garcia wrote:
>>>>>
>>>>> This commit adds a new led_cdev flag LED_BLINK_AT_PANIC, which
>>>>> allows to mark a specific LED to be switched to the "panic"
>>>>> trigger, on a kernel panic.
>>>>>
>>>>> This is useful to allow the user to assign a regular trigger
>>>>> to a given LED, and still blink that LED on a kernel panic.
>>>>>
>>>>> Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
>>>>
>>>>
>>>>
>>>>>    drivers/leds/led-triggers.c | 52
>>>>> +++++++++++++++++++++++++++++++++++++++++++++
>>>>>    include/linux/leds.h        |  1 +
>>>>
>>>>
>>>> Could we get this out of the core? I'm pretty sure most users are not
>>>> interested...
>>
>>
>> Good point.
>>
>
> Not sure how we can get it out of the core, and still implement it.
>
> The goal is to run-time switch user-specified LEDs and blink them when
> the kernel panics, and so it needs to mess up with some core private
> structures.
>
> Unless we don't want this feature at all.
>

It seems that the entire added code can be easily moved to a separate
module, let's say led-panic-notifier.c ? It should select
LEDS_TRIGGER_PANIC, as it would make no sense to have the former enabled
without the latter.

-- 
Best regards,
Jacek Anaszewski

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

Thread

Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to  "panic" on a kernel panic Pavel Machek <pavel@ucw.cz> - 2016-04-24 11:30 +0200
  Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to "panic"  on a kernel panic Jacek Anaszewski <j.anaszewski@samsung.com> - 2016-04-25 09:00 +0200
    Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to  "panic" on a kernel panic Ezequiel Garcia <ezequiel@vanguardiasur.com.ar> - 2016-04-25 18:30 +0200
      Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to "panic"  on a kernel panic Jacek Anaszewski <j.anaszewski@samsung.com> - 2016-04-26 09:20 +0200
        Re: [PATCH 1/5] leds: triggers: Allow to switch the trigger to  "panic" on a kernel panic Ezequiel Garcia <ezequiel@vanguardiasur.com.ar> - 2016-04-27 22:20 +0200

csiph-web