[SOLVED] lid ACPI actions being run twice on 3.12 kernel

bl0tt0 · 12-19-2013, 02:22 PM

So I've been tooling with upgrading to the 3.12 kernel on my thinkpad T400, and I've discovered that my custom acpi scripts are now being run twice on lid events for some reason. Has anyone else run into this?

Code:

# /etc/acpi/events/lid
event=(button/lid)
action=/etc/acpi/actions/smartsleep.sh %e

Code:

 /etc/acpi/actions/smartsleep.sh:
#!/bin/sh

BAT_DIR=/sys/class/power_supply/BAT0
FULL_BAT=`cat ${BAT_DIR}/energy_full`
CUR_BAT=`cat ${BAT_DIR}/energy_now`
AVG=`expr $(expr ${CUR_BAT} \* 100) / ${FULL_BAT}`

logger "Running smartsleep script"

if grep -q 1 /sys/class/power_supply/AC/online; then
  /etc/acpi/actions/sleep.sh
  exit 0
fi

if [ "$AVG" -le "20" ]; then
  echo suspend > /sys/power/disk
  /etc/acpi/actions/hibernate.sh
  exit 0
else
  /etc/acpi/actions/sleep.sh
fi

Here is the output to /var/log/messages when I close the lid:

Code:

# grep smartsleep /var/log/messages
Dec 19 14:13:03 6h057 logger: Running smartsleep script
Dec 19 14:13:19 6h057 logger: Running smartsleep script

This ACPI configuration works fine with the 3.10 kernel, and I know that they dropped the deprecated /proc/acpi/event interface in 3.12, but I didn't think I was using that anyway.

xflow7 · 12-19-2013, 02:32 PM

I would have guessed the second invocation is coming when you open the lid back up (which also causes a change of state of the lid switch).

bl0tt0 · 12-19-2013, 06:03 PM

Quote:

Originally Posted by xflow7

I would have guessed the second invocation is coming when you open the lid back up (which also causes a change of state of the lid switch).

By "Bob", you're right! Just added a line to my script to exit if the lid button state is open, and everything's working again! Funny that the issue only showed up with the 3.12 kernel release though...