All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manoj Iyer <manoj.iyer@canonical.com>
To: Chris Ball <cjb@laptop.org>
Cc: Arnd Bergmann <arnd@arndb.de>,
	Manoj Iyer <manoj.iyer@canonical.com>,
	linux-kernel@vger.kernel.org, jbarnes@virtuousgeek.org,
	matsumur@nts.ricoh.co.jp, linux-pci@vger.kernel.org,
	linux-mmc@vger.kernel.org
Subject: Re: [PATCH] mmc: Added quirks for Ricoh 1180:e823 lower base clock frequency
Date: Wed, 13 Jul 2011 10:42:32 -0500 (CDT)	[thread overview]
Message-ID: <alpine.DEB.2.02.1107131031360.3174@lazy> (raw)
In-Reply-To: <m2sjqa2q93.fsf@bob.laptop.org>


Chris/Arnd,

Here is a series of test I did with the patched kernel.


== cold boot insert SD card ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    4.96M/s
2MiB    6.3M/s
1MiB    6.23M/s
512KiB  6.23M/s
256KiB  6.26M/s

== rerun the same test 2nd time ===
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    6.28M/s
2MiB    6.29M/s
1MiB    6.29M/s
512KiB  6.29M/s
256KiB  6.26M/s

== remove and reinsert the SD card ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    6.42M/s
2MiB    6.28M/s
1MiB    6.22M/s
512KiB  6.23M/s
256KiB  6.26M/s

== remove and reinsert the SD card ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    6.42M/s
2MiB    6.3M/s
1MiB    6.22M/s
512KiB  6.25M/s
256KiB  6.24M/s

== reboot, reinsert the SD card ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
[sudo] password for u:
4MiB    6.45M/s
2MiB    6.33M/s
1MiB    6.25M/s
512KiB  6.29M/s
256KiB  6.3M/s
u@u:~/flash/flashbench$

== rerun the test again ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    6.27M/s
2MiB    6.27M/s
1MiB    6.28M/s
512KiB  6.27M/s
256KiB  6.24M/s
u@u:~/flash/flashbench$

== poweroff, Poweron, insert SD card ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
[sudo] password for u:
4MiB    6.45M/s
2MiB    6.33M/s
1MiB    6.24M/s
512KiB  6.29M/s
256KiB  6.32M/s
u@u:~/flash/flashbench$

== rerun the test again ==
u@u:~/flash/flashbench$ sudo ./flashbench -O --erasesize=$[4 * 1024 * 
1024] --blocksize=$[256 * 1024] /dev/mmcblk0  --open-au-nr=2
4MiB    6.29M/s
2MiB    6.28M/s
1MiB    6.29M/s
512KiB  6.28M/s
256KiB  6.25M/s
u@u:~/flash/flashbench$

On Wed, 13 Jul 2011, Chris Ball wrote:

> Hi Arnd,
>
> On Tue, Jul 12 2011, Arnd Bergmann wrote:
>> I would very much expect that to be nonreproducible. The first row
>> in each test is the result of a single write() system call and does
>> not get averaged out. More importantly the time for each write
>> depends a lot of the state of the card before the write.
>>
>> For instance when you do a lot of random writes to a card, optionally
>> take it out and put it into a different machine, and then do a large
>> linear write, that linear write will be very slow because the
>> card has to garbage collect all the random writes that were done
>> earlier. After a few writes (usually one is enough), it gets back
>> to the full performance.
>
> That makes sense.  Do you think this explains Manoj getting a slower
> first file copy speed (757ms vs. 480ms) after applying his patch?
> (Manoj, perhaps you could retry your test without GC being needed?)
>
> What would we expect lowering the SD base clock frequency from 200MHz
> to 50MHz to do to performance theoretically?
>
> Thanks,
>
> - Chris.
> -- 
> Chris Ball   <cjb@laptop.org>   <http://printf.net/>
> One Laptop Per Child
>
>

--
====================
Manoj Iyer
Ubuntu/Canonical
Hardware Enablement
====================

  reply	other threads:[~2011-07-13 15:42 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-11 21:28 [Patch 0/1] mmc: Added quirks for Ricoh 1180:e823 lower base clock frequency Manoj Iyer
2011-07-11 21:28 ` [PATCH] " Manoj Iyer
2011-07-11 21:53   ` Chris Ball
2011-07-11 22:55     ` Manoj Iyer
2011-07-12 16:01     ` Manoj Iyer
2011-07-12 16:20       ` Chris Ball
2011-07-12 17:09         ` Manoj Iyer
2011-07-12 17:30           ` Chris Ball
2011-07-12 18:00             ` Arnd Bergmann
2011-07-13 14:45               ` Chris Ball
2011-07-13 15:42                 ` Manoj Iyer [this message]
2011-07-13 16:35                   ` Arnd Bergmann
2011-07-13 16:46                     ` Manoj Iyer
2011-07-13 16:55                       ` Chris Ball
2011-07-13 17:02                         ` Manoj Iyer
2011-07-13 17:08                           ` Chris Ball
2011-07-18 20:36                             ` Manoj Iyer
2011-07-18 20:40                               ` Chris Ball
2011-07-18 20:47                                 ` Manoj Iyer
2011-07-18 21:05                                   ` Chris Ball
2011-07-18 21:12                                     ` Manoj Iyer
2011-07-19 19:43                                     ` Manoj Iyer
2012-11-22  0:29                                   ` Chris Ball
2012-11-26 16:42                                     ` Manoj Iyer
2012-11-26 16:57                                       ` Chris Ball
2011-07-13 16:50                     ` Chris Ball
2011-07-13 16:54                       ` Manoj Iyer
2011-07-18 23:27                         ` Jesse Barnes

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=alpine.DEB.2.02.1107131031360.3174@lazy \
    --to=manoj.iyer@canonical.com \
    --cc=arnd@arndb.de \
    --cc=cjb@laptop.org \
    --cc=jbarnes@virtuousgeek.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=matsumur@nts.ricoh.co.jp \
    /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.