[U-Boot] [PATCH v5 07/20] env: Add support for callbacks to environment vars

[Stefan Roese]

Hi Joe,

On 01/15/2013 12:23 AM, Joe Hershberger wrote:

<snip>

>> Here you see the delay in himport_r() is approx 1 second.
>>
>> And with patch (2598090 [env: Add environment variable flags]) this
>> delay even grows to 2 seconds:
>>
>> [  3.096149] <  0.004800> U-Boot 2013.01-rc1-00392-g2598090 (Jan 11 2013 - 08:55:45)
>> [  3.096853] <  0.000704>
>> [  3.103805] <  0.006952> CPU:   AMCC PowerPC 440EPx Rev. A at 528 MHz (PLB=132 OPB=66 EBC=66 PCI=33 MHz)
>> [  3.106575] <  0.002770>        Security/Kasumi support
>> [  3.112135] <  0.005560>        Bootstrap Option H - Boot ROM Location I2C (Addr 0x52)
>> [  3.117683] <  0.005548>        Internal PCI arbiter enabled, PCI async ext clock used
>> [  3.120548] <  0.002865>        32 kB I-Cache 32 kB D-Cache
>> [  3.128716] <  0.008168> Board: Sequoia - AMCC PPC440EPx Evaluation Board, Rev. F, PCI-Async=33 MHz
>> [  3.130099] <  0.001383> I2C:   ready
>> [  3.536470] <  0.406371> DRAM:  256 MiB
>> [  3.699894] <  0.163424> Flash: 64 MiB
>> [  3.701553] <  0.001659> NAND:  32 MiB
>>
>> himport_r()
>>
>> [  5.690365] <  1.988812> PCI:   Bus Dev VenId DevId Class Int
>> [  5.698395] <  0.008030> USB:   Host(int phy) Device(ext phy)
>> [  5.701709] <  0.003314> Net:   ppc_4xx_eth0, ppc_4xx_eth1
>>
>>
>> Joe, I didn't dig deeper into these functions and patches. Do you
>> have an idea why these patches introduce this delay? Can you
>> reproduce this on one of your boards? Has anybody else noticed
>> such issues?
> 
> That is far more than I would expect.  Let me have a look at it.
> Thanks for bringing this up.

Thanks.

> Do you have an extremely large env?

Not really. Here a log of this board:

=> printenv
abc=def
...
yyyy=sdfsdfsfd

Environment size: 4141/8187 bytes

Just let me know if I need to do some more tests to help.

Thanks,
Stefan