On Thursday, July 11, 2024 17:43 IST, "Rafael J. Wysocki" rafael@kernel.org wrote:

Hi,

On Thu, Jul 11, 2024 at 2:10 PM Shreeya Patel shreeya.patel@collabora.com wrote:

...

Add RTC wakeup alarm for devices to resume after specific time interval. This improvement in the test will help in enabling this test in the CI systems and will eliminate the need of manual intervention for resuming back the devices after suspend/hibernation.

Why don't you use rtcwake for this?

You are right, using rtcwake would have been better here. I'll send a v2 with the changes

Thanks, Shreeya Patel

...

Signed-off-by: Shreeya Patel shreeya.patel@collabora.com

tools/testing/selftests/cpufreq/cpufreq.sh | 24 ++++++++++++++++++++++ tools/testing/selftests/cpufreq/main.sh | 13 +++++++++++- 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/tools/testing/selftests/cpufreq/cpufreq.sh b/tools/testing/selftests/cpufreq/cpufreq.sh index a8b1dbc0a3a5..a0f5b944a8fe 100755 --- a/tools/testing/selftests/cpufreq/cpufreq.sh +++ b/tools/testing/selftests/cpufreq/cpufreq.sh @@ -231,6 +231,30 @@ do_suspend()

            for i in `seq 1 $2`; do
                    printf "Starting $1\n"

•                   if [ "$3" = "rtc" ]; then
  

•                           now=$(date +%s)
  

•                           wakeup_time=$((now + 15)) # Wake up after 15 seconds
  

•                           echo $wakeup_time > /sys/class/rtc/rtc0/wakealarm
  

•                           if [ $? -ne 0 ]; then
  

•                                   printf "Failed to set RTC wake alarm\n"
  

•                                   return 1
  

•                           fi
  

•                           # Enable the RTC as a wakeup source
  

•                           echo enabled > /sys/class/rtc/rtc0/device/power/wakeup
  

•                           if [ $? -ne 0 ]; then
  

•                                   printf "Failed to set RTC wake alarm\n"
  

•                                   return 1
  

•                           fi
  

•                           # Reset the wakeup alarm
  

•                           echo 0 > /sys/class/rtc/rtc0/wakealarm
  

•                   fi
  

•                   echo $filename > $SYSFS/power/state
                    printf "Came out of $1\n"
  
  

diff --git a/tools/testing/selftests/cpufreq/main.sh b/tools/testing/selftests/cpufreq/main.sh index a0eb84cf7167..f12ff7416e41 100755 --- a/tools/testing/selftests/cpufreq/main.sh +++ b/tools/testing/selftests/cpufreq/main.sh @@ -24,6 +24,8 @@ helpme() [-t <basic: Basic cpufreq testing suspend: suspend/resume, hibernate: hibernate/resume,

•        suspend_rtc: suspend/resume back using the RTC wakeup alarm,
  

•        hibernate_rtc: hibernate/resume back using the RTC wakeup alarm,
         modtest: test driver or governor modules. Only to be used with -d or -g options,
         sptest1: Simple governor switch to produce lockdep.
         sptest2: Concurrent governor switch to produce lockdep.
  

@@ -76,7 +78,8 @@ parse_arguments() helpme ;;

•                   t) # --func_type (Function to perform: basic, suspend, hibernate, modtest, sptest1/2/3/4 (default: basic))
  

•                   t) # --func_type (Function to perform: basic, suspend, hibernate,
  

•                      # suspend_rtc, hibernate_rtc, modtest, sptest1/2/3/4 (default: basic))
                            FUNC=$OPTARG
                            ;;
  
  

@@ -121,6 +124,14 @@ do_test() do_suspend "hibernate" 1 ;;

•           "suspend_rtc")
  

•            do_suspend "suspend" 1 rtc
  

•            ;;
  

•            "hibernate_rtc")
  

•            do_suspend "hibernate" 1 rtc
  

•            ;;
  

•           "modtest")
            # Do we have modules in place?
            if [ -z $DRIVER_MOD ] && [ -z $GOVERNOR_MOD ]; then
  

-- 2.39.2

---

Kernel mailing list -- kernel@mailman.collabora.com To unsubscribe send an email to kernel-leave@mailman.collabora.com This list is managed by https://mailman.collabora.com