Isaac Richter has uploaded this change for review. ( https://gem5-review.googlesource.com/10321

Change subject: x86: LFENCE should be a serializing instruction. Make it so
......................................................................

x86: LFENCE should be a serializing instruction. Make it so

According to the Intel SDM, no instruction following an LFENCE can begin
execution until after the LFENCE has executed. (This is
less strict than an actual serializing instruction, such as CPUID.)

Serializing instructions (per intel SDM Volume 3A Chapter 8.3) ensure
that no future instruction is fetched until after the serializing
instruction is completed.

By contrast, LFENCE (and other memory-ordering instructions) allows
future instructions to have been fetched; it just prohibits them from
being executed.

Change-Id: If89fcb552192326ab69a581f57d71c95cf5d90e7
Signed-off-by: Isaac Richter <isaac.rich...@rochester.edu>
---
M src/arch/x86/isa/decoder/two_byte_opcodes.isa
1 file changed, 2 insertions(+), 1 deletion(-)



diff --git a/src/arch/x86/isa/decoder/two_byte_opcodes.isa b/src/arch/x86/isa/decoder/two_byte_opcodes.isa
index 339e5a0..4ca23f1 100644
--- a/src/arch/x86/isa/decoder/two_byte_opcodes.isa
+++ b/src/arch/x86/isa/decoder/two_byte_opcodes.isa
@@ -778,7 +778,8 @@
             0x6: decode MODRM_MOD {
                 0x3: decode MODRM_REG {
                     0x5: BasicOperate::LFENCE(
-                                 {{/*Nothing*/}}, IsReadBarrier);
+                                 {{/*Nothing*/}}, IsReadBarrier,
+                                 IsSerializeAfter);
                     0x6: BasicOperate::MFENCE(
                                  {{/*Nothing*/}}, IsMemBarrier);
                     0x7: BasicOperate::SFENCE(

--
To view, visit https://gem5-review.googlesource.com/10321
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: If89fcb552192326ab69a581f57d71c95cf5d90e7
Gerrit-Change-Number: 10321
Gerrit-PatchSet: 1
Gerrit-Owner: Isaac Richter <isaac.rich...@rochester.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to