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

Don't we need "SFENCE" for Non-Temporal Hint? #22

Closed
templexxx opened this issue Jul 30, 2017 · 2 comments
Closed

Don't we need "SFENCE" for Non-Temporal Hint? #22

templexxx opened this issue Jul 30, 2017 · 2 comments

Comments

@templexxx
Copy link

For example:

in raid/xor_gen_avx.asm line 93:

%define XSTR vmovntdq

don't we need to put a "sfence" in front of RET?

@gbtucker
Copy link
Contributor

Technically yes, you do need a fence before doing a read on parity written with NT store instructions. You can do this at any time after calling a series xor_gen().

@templexxx
Copy link
Author

@gbtucker

Thank you!

You really helped me a lot

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

No branches or pull requests

2 participants