[PATCH] kselftest: cpufreq: Add RTC wakeup alarm

Shreeya Patel posted 1 patch 1 year, 5 months ago
There is a newer version of this series
tools/testing/selftests/cpufreq/cpufreq.sh | 24 ++++++++++++++++++++++
tools/testing/selftests/cpufreq/main.sh    | 13 +++++++++++-
2 files changed, 36 insertions(+), 1 deletion(-)
[PATCH] kselftest: cpufreq: Add RTC wakeup alarm
Posted by Shreeya Patel 1 year, 5 months ago
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.

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
Re: [PATCH] kselftest: cpufreq: Add RTC wakeup alarm
Posted by Rafael J. Wysocki 1 year, 5 months ago
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?

> 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
>
>
Re: [PATCH] kselftest: cpufreq:Add RTC wakeup alarm
Posted by Shreeya Patel 1 year, 5 months ago
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