Re: [PATCH 1/3] KVM: VMX: Introduce per-VM I/O permission bitmaps

[Jim Mattson]

Since these pages are typically not used, can we allocate them conditionally?

On Fri, May 11, 2018 at 8:39 AM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
>>       /*
>> -      * A vmexit (to either L1 hypervisor or L0 userspace) is always needed
>> -      * for I/O port accesses.
>> +      * Merging of IO bitmap not currently supported.
>
> s/not/is not/
>
> And with that
>
> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
>
> Thank you!
>> +      * Rather, exit every time.
>>        */
>>       exec_control &= ~CPU_BASED_USE_IO_BITMAPS;
>>       exec_control |= CPU_BASED_UNCOND_IO_EXITING;
>> --
>> 2.7.4
>>