Skip to content
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

lib: cpu: create a new libmetal API metal_yield #311

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

wyr-7
Copy link
Contributor

@wyr-7 wyr-7 commented Oct 8, 2024

metal_yield would be managed at the OS level and dispatched to metal_cpu_yield, metal_sleep_usec, or others, it is more flexible to manage this in libmetal

lib/cpu.h Outdated Show resolved Hide resolved
define metal_yield in sys.h files is more flexible to manage this
in libmetal, metal_yield would be managed at the OS level and
dispatched to metal_cpu_yield, metal_sleep_usec, or others

Signed-off-by: Yongrong Wang <wangyongrong@xiaomi.com>
Copy link
Contributor

@arnopo arnopo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arnopo
Copy link
Contributor

arnopo commented Oct 16, 2024

@edmooring , @tnmysh,
could you have a look for the release , the OpenAMP/open-amp#613 depend on this one

@arnopo arnopo added this to the Release v2024.10 milestone Oct 17, 2024
@wyr-7
Copy link
Contributor Author

wyr-7 commented Oct 18, 2024

@edmooring , @tnmysh, could you have a look for the release , the OpenAMP/open-amp#613 depend on this one

@edmooring @tnmysh could you review this PR please?

Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go.

Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go.

@arnopo arnopo merged commit d0cd928 into OpenAMP:main Oct 21, 2024
6 checks passed
@arnopo arnopo changed the title lib/cpu.h: create a new libmetal API metal_yield lib: cpu: create a new libmetal API metal_yield Oct 22, 2024
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.

5 participants