[U-Boot] [PATCH 1/1] x86: qemu-x86 requires CONFIG_SYS_HZ=2000

[Heinrich Schuchardt]

On 09/06/2018 08:55 AM, Bin Meng wrote:
> On Thu, Sep 6, 2018 at 2:49 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>>
>> Hi Heinrich,
>>
>> On Thu, Sep 6, 2018 at 2:06 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>>
>>> The sleep command only provides correct timings for
>>> CONFIG_SYS_HZ=2000 on qemu-x86_defconfig.
>>>
>>> This can be tested with
>>>
>>>     date && sleep 4 && date
>>
>> With current u-boot/master, with the same test command you provided, I got:
>>
>> => date && sleep 4 && date
>> Date: 2018-09-06 (Thursday)    Time:  6:47:28
>> Date: 2018-09-06 (Thursday)    Time:  6:47:32
>> => date && sleep 4 && date
>> Date: 2018-09-06 (Thursday)    Time:  6:47:41
>> Date: 2018-09-06 (Thursday)    Time:  6:47:45
>> => date && sleep 4 && date
>> Date: 2018-09-06 (Thursday)    Time:  6:47:47
>> Date: 2018-09-06 (Thursday)    Time:  6:47:51
>>
>> It seems there is no issue.
>>
>> QEMU was invoked like this: qemu-system-i386 -nographic -bios u-boot.rom
> 
> with '-smp 4', the issue still cannot be reproduced.
> 
> I am using QEMU 2.5.0, shipped by Ubuntu 16.04.
> 
> Regards,
> Bin
> 
Hello Bin,

thanks for testing.

I reran my tests with the same Debian Buster release and QEMU 2.12.0 on
two machines, one with Intel CPU one with AMD Ryzen and I get different
results.

So my suggestion would be that I reduce the patch to only change
lib/Kconfig to make CONFIG_SYS_HZ editable and keep the 1000 Hz default
value. That way corrections become possible in .config. And we keep the
defconfig as it is.

Best regards

Heinrich