On 2011-06-05 14:21, Avi Kivity wrote:
> On 06/03/2011 06:53 PM, Jan Kiszka wrote:
>> >> @@ -310,6 +310,7 @@ struct kvm_translation {
>> >> struct kvm_interrupt {
>> >> /* in */
>> >> __u32 irq;
>> >> + __u8 raise;
>> >> };
>> >
>> > This touches an existing ABI and corrupts the definition of
>> > KVM_INTERRUPT IOCTL. The might exist jurisdictions considering this a
>> > capital crime. :)
>> >
>> > You rather have to define a new CPU IRQ injection interface that
>> > supports both raising and lowering
>
> This is KVM_IRQ_LINE:
> It's so far associated with in-kernel irqchip input pins, not with raising CPU IRQs. > > >> and declare its availability via a >> > KVM_CAP. Don't forget to make it extensible (flags field) so that >> future >> > requirements can be added without breaking existing users. >> >> Or much easier (this is what PowerPC is doing): Define irq values in a >> way that they include a raise/lower flag. > > Much easier and much horribler. > Less horrible than overloading KVM_IRQ_LINE IMHO. The semantics of kvm_interrupt::irq are in arch hands anyway. Jan
signature.asc
Description: OpenPGP digital signature
