linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: kesavan.abhilash@gmail.com (Abhilash Kesavan)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 0/5] MCPM backend for Exynos5420
Date: Wed, 14 May 2014 09:45:44 +0530	[thread overview]
Message-ID: <CAM4voanvVwj+FuWazTGMHMnM64Cw_qK03D3o=raS43f_zWFfog@mail.gmail.com> (raw)
In-Reply-To: <alpine.LFD.2.11.1405131325060.980@knanqh.ubzr>

Hi Nicolas,

On Tue, May 13, 2014 at 11:25 PM, Nicolas Pitre
<nicolas.pitre@linaro.org> wrote:
> On Tue, 13 May 2014, Abhilash Kesavan wrote:
>
>> This is v6 of the series adding MCPM backend support for SMP secondary boot
>> and core switching on Samsung's Exynos5420. The patches are based on the mcpm
>> support added for Exynos5420 in the Chromium kernel repository here:
>> https://chromium.googlesource.com/chromiumos/third_party/kernel-next/+/chromeos-3.8
>>
>> The patches have been prepared on Kukjin Kim's for-next branch and tested on
>> SMDK5420 EVT1 as well as an exynos5420 based chromebook (peach-pit) using the
>> "/dev/b.L_switcher" user interface. Secondary core boot-up has also been tested
>> on both the boards.
>
> OK... Now it is time for real testing.  :-)
>
> The /dev/b.L_switcher interface tests the switcher.  Here you really
> want to hammer the MCPM functionalities and especially your backend code
> as hard as possible.  I therefore recommend the following test script:
>
> ---------- >8
> #!/bin/bash
>
> echo 0 >/sys/kernel/bL_switcher/active
> sleep 1
>
> pids=
> for cpu in /sys/devices/system/cpu/cpu?/online; do
>   { cpu_nr=${cpu:27:1}
>     while true; do
>       echo 1 > $cpu 2> /dev/null
>       sleep .00$RANDOM
>       val1=$(cat $cpu)
>       echo 0 > $cpu 2> /dev/null
>       sleep .00$RANDOM
>       val0=$(cat $cpu)
>       [ "$val1" = "1" -a "$val0" = "0" ] && echo -n $cpu_nr
>     done
>   } &
>   pids="$pids $!"
> done
>
> trap "kill $pids; echo" 0 15
> wait $pids
> ---------- >8
>
> Leave this running for a couple hours making sure you see all CPU
> numbers being printed.  The printing order will be random, but each CPU
> number should continuously appear.
I tried this script and I get two errors:

1) can't create /sys/devices/system/cpu/cpu//online: nonexistent directory
2) sleep: invalid number '.0026736'


For 1) the cpu number is not being appended. if I give a particular
cpu in the script then hotplug in/out works fine.
For 2) a constant msleep 10 works.

Is it OK for me to modify the script to hotplug in/off a randomly chosed core ?

Regards,
Abhilash
>
>
> Nicolas

  reply	other threads:[~2014-05-14  4:15 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-13 11:58 [PATCH v6 0/5] MCPM backend for Exynos5420 Abhilash Kesavan
2014-05-13 11:58 ` [PATCH v6 1/5] ARM: EXYNOS: Add generic cpu power control functions for all exynos based SoCs Abhilash Kesavan
2014-05-13 11:58 ` [PATCH v6 2/5] ARM: EXYNOS: use generic exynos cpu power control functions Abhilash Kesavan
2014-05-13 11:58 ` [PATCH v6 3/5] arm: exynos: Add generic cluster " Abhilash Kesavan
2014-05-13 11:58 ` [PATCH v6 4/5] ARM: dts: exynos5420: add CCI node Abhilash Kesavan
2014-05-13 11:58 ` [PATCH v6 5/5] arm: exynos: Add MCPM call-back functions Abhilash Kesavan
2014-05-13 16:48   ` Lorenzo Pieralisi
2014-05-14  2:44     ` Abhilash Kesavan
2014-05-14  2:54       ` Chander Kashyap
2014-05-14  3:02         ` Abhilash Kesavan
2014-05-14  3:02         ` Nicolas Pitre
2014-05-14  3:20           ` Chander Kashyap
2014-05-14  3:34             ` Nicolas Pitre
2014-05-14  7:24   ` [PATCH v7 " Abhilash Kesavan
2014-05-13 17:55 ` [PATCH v6 0/5] MCPM backend for Exynos5420 Nicolas Pitre
2014-05-14  4:15   ` Abhilash Kesavan [this message]
2014-05-14  4:45     ` Nicolas Pitre
2014-05-14  7:34       ` Abhilash Kesavan
2014-05-14 13:33         ` Nicolas Pitre
2014-05-14 13:39           ` Abhilash Kesavan
2014-05-15 15:49             ` Abhilash Kesavan
2014-05-15 16:52               ` Nicolas Pitre
2014-05-16 23:55                 ` Kukjin Kim
2014-05-19  2:38                   ` Abhilash Kesavan
2014-05-19  2:38                 ` Abhilash Kesavan
2014-05-20  3:30                   ` Nicolas Pitre
2014-05-20  4:42                     ` Abhilash Kesavan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAM4voanvVwj+FuWazTGMHMnM64Cw_qK03D3o=raS43f_zWFfog@mail.gmail.com' \
    --to=kesavan.abhilash@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).