Re: [RESEND PATCH v5 2/2] leds: trigger: Add block device LED trigger

From: Marek Behún
Date: Fri Oct 08 2021 - 06:01:28 EST


On Wed, 6 Oct 2021 11:07:06 -0500
Ian Pilcher <arequipeno@xxxxxxxxx> wrote:

> I have feeling that per-LED work items are likely to cause contention
> for the mutex, since they will probably all have the same (default)
> interval and they will usually be set up at about the same time (i.e.
> at system boot). Instead, I would propose to have a single work item
> that is simply scheduled for the next time work is "needed" and then
> checks all LEDs that are due at that time.

What about creating one work struct for all different interval values?

That way if the user never changes the interval, there will be only one
work struct.

I wonder if this can be done in a sensible (i.e. not overcomplicated
code) way.

Marek