Il 21/04/2014 21:20, Bandan Das ha scritto:
We track shadow vmcs fields through two static lists,
one for read only and another for r/w fields. However, with
addition of new vmcs fields, not all fields may be supported on
all hosts. If so, copy_vmcs12_to_shadow() trying to vmwrite on
Il 21/04/2014 21:20, Bandan Das ha scritto:
We track shadow vmcs fields through two static lists,
one for read only and another for r/w fields. However, with
addition of new vmcs fields, not all fields may be supported on
all hosts. If so, copy_vmcs12_to_shadow() trying to vmwrite on
Il 22/04/2014 21:31, Bandan Das ha scritto:
Paolo Bonzini writes:
Il 22/04/2014 12:25, Bandan Das ha scritto:
+ if (j < i)
+ shadow_read_write_fields[j] =
+ shadow_read_write_fields[i];
+ j++;
... you need to
Il 22/04/2014 21:31, Bandan Das ha scritto:
Paolo Bonzini pbonz...@redhat.com writes:
Il 22/04/2014 12:25, Bandan Das ha scritto:
+ if (j i)
+ shadow_read_write_fields[j] =
+ shadow_read_write_fields[i];
+ j++;
Paolo Bonzini writes:
> Il 22/04/2014 12:25, Bandan Das ha scritto:
>> + if (j < i)
>> + shadow_read_write_fields[j] =
>> + shadow_read_write_fields[i];
>> + j++;
>>> >
>>> > ... you need to
Il 22/04/2014 12:25, Bandan Das ha scritto:
>> + if (j < i)
>> + shadow_read_write_fields[j] =
>> + shadow_read_write_fields[i];
>> + j++;
>
> ... you need to respin anyway because the j++ is wrong. It should be
>
Paolo Bonzini writes:
> Il 21/04/2014 15:20, Bandan Das ha scritto:
>> +for (i = j = 0; i < max_shadow_read_write_fields; i++) {
>> +
>
> Extra empty line. Not a big deal, but...
>
>> +switch (shadow_read_write_fields[i]) {
>> +case GUEST_BNDCFGS:
>> +
Paolo Bonzini pbonz...@redhat.com writes:
Il 21/04/2014 15:20, Bandan Das ha scritto:
+for (i = j = 0; i max_shadow_read_write_fields; i++) {
+
Extra empty line. Not a big deal, but...
+switch (shadow_read_write_fields[i]) {
+case GUEST_BNDCFGS:
+
Il 22/04/2014 12:25, Bandan Das ha scritto:
+ if (j i)
+ shadow_read_write_fields[j] =
+ shadow_read_write_fields[i];
+ j++;
... you need to respin anyway because the j++ is wrong. It should be
inside the if.
Paolo Bonzini pbonz...@redhat.com writes:
Il 22/04/2014 12:25, Bandan Das ha scritto:
+ if (j i)
+ shadow_read_write_fields[j] =
+ shadow_read_write_fields[i];
+ j++;
... you need to respin anyway
Il 21/04/2014 15:20, Bandan Das ha scritto:
+ for (i = j = 0; i < max_shadow_read_write_fields; i++) {
+
Extra empty line. Not a big deal, but...
+ switch (shadow_read_write_fields[i]) {
+ case GUEST_BNDCFGS:
+ if
We track shadow vmcs fields through two static lists,
one for read only and another for r/w fields. However, with
addition of new vmcs fields, not all fields may be supported on
all hosts. If so, copy_vmcs12_to_shadow() trying to vmwrite on
unsupported hosts will result in a vmwrite error. For
We track shadow vmcs fields through two static lists,
one for read only and another for r/w fields. However, with
addition of new vmcs fields, not all fields may be supported on
all hosts. If so, copy_vmcs12_to_shadow() trying to vmwrite on
unsupported hosts will result in a vmwrite error. For
Il 21/04/2014 15:20, Bandan Das ha scritto:
+ for (i = j = 0; i max_shadow_read_write_fields; i++) {
+
Extra empty line. Not a big deal, but...
+ switch (shadow_read_write_fields[i]) {
+ case GUEST_BNDCFGS:
+ if (!vmx_mpx_supported())
14 matches
Mail list logo