linaro-dev

linaro-dev@lists.linaro.org

4332 discussions

interactive governor issue with linked cpufreq feature and hotplug governor enabled for smp system

by zhoujie wu

Hi Dear Poynor/All cpufreq owner, I found a kernel panic case caused by interactive governor when we enabled both hotplug governor and linked cpufreq features. Below is the background and root cause, We are using the latest interactive governor on dual ca9 smp system, and linked cpufreq feature(cpumask_setall(policy->cpus) for each cpus) is enabled for them. Based on it, cpufreq governor could consider the both cpu's workload and make decision for the smp cores. And we also have a hotplug governor to monitor the system workload to make decision when will cpu1 could be hotplugged. 1. The default cpufreq governor during boot up is userspace. 2. After system is boot up, hotplug governor found system has no workload and cpu1 was hotplugged. Hotplug notifier will set cpu0’s policy->cpus only for cpu0. 3. After that, Cpufreq governor switched to Interactive, and its CPUFREQ_GOV_START will only init the cpu0’s info struct, such as pcpu->policy and governor_enabled etc. 4. A boost event comes, such as touching the screen, it will lead to CPU1 is plugged in at first, that means cpu0’s policy->cpus is linked to cpu0&cpu1 again, but it will not call interactive governor’s CPUFREQ_GOV_START to initialize cpu1’s info struct. 5. After that cpu0’s frequency changed, and it will notify interactive governor, in the notifier function(cpufreq_interactive_notifier) it will call update_load and access CPU1’s pcpu->policy->cur, but it is never initialized and lead to kernel panic. In current interactive governor, if the governor is started after cpu1 is plugged out, later even the cpu1 is plugged in, it still won’t consider cpu1’s status as it thinks cpu1’s governor_enabled is still 0, cpu1's profiling timer will just return in case it thinks interactive governor is not enabled on cpu1. So the linked feature could not ensure it makes decision based on the max load of cpu0 and cpu1 on smp. It is not what we expected. But there is no such issue in b.L system as all cpu has no link relationship. Looks like interactive governor doesn’t consider much for SMP. I made a draft patch to solve the panic as below, but still could solve the issue interactive is enabled after cpu1 is hotplugged. Could you please take a look how to solve the interactive governor issue for SMP? Thanks. Date: Thu, 28 Mar 2013 14:03:40 +0800 Subject: [PATCH] cpufreq: interactive: fix race condition of hotplug and linked cpufreq Signed-off-by: Zhoujie Wu <zhoujiewu(a)gmail.com> --- drivers/cpufreq/cpufreq_interactive.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/cpufreq/cpufreq_interactive.c b/drivers/cpufreq/cpufreq_interactive.c index 7d1952c..6941201 100644 --- a/drivers/cpufreq/cpufreq_interactive.c +++ b/drivers/cpufreq/cpufreq_interactive.c @@ -565,6 +565,8 @@ static int cpufreq_interactive_notifier( for_each_cpu(cpu, pcpu->policy->cpus) { struct cpufreq_interactive_cpuinfo *pjcpu = &per_cpu(cpuinfo, cpu); + if (!pjcpu->governor_enabled) + continue; spin_lock_irqsave(&pjcpu->load_lock, flags); update_load(cpu); spin_unlock_irqrestore(&pjcpu->load_lock, flags); --

12 years, 7 months

big.LITTLE MP Scheduler test summary-13.03

by Naresh Kamboju

Hi, Release test report big.LITTLE MP 13.03: Here is test result summary for big.LITTLE MP Scheduler test on TC2 platform with Android image sched_tests.git No of Test Cases Tests Run Tests Pass Tests Fail Absolute pass rate (%) Failure Analysis/Comments Regression 20 19 19 0 100 % mpbasicsuite 14 10 9 1 90 % mpcoresuite 10 6 5 1 80 % mpextendedsuite 20 10 6 4 60 % mploadbalance 2 2 2 0 100 % Android Build: http://snapshots.linaro.org/android/~linaro-android/vexpress-linaro-mp-13.0… Linux kernel: Linux version 3.9.0-rc3-00212-g0f66281 (jenkins-build@ip-10-165-12-211) (gcc version 4.7.3 20130226 (prerelease) (Linaro GCC 4.7-2013.03) ) #1 SMP Sat Mar 23 15:15:15 UTC 2013 Results spread sheet: https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0Ai9ggMs8bsGJdER3a… Summary: - Linaro Linux kernel updated to 3.9.0-rc3 - Following new test cases have been added to the suite. * core_test_scn03.1a * core_test_scn03.2a And * extd_test_scn01.2a * extd_test_scn01.3a * extd_test_scn03.2a * extd_test_scn03.3a - Following mpbasics test cases use to pass aways. now the deprecated tag has been added to print accurate results * "FT0005" * "FT0006" * "FT0009" * "FT00010" - Following test cases from extended suite are failing due to global load balance defect where it fails to migrate busy tasks if the cores already have other tasks pinned. * extd_test_scn01.3a * extd_test_scn03.2a * extd_test_scn03.3a - A number of fixes to the existing test case and ftrace framework Best regards Naresh Kamboju

12 years, 7 months

(no subject)

by Sanjay Singh Rawat

Basic patch to show thermal related info in the powertop tool.

12 years, 7 months

change .config/i2c not working

by Amber Graner

Hi all, This question is being moved from Zendesk to the mailing list per the Deepak's recommendation. The original requester is CC'd as well. Many thanks in advance for any help you can give Daniel. Daniel asks, "Any time I change .config and run linaro-kernel_build_cmds for Linaro 12.12 for pandaboard, I get "/linaro-kernel is not clean, please run 'make mrproper'" as an error, then by running mrproper, the .config file returns to its inital state. I want to be able to build one of the touchscreen modules into the kernel. Going into the kernel and running make or even just make_modules results in an error. I have tried to take the syntm12xx.c file out, cross compiled it as a loadable kernel module, used adb to push it on and insmod to install it, only I have not seen any i2c data with my oscilloscope on i2c using this method. I've been looking through documentation, but am still pretty unclear on how/where the i2c data bus is actually set. It doesn't appear to be set in the driver itself, so I don't know if that means there's an input configuration file someplace that does it or what. Any assistance you'd be willing to give with these issues would be extremely helpful! Or even just pointing me in the right direction. Thank you!" -- Amber Graner Community and Social Media Manager Linaro.org <http://www.linaro.org/>* **│ *Open source software for ARM SoCs* *** Follow *Linaro: *Facebook <http://www.facebook.com/pages/Linaro> | Twitter<http://twitter.com/#%21/linaroorg> | Blog <http://www.linaro.org/linaro-blog/> *+1.828.582.9469* cell *+1.828.395.1049* office irc: akgraner amber.graner(a)linaro.org (email and Google chat)

12 years, 7 months

[PATCH pmqa] pm-qa: add test for cpu topology attributes

by Sanjay Singh Rawat

From: Daniel Lezcano <daniel.lezcano(a)linaro.org> test checks for topology attributes of cpus Signed-off-by: Sanjay Singh Rawat <sanjay.rawat(a)linaro.org> --- cputopology/Android.mk | 18 ++++++++++++++++++ cputopology/Makefile | 25 +++++++++++++++++++++++++ cputopology/cputopology_01.sh | 38 ++++++++++++++++++++++++++++++++++++++ cputopology/cputopology_01.txt | 1 + cputopology/cputopology_02.sh | 33 +++++++++++++++++++++++++++++++++ cputopology/cputopology_02.txt | 1 + 6 files changed, 116 insertions(+) create mode 100644 cputopology/Android.mk create mode 100644 cputopology/Makefile create mode 100755 cputopology/cputopology_01.sh create mode 100644 cputopology/cputopology_01.txt create mode 100755 cputopology/cputopology_02.sh create mode 100644 cputopology/cputopology_02.txt diff --git a/cputopology/Android.mk b/cputopology/Android.mk new file mode 100644 index 0000000..7f2c1ef --- /dev/null +++ b/cputopology/Android.mk @@ -0,0 +1,18 @@ +include $(call all-subdir-makefiles) +LOCAL_PATH:= $(call my-dir) + +module_name = cputopology + +define $(module_name)_add_executable + include $(CLEAR_VARS) + LOCAL_MODULE_TAGS := optional + LOCAL_MODULE_CLASS := tests + LOCAL_MODULE := $1.sh + systemtarball: $1.sh + LOCAL_SRC_FILES := $1.sh + LOCAL_MODULE_PATH := $(TARGET_OUT_OPTIONAL_EXECUTABLES)/pm-qa/$(module_name) + include $(BUILD_PREBUILT) +endef + +test_num := 01 02 +$(foreach item,$(test_num),$(eval $(call $(module_name)_add_executable, $(module_name)_$(item)))) diff --git a/cputopology/Makefile b/cputopology/Makefile new file mode 100644 index 0000000..a90620c --- /dev/null +++ b/cputopology/Makefile @@ -0,0 +1,25 @@ +# +# PM-QA validation test suite for the power management on Linux +# +# Copyright (C) 2011, Linaro Limited. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License +# as published by the Free Software Foundation; either version 2 +# of the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. +# +# Contributors: +# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) +# - initial API and implementation +# + +include ../Test.mk \ No newline at end of file diff --git a/cputopology/cputopology_01.sh b/cputopology/cputopology_01.sh new file mode 100755 index 0000000..c7cbc26 --- /dev/null +++ b/cputopology/cputopology_01.sh @@ -0,0 +1,38 @@ +#!/bin/bash +# +# PM-QA validation test suite for the power management on Linux +# +# Copyright (C) 2011, Linaro Limited. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License +# as published by the Free Software Foundation; either version 2 +# of the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. +# +# Contributors: +# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) +# - initial API and implementation +# + +# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… + +source ../include/functions.sh + +check_physical_package_id() { + + local package_id=$CPU_PATH/$1/topology/physical_package_id + local val=$(cat $package_id) + + check "topology is enabled" "test \"$val\" != \"-1\"" +} + +for_each_cpu check_physical_package_id || exit 1 diff --git a/cputopology/cputopology_01.txt b/cputopology/cputopology_01.txt new file mode 100644 index 0000000..5493610 --- /dev/null +++ b/cputopology/cputopology_01.txt @@ -0,0 +1 @@ +test the topology is implemented in the kernel diff --git a/cputopology/cputopology_02.sh b/cputopology/cputopology_02.sh new file mode 100755 index 0000000..6520ce5 --- /dev/null +++ b/cputopology/cputopology_02.sh @@ -0,0 +1,33 @@ +#!/bin/bash +# +# PM-QA validation test suite for the power management on Linux +# +# Copyright (C) 2011, Linaro Limited. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License +# as published by the Free Software Foundation; either version 2 +# of the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. +# +# Contributors: +# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) +# - initial API and implementation +# + +# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… + +source ../include/functions.sh + +FILES="core_id core_siblings core_siblings_list physical_package_id \ +thread_siblings thread_siblings_list" + +for_each_cpu check_topology_files $FILES diff --git a/cputopology/cputopology_02.txt b/cputopology/cputopology_02.txt new file mode 100644 index 0000000..b9e8d6f --- /dev/null +++ b/cputopology/cputopology_02.txt @@ -0,0 +1 @@ +test the topology files are present -- 1.7.9.5

12 years, 7 months

[PATCH pmqa] pm-qa: remove sched_mc test

by Sanjay Singh Rawat

Remove the test as sched_mc is not longer supported Signed-off-by: Sanjay Singh Rawat <sanjay.rawat(a)linaro.org> --- Makefile | 2 -- README | 1 - sched_mc/Android.mk | 18 ------------ sched_mc/Makefile | 25 ---------------- sched_mc/sched_01.sh | 32 --------------------- sched_mc/sched_01.txt | 1 - sched_mc/sched_02.sh | 38 ------------------------ sched_mc/sched_02.txt | 1 - sched_mc/sched_03.sh | 33 --------------------- sched_mc/sched_03.txt | 1 - sched_mc/sched_04.sh | 70 --------------------------------------------- sched_mc/sched_04.txt | 1 - sched_mc/sched_sanity.sh | 35 ----------------------- sched_mc/sched_sanity.txt | 1 - 14 files changed, 259 deletions(-) delete mode 100644 sched_mc/Android.mk delete mode 100644 sched_mc/Makefile delete mode 100755 sched_mc/sched_01.sh delete mode 100644 sched_mc/sched_01.txt delete mode 100755 sched_mc/sched_02.sh delete mode 100644 sched_mc/sched_02.txt delete mode 100755 sched_mc/sched_03.sh delete mode 100644 sched_mc/sched_03.txt delete mode 100755 sched_mc/sched_04.sh delete mode 100644 sched_mc/sched_04.txt delete mode 100755 sched_mc/sched_sanity.sh delete mode 100644 sched_mc/sched_sanity.txt diff --git a/Makefile b/Makefile index fa979e0..52aa60d 100644 --- a/Makefile +++ b/Makefile @@ -30,7 +30,6 @@ check: @(cd cpufreq; $(MAKE) check) @(cd cpuhotplug; $(MAKE) check) @(cd cpuidle; $(MAKE) check) - @(cd sched_mc; $(MAKE) check) # @(cd suspend; $(MAKE) check) @(cd thermal; $(MAKE) check) # @(cd powertop; $(MAKE) check) @@ -39,7 +38,6 @@ uncheck: @(cd cpufreq; $(MAKE) uncheck) @(cd cpuhotplug; $(MAKE) uncheck) @(cd cpuidle; $(MAKE) uncheck) - @(cd sched_mc; $(MAKE) uncheck) # @(cd suspend; $(MAKE) uncheck) @(cd thermal; $(MAKE) uncheck) diff --git a/README b/README index a22a3c8..d689e11 100644 --- a/README +++ b/README @@ -8,5 +8,4 @@ If you want the test suite to continue when one error has been found, do: If you want to run a subset of the tests, do: - make -C sched_mc check make -C cpufreq check diff --git a/sched_mc/Android.mk b/sched_mc/Android.mk deleted file mode 100644 index f9f9d90..0000000 --- a/sched_mc/Android.mk +++ /dev/null @@ -1,18 +0,0 @@ -include $(call all-subdir-makefiles) -LOCAL_PATH:= $(call my-dir) - -module_name = sched - -define $(module_name)_add_executable - include $(CLEAR_VARS) - LOCAL_MODULE_TAGS := optional - LOCAL_MODULE_CLASS := tests - LOCAL_MODULE := $1.sh - systemtarball: $1.sh - LOCAL_SRC_FILES := $1.sh - LOCAL_MODULE_PATH := $(TARGET_OUT_OPTIONAL_EXECUTABLES)/pm-qa/$(module_name)_mc - include $(BUILD_PREBUILT) -endef - -test_num := 01 02 03 04 -$(foreach item,$(test_num),$(eval $(call $(module_name)_add_executable, $(module_name)_$(item)))) diff --git a/sched_mc/Makefile b/sched_mc/Makefile deleted file mode 100644 index a90620c..0000000 --- a/sched_mc/Makefile +++ /dev/null @@ -1,25 +0,0 @@ -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -include ../Test.mk \ No newline at end of file diff --git a/sched_mc/sched_01.sh b/sched_mc/sched_01.sh deleted file mode 100755 index d4414b6..0000000 --- a/sched_mc/sched_01.sh +++ /dev/null @@ -1,32 +0,0 @@ -#!/bin/bash -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… - -source ../include/functions.sh - -FILES="sched_mc_power_savings" - -check_sched_mc_files $FILES diff --git a/sched_mc/sched_01.txt b/sched_mc/sched_01.txt deleted file mode 100644 index 6783a3a..0000000 --- a/sched_mc/sched_01.txt +++ /dev/null @@ -1 +0,0 @@ -test the presence of the 'sched_mc_power_savings' file diff --git a/sched_mc/sched_02.sh b/sched_mc/sched_02.sh deleted file mode 100755 index c7cbc26..0000000 --- a/sched_mc/sched_02.sh +++ /dev/null @@ -1,38 +0,0 @@ -#!/bin/bash -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… - -source ../include/functions.sh - -check_physical_package_id() { - - local package_id=$CPU_PATH/$1/topology/physical_package_id - local val=$(cat $package_id) - - check "topology is enabled" "test \"$val\" != \"-1\"" -} - -for_each_cpu check_physical_package_id || exit 1 diff --git a/sched_mc/sched_02.txt b/sched_mc/sched_02.txt deleted file mode 100644 index 5493610..0000000 --- a/sched_mc/sched_02.txt +++ /dev/null @@ -1 +0,0 @@ -test the topology is implemented in the kernel diff --git a/sched_mc/sched_03.sh b/sched_mc/sched_03.sh deleted file mode 100755 index 6520ce5..0000000 --- a/sched_mc/sched_03.sh +++ /dev/null @@ -1,33 +0,0 @@ -#!/bin/bash -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… - -source ../include/functions.sh - -FILES="core_id core_siblings core_siblings_list physical_package_id \ -thread_siblings thread_siblings_list" - -for_each_cpu check_topology_files $FILES diff --git a/sched_mc/sched_03.txt b/sched_mc/sched_03.txt deleted file mode 100644 index b9e8d6f..0000000 --- a/sched_mc/sched_03.txt +++ /dev/null @@ -1 +0,0 @@ -test the topology files are present diff --git a/sched_mc/sched_04.sh b/sched_mc/sched_04.sh deleted file mode 100755 index f9266e8..0000000 --- a/sched_mc/sched_04.sh +++ /dev/null @@ -1,70 +0,0 @@ -#!/bin/bash -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -# URL : https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/QA/Scripts#sched_… - -source ../include/functions.sh - -check_change() { - local val=$1 - local path=$2 - - echo $val > $path -} - -check_invalid_change() { - - local val=$1 - local path=$2 - - echo $val > $path - if [ "$?" != "0" ]; then - return 0 - fi - - return 1 -} - -check_sched_mc_change() { - - local path=$CPU_PATH/sched_mc_power_savings - local oldval=$(cat $path) - - check "setting value to 0" check_change 0 $path - check "setting value to 1" check_change 1 $path - check "setting value to 2" check_change 2 $path - check "setting invalid value to 3" check_invalid_change 3 $path - check "setting invalid value to -1" check_invalid_change -1 $path - - echo $oldval > $path -} - -if [ $(id -u) != 0 ]; then - log_skip "run as non-root" - exit 0 -fi - -# check_sched_mc_files sched_mc_power_savings || exit 1 -check_sched_mc_change diff --git a/sched_mc/sched_04.txt b/sched_mc/sched_04.txt deleted file mode 100644 index 7f5ed4a..0000000 --- a/sched_mc/sched_04.txt +++ /dev/null @@ -1 +0,0 @@ -test the change are effective for the 'sched_mc_power_savings' file diff --git a/sched_mc/sched_sanity.sh b/sched_mc/sched_sanity.sh deleted file mode 100755 index 1b1b58b..0000000 --- a/sched_mc/sched_sanity.sh +++ /dev/null @@ -1,35 +0,0 @@ -#!/bin/bash -# -# PM-QA validation test suite for the power management on Linux -# -# Copyright (C) 2011, Linaro Limited. -# -# This program is free software; you can redistribute it and/or -# modify it under the terms of the GNU General Public License -# as published by the Free Software Foundation; either version 2 -# of the License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with this program; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. -# -# Contributors: -# Daniel Lezcano <daniel.lezcano(a)linaro.org> (IBM Corporation) -# - initial API and implementation -# - -source ../include/functions.sh - -check_sched_mc_sysfs_entry() { - - local filepath=$CPU_PATH/sched_mc_power_savings - - test -f $filepath && return 1 || return 0 -} - -check_sched_mc_sysfs_entry diff --git a/sched_mc/sched_sanity.txt b/sched_mc/sched_sanity.txt deleted file mode 100644 index c79c23e..0000000 --- a/sched_mc/sched_sanity.txt +++ /dev/null @@ -1 +0,0 @@ -sched_mc is not enabled. Skipping all sched_mc tests... -- 1.7.9.5

12 years, 7 months

boot Ubuntu using PXE from U-Boot on the Samsung XE303C12-A01US Chromebook

by Mario

Hello. I'm trying to boot Ubuntu using PXE from U-Boot on the Samsung XE303C12-A01US Chromebook. I would like to understand better the point 5 of the tutorial here : https://wiki.linaro.org/Boards/Arndale/Setup/PXEBoot it says : "At the u-boot prompt, set the environmental variables "serverip" and "ipaddr" as per the board and host PC configuration. Now set the following environmental variables as mentioned..." the problem is that the Samsung XE303C12-A01US does not have a serial port and I can't stop the u-boot booting and set the enviromental variables manually. I would like to know how can I embed them inside u-boot. Also because there is not a configuration file like boot.cmd/boot.scr like there is on the pandaboard that can be used to pass the parameters. Thanks. -- Mario.

12 years, 7 months

big.LITTLE MP Scheduler test summary-week-12

by Naresh Kamboju

Hi, Release test report week-12: Here is test result summary for big.LITTLE MP Scheduler test on TC2 platform with Android image sched_tests.git No of Test Cases Tests Run Tests Pass Tests Fail Absolute pass rate (%) Failure Analysis/Comments Regression 20 19 19 0 100 - mpbasicsuite 10 9 9 1 90 - mpcoresuite 4 4 3 1 75 - mpextendedsuite 6 6 4 2 66.7 - mploadbalance 2 2 2 0 100 - Android Build: https://android-build.linaro.org/builds/~linaro-android/vexpress-linaro-mp/… Linux kernel: Linux version 3.9.0-rc3-00212-g0f66281 (jenkins-build@ip-10-62-11 1-35) (gcc version 4.7.3 20130226 (prerelease) (Linaro GCC 4.7-2013.03) ) #1 SMP Fri Mar 22 12:35:36 UTC 2013 Results spread sheet: https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0Ai9ggMs8bsGJdER3a… Summary: Linaro Linux kernel updated to 3.9.0-rc3 one extra failure reported in mpcoresuite, when compare with last week results. But this test results are uncertain. Best regards Naresh Kamboju

12 years, 7 months

Test Result Summary of Calendar Week 12, 2013 for Linux Linaro ubuntu Quantal.

by Botao Sun

Calendar Week 12, 2013: Here is test result summary for Linux Linaro ubuntu Quantal image on following boards: 1) ARM Versatile Express A9; 2) Samsung Arndale; 3) TI Panda 4430; 4) TI Panda 4460; 5) ST Ericsson Snowball. Synopsis: All boards keep exactly same status as Linaro 13.02 release image except Samsung Arndale board: Kernel panic has gone in power management test. 1. ARM Versatile Express A9 + Linux Linaro Quantal (Column N): https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AroPySpr4FnEdFNmV… It keeps exactly same status from calendar week 50 last year: only "Halt" & "Device Tree" test failed, all other features work well. 2. Samsung Arndale + Linux Linaro Quantal (Column H): https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AgB-fT5LL31CdGZJS… Kernel panic has gone in power management test, but still several failed test cases observed. All other features work well except Mini HDMI display output. 3. TI Panda 4430 + Linux Linaro Quantal (Column N): https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AroPySpr4FnEdEwwZ… WiFi and display output are unavailable, all other features work well. 4. TI Panda 4460 + Linux Linaro Quantal (Column N): https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AroPySpr4FnEdEwwZ… Boot failed, all test is blocked. 5. ST Ericsson Snowball + Linux Linaro Quantal (Column N): https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AroPySpr4FnEdFJ4X… SD MMC, Mini HDMI display, Reboot and Power Management test failed, all other features work well. For Linaro 13.02 release test summary, please refer to attachment. Thank you. Best Regards Botao Sun

12 years, 7 months

[RFC 1/1] clk: Add notifier support in clk_prepare_enable/clk_disable_unprepare

by Bill Huang

Add the below four notifier events so drivers which are interested in knowing the clock status can act accordingly. This is extremely useful in some of the DVFS (Dynamic Voltage Frequency Scaling) design. PRE_CLK_ENABLE POST_CLK_ENABLE PRE_CLK_DISABLE POST_CLK_DISABLE Signed-off-by: Bill Huang <bilhuang(a)nvidia.com> --- drivers/clk/clk.c | 58 +++++++++++++++++++++++++++++++++++++++++++++++++++ include/linux/clk.h | 40 ++++++++++++++++++++--------------- 2 files changed, 81 insertions(+), 17 deletions(-) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index ed87b24..720a16a 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -1104,6 +1104,64 @@ struct clk *clk_get_parent(struct clk *clk) } EXPORT_SYMBOL_GPL(clk_get_parent); +/** + * clk_prepare_enable - call clk_enable in non-atomic context. + * @clk: the clk to enable + * + * Send PRE_CLK_ENABLE/POST_CLK_ENABLE before/after calling + * clk_enalbe respectively + * + * Return success (0) or errno. + */ +int clk_prepare_enable(struct clk *clk) +{ + int ret; + + mutex_lock(&prepare_lock); + ret = __clk_prepare(clk); + if (ret) + return ret; + + if (clk->notifier_count) + __clk_notify(clk, PRE_CLK_ENABLE, clk->rate, clk->rate); + + ret = clk_enable(clk); + if (ret) + __clk_unprepare(clk); + + if (clk->notifier_count) + __clk_notify(clk, POST_CLK_ENABLE, clk->rate, clk->rate); + + mutex_unlock(&prepare_lock); + + return ret; +} +EXPORT_SYMBOL_GPL(clk_prepare_enable); + +/** + * clk_disable_unprepare - call clk_disable in non-atomic context. + * @clk: the clk to enable + * + * Send PRE_CLK_DISABLE/POST_CLK_DISABLE before/after calling + * clk_disalbe respectively + */ +void clk_disable_unprepare(struct clk *clk) +{ + mutex_lock(&prepare_lock); + + if (clk->notifier_count) + __clk_notify(clk, PRE_CLK_DISABLE, clk->rate, clk->rate); + + clk_disable(clk); + __clk_unprepare(clk); + + if (clk->notifier_count) + __clk_notify(clk, POST_CLK_DISABLE, clk->rate, clk->rate); + + mutex_unlock(&prepare_lock); +} +EXPORT_SYMBOL_GPL(clk_disable_unprepare); + /* * .get_parent is mandatory for clocks with multiple possible parents. It is * optional for single-parent clocks. Always call .get_parent if it is diff --git a/include/linux/clk.h b/include/linux/clk.h index b3ac22d..4bbe1903 100644 --- a/include/linux/clk.h +++ b/include/linux/clk.h @@ -43,6 +43,10 @@ struct clk; #define PRE_RATE_CHANGE BIT(0) #define POST_RATE_CHANGE BIT(1) #define ABORT_RATE_CHANGE BIT(2) +#define PRE_CLK_ENABLE BIT(3) +#define POST_CLK_ENABLE BIT(4) +#define PRE_CLK_DISABLE BIT(5) +#define POST_CLK_DISABLE BIT(6) /** * struct clk_notifier - associate a clk with a notifier @@ -276,6 +280,20 @@ struct clk *clk_get_parent(struct clk *clk); */ struct clk *clk_get_sys(const char *dev_id, const char *con_id); +/** + * clk_prepare_enable - helps cases using clk_enable in non-atomic context. + * @clk: clock source + * + * Return success (0) or nagative errno. + */ +int clk_prepare_enable(struct clk *clk); + +/** + * clk_disable_unprepare - helps cases using clk_disable in non-atomic context. + * @clk: clock source + */ +void clk_disable_unprepare(struct clk *clk); + #else /* !CONFIG_HAVE_CLK */ static inline struct clk *clk_get(struct device *dev, const char *id) @@ -324,30 +342,18 @@ static inline struct clk *clk_get_parent(struct clk *clk) return NULL; } -#endif - -/* clk_prepare_enable helps cases using clk_enable in non-atomic context. */ static inline int clk_prepare_enable(struct clk *clk) { - int ret; - - ret = clk_prepare(clk); - if (ret) - return ret; - ret = clk_enable(clk); - if (ret) - clk_unprepare(clk); - - return ret; + return 0; } -/* clk_disable_unprepare helps cases using clk_disable in non-atomic context. */ -static inline void clk_disable_unprepare(struct clk *clk) +static inline int clk_disable_unprepare(struct clk *clk) { - clk_disable(clk); - clk_unprepare(clk); + return 0; } +#endif + /** * clk_add_alias - add a new clock alias * @alias: name for clock alias -- 1.7.9.5

12 years, 7 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

linaro-dev