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

Guoweikang/aarch64 vfp fix #17

Merged
merged 2 commits into from
Jul 31, 2024
Merged

Guoweikang/aarch64 vfp fix #17

merged 2 commits into from
Jul 31, 2024

Conversation

guoweikang
Copy link
Contributor

1 Exception doesn't save VFP registers
2 kernel generate code shouldn't use VFP registers,
  need close -neon and -fp-armv8
3 taskctx need save/restore user application VFP ctx,
  need access VFP registers

After rustc update, it would complain when 2&3 happend,
on this commit :65a4f5896317115cf1edc74800f0da5d92923243,
opened neon when fp_smid enable,it broken 2, so it create
bug; after return user from kernel trap, usr vfp registers
ctx are changed;

This commit close neon when complie kernel, taskctx need access
VFP registers through build.rs to fix

Signed-off-by: guoweikang <[email protected]>
issue: #16

--------
1 Exception doesn't save VFP registers
2 kernel generate code shouldn't use VFP registers,
  need close -neon and -fp-armv8
3 taskctx need save/restore user application VFP ctx,
  need access VFP registers

After rustc update, it would complain when 2&3 happend,
on this commit :65a4f5896317115cf1edc74800f0da5d92923243,
opened neon when fp_smid enable,it broken 2, so it create
bug; after return user from kernel trap, usr vfp registers
ctx are changed;

This commit close neon when complie kernel, taskctx need access
VFP registers through build.rs to fix

Signed-off-by: guoweikang <[email protected]>
@guoweikang guoweikang force-pushed the guoweikang/aarch64_vfp_fix branch from bf621a8 to aba94e9 Compare July 26, 2024 10:21
@Azure-stars Azure-stars merged commit a8a3df3 into main Jul 31, 2024
20 of 51 checks passed
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.

2 participants