From: Shawn Lin <shawn.lin@rock-chips.com>
To: Jaehoon Chung <jh80.chung@samsung.com>,
Ulf Hansson <ulf.hansson@linaro.org>,
Alan Cooper <alcooperx@gmail.com>
Cc: shawn.lin@rock-chips.com, linux-mmc <linux-mmc@vger.kernel.org>
Subject: Re: [PATCH] mmc: core: Optimize boot time by detecting cards simultaneously
Date: Fri, 18 Dec 2015 11:05:23 +0800 [thread overview]
Message-ID: <567377F3.2060906@rock-chips.com> (raw)
In-Reply-To: <5673612C.9050808@samsung.com>
On 2015/12/18 9:28, Jaehoon Chung wrote:
> Hi,
>
> On 12/18/2015 06:49 AM, Ulf Hansson wrote:
>> On 17 December 2015 at 20:47, Alan Cooper <alcooperx@gmail.com> wrote:
>>> On Wed, Dec 16, 2015 at 4:07 AM, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>>>> To find the improved card detection times, I just used the kernel boot
>>>> log and checked the time stamps for when the cards get detected. I ran
>>>> the tests ~10 times and picked up some average values.
>>>>
>>>> To notice the improvement you need at least two cards to be inserted
>>>> during boot. Perhaps you have a platform with one eMMC and one SD card
>>>> slot, you can test.
>>>
>>> I tested the patch on one of our ARM bases SoC's. The SoC has an SD
>>> card slot on the first controller and an eMMC device on the second
>>> controller. I found ~100ms improvement on the time it took for the
>>> rootfs on the eMMC device to be ready for use. Interestingly, I did
>>> see the SD/eMMC order swap once in 20 tries. This is not an issue for
>>> us as we already have to use UUID for the eMMC rootfs in case an SD
>>> card is installed.
>>>
hi Ulf,
I'm interested in this patch since I also spent a little time
optimzing the boot time and sequence for my local branch based on kernel
3.1x. With you patch, it seems, the tablet only achieves very small
improvement for the SD controller(less than 20ms).
My test environment:
Kernel: 3.14
Slot: 1 emmc(1st) + 1 SD card(2nd) + 1 sdio card(3rd)
I happen to run massive reboot test for my stable kernel branch this
weekend, so I will apply this patch to see if any problem occurs. :)
>>> Al
>>
>> Al, thanks for your effort in testing. I assume I can apply a
>> tested-by tag for that!
>>
>> BTW, if you want to see an even better improvement, just find a
>> crappier SD card as I guess you have a quite good one. :-)
>
> I didn't test with this yet..actually, i have tested with only one board(Exynos4412).
> There is no the changing index. (Not too much for testing)
> but in my experiment, our platform(Tizen) should be produced the problem in SD-card case.
> I need to check with SystemFW guys. I will check this..ASAP.
>
> Best Regards,
> Jaehoon Chung
>
>>
>> Kind regards
>> Uffe
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
>
--
Best Regards
Shawn Lin
next prev parent reply other threads:[~2015-12-18 3:05 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-01 15:15 [PATCH] mmc: core: Optimize boot time by detecting cards simultaneously Ulf Hansson
2015-12-14 15:50 ` Ulf Hansson
2015-12-15 23:22 ` Jaehoon Chung
2015-12-16 9:07 ` Ulf Hansson
2015-12-17 19:47 ` Alan Cooper
2015-12-17 21:49 ` Ulf Hansson
2015-12-18 1:28 ` Jaehoon Chung
2015-12-18 3:05 ` Shawn Lin [this message]
2015-12-28 9:31 ` Ulf Hansson
2015-12-28 9:44 ` Shawn Lin
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=567377F3.2060906@rock-chips.com \
--to=shawn.lin@rock-chips.com \
--cc=alcooperx@gmail.com \
--cc=jh80.chung@samsung.com \
--cc=linux-mmc@vger.kernel.org \
--cc=ulf.hansson@linaro.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