We don't want to turn this into a full instruction decoder, so there are
just a few hardcoded added cases to handle certain move instructions
that we see.

Signed-off-by: Gan Shun <[email protected]>
Change-Id: I0c51251232b27e2d33ffe2c0a30bacafe233f6af
---
 user/vmm/decode.c | 40 +++++++++++++++++++++++++++++++---------
 1 file changed, 31 insertions(+), 9 deletions(-)

diff --git a/user/vmm/decode.c b/user/vmm/decode.c
index 5df5095..a4446ae 100644
--- a/user/vmm/decode.c
+++ b/user/vmm/decode.c
@@ -98,24 +98,38 @@ static int target(void *insn, int *store)
                s = 4;
                break;
        case 0x0f:
-       switch(*word) {
-               case 0xb70f:
-                       s = 2;
-                       break;
-               default:
-                       fprintf(stderr, "can't get size of %02x/%04x @ %p\n", 
*byte, *word, byte);
-                       return -1;
-                       break;
+               switch (*word) {
+                       case 0xb70f:
+                               s = 2;
+                               break;
+                       default:
+                               fprintf(stderr, "can't get size of %02x/%04x @ 
%p\n", *byte,
+                                       *word, byte);
+                               return -1;
+                       }
+               break;
+       case 0x41:
+               /* VEX byte for modrm field */
+               switch (*word) {
+                       case 0x8a41:
+                               s = 1;
+                               break;
+                       default:
+                               fprintf(stderr, "unparsed vex instruction 
%02x/%04x @ %p\n",
+                                       *byte, *word, byte);
+                               return -1;
                }
                break;
        default:
                fprintf(stderr, "can't get size of %02x @ %p\n", *byte, byte);
+               fprintf(stderr, "can't get WORD of %04x @ %p\n", *word, word);
                return -1;
                break;
        }
 
        switch(*byte) {
        case 0x0f:
+       case 0x41:
                break;
        case 0x3a:
        case 0x8a:
@@ -147,6 +161,12 @@ static int insize(void *rip)
                kva++;
        }
 
+       /* return 3 to handle this specific instruction case. We don't want this
+        * to turn into a fully fledged decode.
+        * This specific instruction is an extended move using r9. It uses the
+        * VEX byte to extend the register bits. */
+       if (kva[0] == 0x41 && kva[1] == 0x8a && kva[2] == 0x01)
+               return 3;
        /* the dreaded mod/rm byte. */
        int mod = kva[1]>>6;
        int rm = kva[1] & 7;
@@ -226,7 +246,9 @@ int decode(struct guest_thread *vm_thread, uint64_t *gpa, 
uint8_t *destreg,
 
        *advance = insize(kva);
 
-       uint16_t ins = *(uint16_t *)(kva + (kva[0] == 0x44) + (kva[0] == 0x0f));
+       uint16_t ins = *(uint16_t *)(kva +
+           ((kva[0] == 0x44) || (kva[0] == 0x0f) || (kva[0] == 0x41)));
+
        DPRINTF("ins is %04x\n", ins);
 
        *destreg = (ins>>11) & 7;
-- 
2.8.0.rc3.226.g39d4020

-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to