Avi Kivity wrote:
> Laurent Vivier wrote:
>> Avi Kivity wrote:
>>   
>>> Laurent Vivier wrote:
>>>     
>>>> These patches split the emulator in two parts: one to decode the
>>>> instruction,
>>>> the other to execute it. The decode part is then called only when needed.
>>>>
>>>>   
>>>>       
>>> Patchset looks good, but fails booting FC6 x86-64 on Intel.  It may be a
>>> merge error (did not apply cleanly due to other changes).  I pushed this
>>> as a 'split-emulator' branch on the kvm.git repository.
>>>
>>>     
>> I think I found the bug (not a merge error...): I just supposed that an
>> instruction fetch cannot failed.
>>
>>   
> 
> Interesting.  I don't see how an instruction fetch can fail on
> uniprocessor.  Can you give details of the failure?
> 
> Instruction fetches can fail on SMP so a fix is certainly needed.

OK, I spoke too fast.

x86_decode_insn() fails because it is not able to decode:

0xffffffff8110b7ef <__copy_user_nocache+47>:    movnti %r11,(%rdi)
or
0xffffffff8110b7ef <__copy_user_nocache+47>:    0x4c    0x0f    0xc3    0x1f

0x4c is decoded as a REX prefix.
0x0f is decoded as a Two-byte opcode
but 0xc3 is unknown in twobyte_table, so we exit because of an unrecognized
opcode ("Cannot emulate").

Some comments ?

Laurent
-- 
------------- [EMAIL PROTECTED]  --------------
          "Software is hard" - Donald Knuth

Attachment: signature.asc
Description: OpenPGP digital signature

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Reply via email to