| Message ID | 20250414220439.529572-1-trini@konsulko.com |
|---|---|
| State | Deferred |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | Revert "CI: Disable evb-ast2600" | expand |
On Tue, 15 Apr 2025 at 07:34, Tom Rini <trini@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@konsulko.com> Thanks Tom! Sorry we couldn't help with the failure. Acked-by: Joel Stanley <joel@jms.id.au> > --- > Cc: Ryan Chen <ryan_chen@aspeedtech.com> > Cc: Chia-Wei Wang <chiawei_wang@aspeedtech.com> > Cc: Aspeed BMC SW team <BMC-SW@aspeedtech.com> > Cc: Joel Stanley <joel@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 >
On 4/15/25 01:44, Joel Stanley wrote: > On Tue, 15 Apr 2025 at 07:34, Tom Rini <trini@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@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@jms.id.au> > >> --- >> Cc: Ryan Chen <ryan_chen@aspeedtech.com> >> Cc: Chia-Wei Wang <chiawei_wang@aspeedtech.com> >> Cc: Aspeed BMC SW team <BMC-SW@aspeedtech.com> >> Cc: Joel Stanley <joel@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 >>
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}
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@konsulko.com> --- Cc: Ryan Chen <ryan_chen@aspeedtech.com> Cc: Chia-Wei Wang <chiawei_wang@aspeedtech.com> Cc: Aspeed BMC SW team <BMC-SW@aspeedtech.com> Cc: Joel Stanley <joel@jms.id.au> --- .azure-pipelines.yml | 4 ++++ .gitlab-ci.yml | 7 +++++++ 2 files changed, 11 insertions(+)