Skip to content

Do not run PGO/BOLT in x64 Linux alt builds #141777

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented May 30, 2025

Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed here.

r? @marcoieni

try-job: dist-x86_64-linux*

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels May 30, 2025
@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit e28d0c7 with merge 31b9ec6

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from e28d0c7 to 318d1e9 Compare May 30, 2025 16:34
@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 318d1e9 with merge 6df9b4f

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from 318d1e9 to 65bb0d4 Compare May 30, 2025 17:38
@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 65bb0d4 with merge 841e227

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

Signed-off-by: Jakub Beránek <berykubik@gmail.com>
@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from 65bb0d4 to 033ea63 Compare May 30, 2025 18:36
@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

@bors2 try

Fucking submodules.

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 033ea63 with merge dce110a

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

@bors2 try cancel

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

Try build cancelled.
Cancelled workflows:

@Kobzol
Copy link
Contributor Author

Kobzol commented May 30, 2025

Nice, the alt job took only ~1.5h 🎉 Now let's see how long it takes on a free runner.

@bors2 try

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

⌛ Trying commit 6e64f8f with merge 713ede4

rust-bors bot added a commit that referenced this pull request May 30, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@mati865
Copy link
Member

mati865 commented May 30, 2025

IIRC results from this build are used for test jobs. So, they will likely slow down a bit but probably not by much.

@rust-bors
Copy link

rust-bors bot commented May 30, 2025

💔 Test failed

@Kobzol
Copy link
Contributor Author

Kobzol commented May 31, 2025

Ok, looks like free runner is a no-go because of disk space, I guess that makes sense.

IIRC results from this build are used for test jobs. So, they will likely slow down a bit but probably not by much.

Thanks, I'll have to check that out.

@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from 6e64f8f to eaaa846 Compare June 2, 2025 10:03
@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 2, 2025

The PGO/BOLT optimizations will be only missing from LLVM, I hope that won't be such an issue, given that most tests execute only very short invocations of rustc anyway. Let's try with CodeBuild now.

@bors2 try

rust-bors bot added a commit that referenced this pull request Jun 2, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works.

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@rust-bors
Copy link

rust-bors bot commented Jun 2, 2025

⌛ Trying commit eaaa846 with merge 336a518

@rust-bors
Copy link

rust-bors bot commented Jun 2, 2025

☀️ Try build successful (CI)
Build commit: 336a518 (336a51809f5af5b476831469f189e6457aa71ef3)

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 2, 2025

~1h 45m on Codebuild, ideal.

@Kobzol Kobzol marked this pull request as ready for review June 2, 2025 13:14
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 2, 2025
@Kobzol Kobzol force-pushed the dist-linux-alt-no-pgo-bolt branch from eaaa846 to 8f22a50 Compare June 3, 2025 08:23
@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 3, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented Jun 3, 2025

⌛ Trying commit 8f22a50 with merge 6a184ec

rust-bors bot added a commit that referenced this pull request Jun 3, 2025
Do not run PGO/BOLT in x64 Linux alt builds

Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
@marcoieni
Copy link
Member

this is great, thanks ❤️
Feel free to r=me if the timing is good

@rust-bors
Copy link

rust-bors bot commented Jun 3, 2025

☀️ Try build successful (CI)
Build commit: 6a184ec (6a184ec3874dd1909ca928da012736b935aedb3e)

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 3, 2025

2h 45m. A bit borderline, but we can always switch to the 8-core runner if it becomes too much. And at least it shouldn't be the bottleneck anymore.

@bors r=marcoieni

@bors
Copy link
Collaborator

bors commented Jun 3, 2025

📌 Commit 8f22a50 has been approved by marcoieni

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 3, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 5, 2025
… r=marcoieni

Do not run PGO/BOLT in x64 Linux alt builds

Should unblock rust-lang#131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? `@marcoieni`

try-job: `dist-x86_64-linux*`
bors added a commit that referenced this pull request Jun 5, 2025
Rollup of 8 pull requests

Successful merges:

 - #140638 (UnsafePinned: also include the effects of UnsafeCell)
 - #141272 (modularize the config module bootstrap)
 - #141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - #141870 (Fix broken link to rustc_type_ir module in serialization docs)
 - #141938 (update rust offload bootstrap)
 - #141962 (rustc-dev-guide subtree update)
 - #141965 (`tests/ui`: A New Order [3/N])
 - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 5, 2025
… r=marcoieni

Do not run PGO/BOLT in x64 Linux alt builds

Should unblock rust-lang#131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).

Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477).

r? ``@marcoieni``

try-job: `dist-x86_64-linux*`
bors added a commit that referenced this pull request Jun 5, 2025
Rollup of 6 pull requests

Successful merges:

 - #140638 (UnsafePinned: also include the effects of UnsafeCell)
 - #141777 (Do not run PGO/BOLT in x64 Linux alt builds)
 - #141938 (update rust offload bootstrap)
 - #141962 (rustc-dev-guide subtree update)
 - #141965 (`tests/ui`: A New Order [3/N])
 - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`)

r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants