rfkill: prevent unnecessary event generation
Prevent unnecessary rfkill event generation when the state has not actually changed. These events have to be delivered to relevant userspace processes, causing these processes to wake up and do something while they could as well have slept. This obviously results in more CPU usage, longer time-to-sleep-again and therefore higher power consumption. Signed-off-by: Vitaly Wool <vitalywool@gmail.com> Signed-off-by: Mykyta Iziumtsev <nikita.izyumtsev@gmail.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
parent
c6f219dc83
commit
eab48345c2
@ -270,6 +270,7 @@ static bool __rfkill_set_hw_state(struct rfkill *rfkill,
|
||||
static void rfkill_set_block(struct rfkill *rfkill, bool blocked)
|
||||
{
|
||||
unsigned long flags;
|
||||
bool prev, curr;
|
||||
int err;
|
||||
|
||||
if (unlikely(rfkill->dev.power.power_state.event & PM_EVENT_SLEEP))
|
||||
@ -284,6 +285,8 @@ static void rfkill_set_block(struct rfkill *rfkill, bool blocked)
|
||||
rfkill->ops->query(rfkill, rfkill->data);
|
||||
|
||||
spin_lock_irqsave(&rfkill->lock, flags);
|
||||
prev = rfkill->state & RFKILL_BLOCK_SW;
|
||||
|
||||
if (rfkill->state & RFKILL_BLOCK_SW)
|
||||
rfkill->state |= RFKILL_BLOCK_SW_PREV;
|
||||
else
|
||||
@ -313,10 +316,13 @@ static void rfkill_set_block(struct rfkill *rfkill, bool blocked)
|
||||
}
|
||||
rfkill->state &= ~RFKILL_BLOCK_SW_SETCALL;
|
||||
rfkill->state &= ~RFKILL_BLOCK_SW_PREV;
|
||||
curr = rfkill->state & RFKILL_BLOCK_SW;
|
||||
spin_unlock_irqrestore(&rfkill->lock, flags);
|
||||
|
||||
rfkill_led_trigger_event(rfkill);
|
||||
rfkill_event(rfkill);
|
||||
|
||||
if (prev != curr)
|
||||
rfkill_event(rfkill);
|
||||
}
|
||||
|
||||
#ifdef CONFIG_RFKILL_INPUT
|
||||
|
Loading…
Reference in New Issue
Block a user