8000 [config] read configuration from exported values instead of envs by Azure-stars · Pull Request #10 · arceos-org/arceos-apps · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[config] read configuration from exported values instead of envs #10

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 6, 2025
Merged

[config] read configuration from exported values instead of envs #10

merged 2 commits into from
Jul 6, 2025

Conversation

Azure-stars
Copy link
Contributor

This PR is an adaptation of the changes in axplat_crates:

  1. It use CPU_NUM instead of SMP to specify number of CPUs.
  2. It read the configuration from the exported values instead of environment variables.

And it remove the aarch64-unknown-none target in CICD because of arceos-org/arceos#255.

@Azure-stars Azure-stars marked this pull request as draft June 18, 2025 06:44
@Azure-stars Azure-stars marked this pull request as ready for review June 18, 2025 06:52
equation314
equation314 reviewed Jun 18, 2025
View reviewed changes
@equation314
Copy link
Member
equation314 commented Jun 18, 2025 •
edited
Loading

Oops, there are still some apps fail to compile without axstd enabled.

To entirely resolve this problem and get rid of all special judgments, we can implement std::thread::available_parallelism in axstd (in a separate PR).

equation314
equation314 reviewed Jun 21, 2025
View reviewed changes
@Azure-stars Azure-stars changed the title [config] Read CPU_NUM from configuration file instead of environment variable [config] read configuration from exported values instead of envs Jul 2, 2025
equation314
equation314 reviewed Jul 3, 2025
View reviewed changes
@Azure-stars Azure-stars merged commit 7ad6143 into main Jul 6, 2025
10 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@equation314 equation314 equation314 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Azure-stars @equation314
0