Skip to content

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented May 11, 2023

3 commits. Main one:

rebuild-testbench.sh: add -j jobs option

This is required to investigate build failures. Because of the way the
build is split, a failure in one component can become totally drown in
the long build logs of another component.

Example: with -j2 the parser_ep build failure below is completely
drown by the long (and successful!) build logs of sof_ep:

./scripts/rebuild-testbench.sh -j 1 -p tgl

sof/tools/testbench/build_xt_testbench/sof_parser/build/include/alsa/
sound/asoc.h:196: error: expected specifier-qualifier-list before ‘__le32’

Signed-off-by: Marc Herbert marc.herbert@intel.com

marc-hb added 3 commits May 11, 2023 19:02
Also: don't fail silently.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Fixes last shellcheck -x warning

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
This is required to investigate build failures. Because of the way the
build is split, a failure in one component can become totally drown in
the long build logs of another component.

Example: with -j2 the `parser_ep` build failure below is completely
drown by the long (and successful!) build logs of `sof_ep`:

```
./scripts/rebuild-testbench.sh -j 1 -p tgl

sof/tools/testbench/build_xt_testbench/sof_parser/build/include/alsa/
sound/asoc.h:196: error: expected specifier-qualifier-list before ‘__le32’
```

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb marked this pull request as ready for review May 11, 2023 19:07
@marc-hb
Copy link
Collaborator Author

marc-hb commented May 11, 2023

imx8m failure in https://github.com/thesofproject/sof/actions/runs/4951554044/jobs/8856711673?pr=7594 is well known and will be fixed by #7586. Everything else is green.

https://sof-ci.01.org/sof-pr-viewer/#/build/PR7594/build11976639 has the usual timeout. It does not use this script at all.

@marc-hb marc-hb requested review from kv2019i and lgirdwood May 12, 2023 17:03
@kv2019i kv2019i merged commit c1ae62e into thesofproject:main May 17, 2023
@marc-hb marc-hb deleted the testbench-j branch May 17, 2023 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants