Re: [PATCH] KVM: x86: Extend KVM_SET_VCPU_EVENTS with selective updates

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

 



Avi Kivity wrote:
> On 12/06/2009 07:12 PM, Jan Kiszka wrote:
>> Avi Kivity wrote:
>>   
>>> On 12/06/2009 06:55 PM, Jan Kiszka wrote:
>>>     
>>>> User space may not want to overwrite asynchronously changing VCPU event
>>>> states on write-back. So allow to skip nmi.pending and sipi_vector by
>>>> setting corresponding bits in the flags field of kvm_vcpu_events.
>>>>
>>>>        
>>> I think a positive flag (do update nmi and sipi_vector) will cause less
>>> confusion, no?  If we do that, we'll need to set them on
>>> KVM_GET_VCPU_EVENTS.
>>>      
>> I'm fine with the former but I don't understand the latter.
>>
>>    
> 
> What I meant was the on KVM_GET_VCPU_EVENTS, the two flags should be
> set.  This way, the meaning of the flags is identical on both: the
> associated field contains valid data.
> 
> I made this change to your v2 and applied and queued for .33.  Please
> review (in next) to make sure it makes sense.
> 

OK, but calling these bits "SET" makes no sense anymore. What about
KVM_VCPUEVENT_VALID_*?

Jan

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux