Re: [PATCH v3 0/4] KVM & genirq: Enable adaptive IRQ sharing for passed-through devices

2010-12-14 Thread Thomas Gleixner
On Mon, 13 Dec 2010, Jan Kiszka wrote:

> This addresses the review comments of the previous round:
>  - renamed irq_data::status to drv_status
>  - moved drv_status around to unbreak GENERIC_HARDIRQS_NO_DEPRECATED
>  - fixed signature of get_irq_status (irq is now unsigned int)
>  - converted register_lock into a global one
>  - fixed critical white space breakage (that I just left in to check if
>anyone is actually reading the code, of course...)

Just for the record, you either missed or introduced some new white
space noise :)
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 0/4] KVM & genirq: Enable adaptive IRQ sharing for passed-through devices

2010-12-14 Thread Avi Kivity

On 12/14/2010 12:59 AM, Jan Kiszka wrote:

Final but critical question: Who will pick up which bits?



The procedure which has served us well in the past is that tip picks up 
the irq stuff and sticks them in a fast-forward-only branch; kvm merges 
the branch and applies the kvm bits on top.


--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v3 0/4] KVM & genirq: Enable adaptive IRQ sharing for passed-through devices

2010-12-13 Thread Jan Kiszka
This addresses the review comments of the previous round:
 - renamed irq_data::status to drv_status
 - moved drv_status around to unbreak GENERIC_HARDIRQS_NO_DEPRECATED
 - fixed signature of get_irq_status (irq is now unsigned int)
 - converted register_lock into a global one
 - fixed critical white space breakage (that I just left in to check if
   anyone is actually reading the code, of course...)

Note: The KVM patch still depends on
http://thread.gmane.org/gmane.comp.emulators.kvm.devel/64515

Thanks for all comments!

Final but critical question: Who will pick up which bits?

Jan Kiszka (4):
  genirq: Introduce driver-readable IRQ status word
  genirq: Inform handler about line sharing state
  genirq: Add support for IRQF_COND_ONESHOT
  KVM: Allow host IRQ sharing for passed-through PCI 2.3 devices

 Documentation/kvm/api.txt |   27 
 arch/x86/kvm/x86.c|1 +
 include/linux/interrupt.h |   15 ++
 include/linux/irq.h   |2 +
 include/linux/kvm.h   |6 +
 include/linux/kvm_host.h  |   10 ++-
 kernel/irq/manage.c   |   77 ++-
 virt/kvm/assigned-dev.c   |  336 -
 8 files changed, 436 insertions(+), 38 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html