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.

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

Implement FXRSTOR and FXLOAD #1489

Open
MrSynAckSter-zz opened this issue Jul 19, 2019 · 4 comments
Open

Implement FXRSTOR and FXLOAD #1489

MrSynAckSter-zz opened this issue Jul 19, 2019 · 4 comments
Assignees
@Boyan-MILANOV
Labels
CHESS x86

Comments

@MrSynAckSter-zz
Copy link

FXRSTOR and FXLOAD are currently not implemented. We should implement them.

m.n.c.abstractcpu:WARNING: Unimplemented instruction: 0x00007fffffdcedc1:	0f ae 44 24 40	fxsave	[rsp + 0x40]

2019-07-19 12:06:07,684: [18842] m.n.c.abstractcpu:WARNING: Unimplemented instruction: 0x00007fffffdcedd6:	0f ae 4c 24 40	fxrstor	[rsp + 0x40]

This will be need separate implementations on x86 versus amd64 due to differing memory layouts.
@woodruffw woodruffw mentioned this issue Jul 19, 2019
Closed
@woodruffw
Copy link
Member

FXSAVE: https://www.felixcloutier.com/x86/fxsave
FXRSTOR: https://www.felixcloutier.com/x86/fxrstor

@tpetersonkth
Copy link

@woodruffw Are you trying to say that we don't need to implement them since they are only related to context switching?

@woodruffw
Copy link
Member

@pythonpsycho1337 Sorry, I'm not sure what you're referring to. We do indeed want to implement these.

@woodruffw
Copy link
Member

cc @Boyan-MILANOV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Boyan-MILANOV Boyan-MILANOV

Labels
CHESS x86
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@MrSynAckSter-zz @woodruffw @tpetersonkth @Boyan-MILANOV