From: Khiem Nguyen <khiem.nguyen.xt@renesas.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH 0/5] arm: shmobile: Lager Add USB and SATA support
Date: Thu, 11 Sep 2014 10:20:38 +0000 [thread overview]
Message-ID: <54117776.8040807@renesas.com> (raw)
In-Reply-To: <1387370660-20481-1-git-send-email-valentine.barshak@cogentembedded.com>
Hi Hiep-san,
Thanks for your test result.
On 9/11/2014 6:35 PM, カオ ミン ヒェップ wrote:
> Hi Khiem-san
>
> Sorry for late replying!
>
> On 09/10/2014 06:32 PM, Khiem Nguyen wrote:
>> Hi Hiep-san, Sakato-san,
>>
>> Sorry for interruption.
>>
>> Could you also confirm performance of SATA driver
>> in case that CPUFreq uses performance governor ?
>>
>> If SATA performance is degraded in that case,
>> I guess we still need countermeasure.
>> (because CPUFreq will definitely be enabled in multi-platform defconfig.)
> Here is the results of SATA driver writing speed in case CPUFreq uses governor:
> Data size: 350MB,
> root@linaro-nano:~# uname -a
> Linux linaro-nano 3.17.0-rc2 #3 SMP Thu Sep 11 17:15:17 JST 2014 armv7l armv7l armv7l GNU/Lix
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor
> ondemand
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor
> ondemand
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu2/cpufreq/scaling_governor
> ondemand
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu3/cpufreq/scaling_governor
> ondemand
>
> # mount -t tmpfs -o size@0m tmpfs /tmp
> root@linaro-nano:~# dd if=/dev/urandom of=/tmp/file-350mb bs=1M count50
>
> 350+0 records in
> 350+0 records out
> 367001600 bytes (367 MB) copied, 88.5655 s, 4.1 MB/s
> root@linaro-nano:~# mount /dev/sda1 /mnt/sata/
> kjournald starting. Commit interval 5 seconds
> EXT3-fs (sda1): using internal journal
> EXT3-fs (sda1): mounted filesystem with ordered data mode
> oot@linaro-nano:~# time cp /tmp/file-350mb /mnt/sata/; time umount /mnt/sata/
>
> real 0m6.665s
> user 0m0.020s
> sys 0m5.690s
>
> real 0m1.243s
> user 0m0.010s
> sys 0m0.660s
>
> Total: 7.9s
> Speed: 46.45MB/s
> =========================
>
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor
> performance
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor
> performance
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu2/cpufreq/scaling_governor
> performance
> root@linaro-nano:~# cat /sys/devices/system/cpu/cpu3/cpufreq/scaling_governor
> performance
> # time cp /tmp/file-350mb /mnt/sata/; time umount /mnt/sata/
>
> real 0m5.331s
> user 0m0.010s
> sys 0m5.250s
>
> real 0m1.084s
> user 0m0.030s
> sys 0m0.500s
>
> Total: 6.3s
> Speed: 58.3MB/s
It seems SATA performance (write speed) is degraded after CPUFreq is enabled
(even performance governor is selected).
Base on your test result so far:
T1 Upstream-v3.16-rc5: 66 MB/s
T2 v3.17-rc2 (CPUFreq disabled): ~ 66 MB/s
T3 v3.17-rc2 (CPUFreq enable / on-demand governor): 42.61MB/s
T4 v3.17-rc2 (CPUFreq disabled / performance governor ): 58.3MB/s
The diff between T2 and T4 is ~8 MB/s (or 8% of 66 MB/s).
I guess we should try to find the cause of this regression.
(I wonder similar phenomenon occurs in backport version ...)
(I added Inami-san, CPUFreq developer for his info.)
>
>
> Thank you!
> Cao Minh Hiep.
--
Best regards,
KHIEM Nguyen
next prev parent reply other threads:[~2014-09-11 10:20 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-18 12:44 [PATCH 0/5] arm: shmobile: Lager Add USB and SATA support Valentine Barshak
2013-12-19 10:09 ` Magnus Damm
2013-12-19 13:02 ` Valentine
2013-12-19 17:56 ` Valentine
2013-12-20 14:07 ` Valentine
2014-09-10 1:21 ` Ryusuke Sakato
2014-09-10 5:18 ` カオ ミン ヒェップ
2014-09-10 7:15 ` Ryusuke Sakato
2014-09-10 9:16 ` カオ ミン ヒェップ
2014-09-10 9:32 ` Khiem Nguyen
2014-09-11 9:35 ` カオ ミン ヒェップ
2014-09-11 10:20 ` Khiem Nguyen [this message]
2014-09-11 11:26 ` カオ ミン ヒェップ
2014-09-12 0:39 ` Khiem Nguyen
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=54117776.8040807@renesas.com \
--to=khiem.nguyen.xt@renesas.com \
--cc=linux-sh@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.