[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH uq/master] kvm: Clean up irqfd API
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [PATCH uq/master] kvm: Clean up irqfd API |
Date: |
Sun, 09 Sep 2012 17:01:46 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120828 Thunderbird/15.0 |
On 08/20/2012 11:55 AM, Jan Kiszka wrote:
> No need to expose the fd-based interface, everyone will already be fine
> with the more handy EventNotifier variant. Rename the latter to clarify
> that we are still talking about irqfds here.
Thanks, applied.
>
> -int kvm_irqchip_add_irqfd(KVMState *s, int fd, int virq);
> -int kvm_irqchip_remove_irqfd(KVMState *s, int fd, int virq);
> -int kvm_irqchip_add_irq_notifier(KVMState *s, EventNotifier *n, int virq);
> -int kvm_irqchip_remove_irq_notifier(KVMState *s, EventNotifier *n, int virq);
> +int kvm_irqchip_add_irqfd_notifier(KVMState *s, EventNotifier *n, int virq);
> +int kvm_irqchip_remove_irqfd_notifier(KVMState *s, EventNotifier *n, int
> virq);
> #endif
Those names aren't particularly satisfying. add_irqfd_notifier implies
you want to be notified about irqfd events, but that's not what the
function does. Not sure what a good name would be.
--
error compiling committee.c: too many arguments to function
- Re: [Qemu-devel] [PATCH uq/master] kvm: Clean up irqfd API,
Avi Kivity <=