public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [PATCH 00/16] Refactor the architecture parts of mt7628
Date: Fri, 10 Jan 2020 14:13:21 +0100	[thread overview]
Message-ID: <2d9110fa-3f36-cd1b-3041-b292b5b1024e@denx.de> (raw)
In-Reply-To: <1578653321.31096.38.camel@mcddlt001>

Hi Weijie,

On 10.01.20 11:48, Weijie Gao wrote:

<snip>

>>>> Did you compare the boot speed with using SPL + main U-Boot vs U-Boot
>>>> only?
>>>
>>> As far as I tested the difference of boot time should be less than 1s.
>>
>> 1 second is quite a lot of time. I prefer to stay with the non compressed
>> boot configuration then. Testing shows that the new SPL based version
>> also boots pretty fast, when the main U-Boot proper is not compressed.
>>    
>>>>
>>>> In the current combined image the non-compressed U-Boot proper is
>>>> included. Why did you not include the LZMA compressed image per default?
>>>> Is the decompression too slow?
>>>
>>> I just kept these two boards untouched. I can switch them to compressed
>>> mode if you have no concern.
>>
>> I did not test yet. If its really a difference in time of 1 second (or
>> similar), then let stay with uncompressed for these board please.
>>
>> Thanks,
>> Stefan
> 
> 
> I did a precise measurement, and this is the result:
> 
> type       | spl load/decompression time | boot to shell (total time)
> spl + lzma | 75ms                        | 563ms
> spl + none | 139ms                       | 627ms
> non-spl    | N/A                         | 626ms

Great. Thanks for doing these tests and reporting here.
  
> Actually spl+lzma is the fastest. I think this is because it has the
> smallest binary size.

Makes sense. SPI NOR load time savage is bigger than the added
decompression time.
  
> It's OK to switch all of them to compressed mode.

Yes, please do so. This might be a nice boot time increase for other
boards as well. I'll do some tests on i.MX6 soon. :)

Thanks,
Stefan

      reply	other threads:[~2020-01-10 13:13 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-08  2:59 [PATCH 00/16] Refactor the architecture parts of mt7628 Weijie Gao
2020-01-09  7:52 ` Stefan Roese
2020-01-09  8:49   ` Weijie Gao
2020-01-09 12:36     ` Stefan Roese
2020-01-10 10:48       ` Weijie Gao
2020-01-10 13:13         ` Stefan Roese [this message]

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=2d9110fa-3f36-cd1b-3041-b292b5b1024e@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /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