diff options
| author | Gabe Black <gblack@eecs.umich.edu> | 2009-02-01 17:05:37 -0800 |
|---|---|---|
| committer | Gabe Black <gblack@eecs.umich.edu> | 2009-02-01 17:05:37 -0800 |
| commit | 6b53b8387e1a0c05d878d47962c27bf7493b6d36 (patch) | |
| tree | 07f203425bf7401c3de0b473b0c594dc595ed0ed /src/arch | |
| parent | c0cd58812e23ff540c0ef80203f1baad16a9eda1 (diff) | |
| download | gem5-6b53b8387e1a0c05d878d47962c27bf7493b6d36.tar.xz | |
X86: Make the chks microop check for the right int descriptor type.
Diffstat (limited to 'src/arch')
| -rw-r--r-- | src/arch/x86/isa/insts/romutil.py | 11 | ||||
| -rw-r--r-- | src/arch/x86/isa/microops/regop.isa | 6 |
2 files changed, 7 insertions, 10 deletions
diff --git a/src/arch/x86/isa/insts/romutil.py b/src/arch/x86/isa/insts/romutil.py index 2724637d1..9b4a80d44 100644 --- a/src/arch/x86/isa/insts/romutil.py +++ b/src/arch/x86/isa/insts/romutil.py @@ -43,18 +43,9 @@ def rom ld t2, idtr, [1, t0, t4], 8, dataSize=8, addressSize=8 ld t4, idtr, [1, t0, t4], dataSize=8, addressSize=8 - # Check permissions + # Make sure the descriptor is a legal gate. chks t1, t4, IntGateCheck - mov t1, t1, t4, dataSize=8 - - # Check that it's the right type - srli t4, t1, 40, dataSize=8 - andi t4, t4, 0xe, dataSize=8 - xori t4, t4, 0xe, flags=(EZF,), dataSize=8 - fault "new GeneralProtection(0)", flags=(nCEZF,) - - # # Get the target CS descriptor using the selector in the gate # descriptor. diff --git a/src/arch/x86/isa/microops/regop.isa b/src/arch/x86/isa/microops/regop.isa index 492452a51..200024690 100644 --- a/src/arch/x86/isa/microops/regop.isa +++ b/src/arch/x86/isa/microops/regop.isa @@ -1067,9 +1067,15 @@ let {{ "not implemented.\\n"); break; case SegIntGateCheck: + // Check permissions. if (desc.dpl < m5reg.cpl) { fault = new GeneralProtection((uint16_t)selector); } + // Make sure the gate's the right type. + if (m5reg.mode == LongMode && ((desc.type & 0xe) != 0xe) || + ((desc.type & 0x6) != 0x6)) { + fault = new GeneralProtection(0); + } break; case SegSSCheck: if (selector.si || selector.ti) { |