From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754226AbZGBPiX (ORCPT ); Thu, 2 Jul 2009 11:38:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752957AbZGBPiJ (ORCPT ); Thu, 2 Jul 2009 11:38:09 -0400 Received: from victor.provo.novell.com ([137.65.250.26]:46234 "EHLO victor.provo.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752761AbZGBPiI (ORCPT ); Thu, 2 Jul 2009 11:38:08 -0400 From: Gregory Haskins Subject: [KVM PATCH v9 0/5] irqfd fixes and enhancements To: kvm@vger.kernel.org Cc: linux-kernel@vger.kernel.org, mst@redhat.com, avi@redhat.com, davidel@xmailserver.org Date: Thu, 02 Jul 2009 11:37:55 -0400 Message-ID: <20090702153454.20186.99191.stgit@dev.haskins.net> User-Agent: StGIT/0.14.3 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org (Applies to kvm.git/master:1f9050fd) The following is the latest attempt to fix the races in irqfd/eventfd, as well as restore DEASSIGN support. For more details, please read the patch headers. As always, this series has been tested against the kvm-eventfd unit test and everything appears to be functioning properly. You can download this test here: ftp://ftp.novell.com/dev/ghaskins/kvm-eventfd.tar.bz2 Kind Regards, -Greg [Changelog: v9: *) Re-split v8:1/3 into v9[1/5, 2/5, 3/5] *) Fixed bad error handing in 3/5 *) Changed "init" bitfield in 5/5 to a bool *) Rebased to kvm.git/master:1f9050fd) v8: *) Rebased to kvm.git/master:beeaacd1) *) Dropped Davide's patch (2/5 in v7) since it's now upstream *) Folded v7's 1/5 and 3/5 together, and added a single eventfd hunk to convert wake_up_locked_polled to wake_up_polled *) Dropped irqfd->active bit in favor of irqfd_is_active() function *) Cleaned up comments in 1/3 *) Dropped v7's 5/5 (slow-work) *) Added new patch (3/3) which makes the cleanup-wq's creation dynamic so to avoid the resource penalty for guests that do not use irqfd. v7: *) Addressed minor-nit feedback from Michael *) Cleaned up patch headers *) Re-added separate slow-work feature patch to end for comparison v6: *) Removed slow-work in favor of using a dedicated single-thread workqueue. *) Condensed cleanup path to always use deferred shutdown *) Saved about 56 lines over v5, with the following diffstat: include/linux/kvm_host.h | 2 virt/kvm/eventfd.c | 248 ++++++++++++++++++----------------------------- 2 files changed, 97 insertions(+), 153 deletions(-) v5: Untracked.. ] --- Gregory Haskins (5): KVM: create irqfd-cleanup-wq on demand KVM: add irqfd DEASSIGN feature KVM: Fix races in irqfd using new eventfd_kref_get interface eventfd: use locked POLLHUP kvm: prepare irqfd for having interrupts disabled during eventfd->release fs/eventfd.c | 7 - include/linux/kvm.h | 2 include/linux/kvm_host.h | 6 + virt/kvm/eventfd.c | 281 ++++++++++++++++++++++++++++++++++++---------- 4 files changed, 229 insertions(+), 67 deletions(-) -- Signature