[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH for-4.5 v6 04/16] xen: Add vmware_port support



>>> On 23.09.14 at 19:16, <boris.ostrovsky@xxxxxxxxxx> wrote:
> On 09/20/2014 02:07 PM, Don Slutz wrote:
>> @@ -2064,6 +2065,42 @@ svm_vmexit_do_vmsave(struct vmcb_struct *vmcb,
>>       return;
>>   }
>>   
>> +static void svm_vmexit_gp_intercept(struct cpu_user_regs *regs,
>> +                                    struct vcpu *v)
>> +{
>> +    struct vmcb_struct *vmcb = v->arch.hvm_svm.vmcb;
>> +    /*
>> +     * Just use 15 for the instruction length; vmport_gp_check will
>> +     * adjust it.  This is because
>> +     * __get_instruction_length_from_list() has issues, and may
>> +     * require a double read of the instruction bytes.  At some
>> +     * point a new routine could be added that is based on the code
>> +     * in vmport_gp_check with extensions to make it more general.
>> +     * Since that routine is the only user of this code this can be
>> +     * done later.
>> +     */
>> +    unsigned long inst_len = 15;
> 
> Can you add a comment describing why you chose 15?

I think this much of architecture knowledge can be assumed as given.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.