[PATCH] Revert "CI: Disable evb-ast2600"

[Cédric Le Goater]

On 4/15/25 01:44, Joel Stanley wrote:
> On Tue, 15 Apr 2025 at 07:34, Tom Rini <trini at konsulko.com> wrote:
>>
>> Upstream developers have been unable to replicate this failure exactly.
>> Furthermore, with commit 8c98b57d72d5 ('Merge patch series "Static
>> initcalls"') we have now replaced the part of the U-Boot code which the
>> platform was hanging in when I was debugging. Moving forward, if we have
>> a failure again the next steps are to upgrade QEMU (which should be done
>> regardless).
>>
>> This reverts commit 8a2cf6307a2ccc09c39dde486b6d9375b78c82c2.
>>
>> Signed-off-by: Tom Rini <trini at konsulko.com>
> 
> Thanks Tom! Sorry we couldn't help with the failure.


One thing to know about the Aspeed SoC AST2600 is that the Cortex
A7 CPU has an FPU implementing VFPv4 without NEON support, which
can break user space. buildroot was fixed with BR2_ARM_FPU_VFPV4D16
and the QEMU model was adjusted to be more precise with HW,
in version 8.1 and above.

It might be related. Having some logs would help to corner the issue.

Thanks,

C.


> 
> Acked-by: Joel Stanley <joel at jms.id.au>
> 
>> ---
>> Cc: Ryan Chen <ryan_chen at aspeedtech.com>
>> Cc: Chia-Wei Wang <chiawei_wang at aspeedtech.com>
>> Cc: Aspeed BMC SW team <BMC-SW at aspeedtech.com>
>> Cc: Joel Stanley <joel at jms.id.au>
>> ---
>>   .azure-pipelines.yml | 4 ++++
>>   .gitlab-ci.yml       | 7 +++++++
>>   2 files changed, 11 insertions(+)
>>
>> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
>> index cb787d32f142..24b6661c9c94 100644
>> --- a/.azure-pipelines.yml
>> +++ b/.azure-pipelines.yml
>> @@ -424,6 +424,10 @@ stages:
>>             TEST_PY_BD: "evb-ast2500"
>>             TEST_PY_ID: "--id qemu"
>>             TEST_PY_TEST_SPEC: "not sleep"
>> +        evb_ast2600:
>> +          TEST_PY_BD: "evb-ast2600"
>> +          TEST_PY_ID: "--id qemu"
>> +          TEST_PY_TEST_SPEC: "not sleep"
>>           vexpress_ca9x4:
>>             TEST_PY_BD: "vexpress_ca9x4"
>>             TEST_PY_ID: "--id qemu"
>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
>> index 5f3418e482fe..aa15e009b39e 100644
>> --- a/.gitlab-ci.yml
>> +++ b/.gitlab-ci.yml
>> @@ -331,6 +331,13 @@ evb-ast2500 test.py:
>>       TEST_PY_ID: "--id qemu"
>>     <<: *buildman_and_testpy_dfn
>>
>> +evb-ast2600 test.py:
>> +  variables:
>> +    TEST_PY_BD: "evb-ast2600"
>> +    TEST_PY_TEST_SPEC: "not sleep"
>> +    TEST_PY_ID: "--id qemu"
>> +  <<: *buildman_and_testpy_dfn
>> +
>>   sandbox_flattree test.py:
>>     tags:
>>       - ${DEFAULT_AMD64_TAG}
>> --
>> 2.43.0
>>