Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755409AbZGFUjz (ORCPT ); Mon, 6 Jul 2009 16:39:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753676AbZGFUjr (ORCPT ); Mon, 6 Jul 2009 16:39:47 -0400 Received: from victor.provo.novell.com ([137.65.250.26]:54799 "EHLO victor.provo.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752100AbZGFUjq (ORCPT ); Mon, 6 Jul 2009 16:39:46 -0400 From: Gregory Haskins Subject: [KVM PATCH v9 0/2] iosignalfd To: kvm@vger.kernel.org Cc: linux-kernel@vger.kernel.org, mst@redhat.com, avi@redhat.com Date: Mon, 06 Jul 2009 16:33:10 -0400 Message-ID: <20090706202742.14222.65548.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 Content-Length: 4336 Lines: 112 (Applies to kvm.git/master:c5b13264e) This is v9 of the series. For more details, please see the header to patch 2/2. This series has been tested against the kvm-eventfd unit test, and appears to be functioning properly. You can download this test here: ftp://ftp.novell.com/dev/ghaskins/kvm-eventfd.tar.bz2 (Note: unit test has been updated to accomodate new feature-set) Please consider for inclusion to kvm.git [ Changelog: v9: *) Rebased onto Michael's new iodev locking scheme *) Removed group+item nesting in favor of using iodev facility *) Removed iodev-limit patch since we no longer create our own devices. The limit feature itself may be useful but it is now out of scope from iosignalfd and thus should be addressed separately. *) Rebased to kvm.git/master:c5b13264e v8: *) Addressed Avi's review comments: *) Simplified the unregister_dev logic in patch 1/3 *) Implemented a per-vm io-device limit (patch 2/3) *) Removed spurious whitespace hunk *) changed the data-match from a void* to a u64 *) check group-length violations before wildcarding v7: *) Implemented a resource limit (CONFIG_KVM_MAX_IOSIGNALFD_ITEMS) to limit malicious/broken userspace from consuming arbitrary kernel memory. *) Rebased to kvm.git/master:c27b64a0, which already includes Marcelo's irq-lock rework. v6: *) Removed "FIXME?" comment on choice over RCU vs SRCU after discussion/numbers from Paul. I think RCU is fine to use for now based on the conversation. We can always convert it later if need be. *) Fixed the "group" free path to eliminate an RCU related race *) Fixed a memory/eventfd leak on shutdown for any iosignalfd's which were still active at the time the guest shuts down. *) Beefed up comments *) Rebased to kvm.git/master:0281e88f + irq locking rework and verified that kvm-eventfd unit test still passes. v5: *) Removed "cookie" field, which was a misunderstanding on my part on what Avi wanted for a data-match feature *) Added a new "trigger" data-match feature which I think is much closer to what we need. *) We retain the dev_count field in the io_bus infrastructure and instead back-fill the array on removal. *) Various minor cleanups *) Rebased to kvm.git/master:25deed73 v4: *) Fixed a bug in the original 2/4 where the PIT failure case would potentially leave the io_bus components registered. *) Condensed the v3 2/4 and 3/4 into one patch (2/2) since the patches became interdependent with the fix described above *) Rebased to kvm.git/master:74dfca0a v3: *) fixed patch 2/4 to handle error cases instead of BUG_ON *) implemented same HAVE_EVENTFD protection mechanism as irqfd to prevent compilation errors on unsupported arches *) completed testing *) rebased to kvm.git/master:7391a6d5 v2: *) added optional data-matching capability (via cookie field) *) changed name from iofd to iosignalfd *) added io_bus unregister function *) implemented deassign feature v1: *) original release (integrated into irqfd v7 series as "iofd") ] --- Gregory Haskins (2): KVM: add iosignalfd support KVM: make io_bus interface more robust arch/x86/kvm/i8254.c | 22 +++- arch/x86/kvm/i8259.c | 9 + arch/x86/kvm/x86.c | 1 include/linux/kvm.h | 15 ++ include/linux/kvm_host.h | 20 ++- virt/kvm/coalesced_mmio.c | 8 + virt/kvm/eventfd.c | 280 +++++++++++++++++++++++++++++++++++++++++++++ virt/kvm/ioapic.c | 8 + virt/kvm/kvm_main.c | 52 +++++++- 9 files changed, 396 insertions(+), 19 deletions(-) -- Signature -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/