Skip to content
This repository was archived by the owner on Apr 13, 2024. It is now read-only.

Increase aarch64 timeout from 2m to 3m#283

Merged
nickdesaulniers merged 1 commit into
masterfrom
aarch64_timeout
Jun 23, 2020
Merged

Increase aarch64 timeout from 2m to 3m#283
nickdesaulniers merged 1 commit into
masterfrom
aarch64_timeout

Conversation

@nickdesaulniers

Copy link
Copy Markdown
Member

@nickdesaulniers

Copy link
Copy Markdown
Member Author

@nathanchance

Copy link
Copy Markdown
Member

Hmmm my local builds aren’t timing out but it might be worth seeing if there is a significant regression in boot time from 5.7 to latest mainline.

@nickdesaulniers

Copy link
Copy Markdown
Member Author

it may be one of the kselftests regressing in runtime. (maybe more tests were added; maybe there's an unexpected regression)

@nathanchance

Copy link
Copy Markdown
Member

This appears related to ClangBuiltLinux/boot-utils#20. I tested an arm64 defconfig + common.config at v5.7.

$ make -skj"$(nproc)" ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- LLVM=1 O=out/arm64 distclean defconfig

$ scripts/kconfig/merge_config.sh -O out/arm64 -m out/arm64/.config ~/cbl/github/ci/configs/common.config
...

$ make -skj"$(nproc)" ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- LLVM=1 O=out/arm64 olddefconfig Image.gz

-cpu cortex-a57:

[   15.626943] reboot: Power down

-cpu max:

[  134.659678] reboot: Power down

With GCC 10.1.0, I see a similar slow down.

-cpu cortex-a57:

[   15.596947] reboot: Power down

-cpu max:

[  115.585043] reboot: Power down

We should probably increase the timeout to 5m in that case.

@nickdesaulniers

nickdesaulniers commented Jun 19, 2020

Copy link
Copy Markdown
Member Author

Interesting measurements, thanks for taking those. cc @broonie , as a heads up, it seems that maybe emulation of PAC+BTI is relatively expensive in QEMU.

@nathanchance let me bump it to 5m for a CI run, then try to tighten it up once we have a run from TravisCI.

@nathanchance

Copy link
Copy Markdown
Member
[  227.350907] reboot: Power down

Could probably get away with 4m but I would rather stick with 5m so that we can be absolutely sure we timed out. Additionally, I think that it is worth mentioning in the comments why we increased the timeout. If you disagree, let me know and I'll approve this.

Changing -cpu to max in
ClangBuiltLinux/boot-utils#20
for PAC+BTI emulation seems to have slowed our boot tests down.

Fixes: https://github.com/ClangBuiltLinux/linux/issues/1058
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
@nickdesaulniers

Copy link
Copy Markdown
Member Author

I think that it is worth mentioning in the comments why we increased the timeout.

Done.

@nickdesaulniers nickdesaulniers merged commit 75ce73e into master Jun 23, 2020
@nickdesaulniers nickdesaulniers deleted the aarch64_timeout branch June 23, 2020 17:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants