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

xtensa: fix insecure userspace warning wording #79336

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

dcpleung
Copy link
Member

@dcpleung dcpleung commented Oct 2, 2024

"in behave of" -> "on behalf of", which is more accurate of what is actually happening in the code.

"in behave of" -> "on behalf of", which is more accurate of
what is actually happening in the code.

Signed-off-by: Daniel Leung <[email protected]>
@dcpleung dcpleung added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Oct 2, 2024
@dcpleung dcpleung marked this pull request as ready for review October 2, 2024 18:28
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code area: Xtensa Xtensa Architecture labels Oct 2, 2024
@fabiobaltieri fabiobaltieri merged commit 8638e41 into zephyrproject-rtos:main Oct 3, 2024
25 checks passed
@@ -7,5 +7,5 @@ add_subdirectory(core)
if (CONFIG_XTENSA_INSECURE_USERSPACE)
message(WARNING "
This userspace implementation uses the window ABI this means that the kernel
will spill registers in behave of the userpsace. Use it carefully.")
will spill registers on behalf of the userpsace. Use it carefully.")
Copy link
Member

Choose a reason for hiding this comment

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

Thanks! We missed the misspelled "userpsace", though.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ooooooops... will fix.

@dcpleung dcpleung deleted the xtensa/mmu_pharse branch October 3, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Xtensa Xtensa Architecture size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants