From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:35853) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ugvg3-0000bW-MJ for qemu-devel@nongnu.org; Mon, 27 May 2013 07:33:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ugvg1-0004Dx-4e for qemu-devel@nongnu.org; Mon, 27 May 2013 07:33:23 -0400 Received: from mail-bk0-x22a.google.com ([2a00:1450:4008:c01::22a]:49364) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ugvg0-0004Dn-V1 for qemu-devel@nongnu.org; Mon, 27 May 2013 07:33:21 -0400 Received: by mail-bk0-f42.google.com with SMTP id jk13so3645966bkc.1 for ; Mon, 27 May 2013 04:33:20 -0700 (PDT) Date: Mon, 27 May 2013 13:33:16 +0200 From: Stefan Hajnoczi Message-ID: <20130527113316.GA22343@stefanha-thinkpad.redhat.com> References: <1369435455-15885-1-git-send-email-akong@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1369435455-15885-1-git-send-email-akong@redhat.com> Subject: Re: [Qemu-devel] [PATCH] kvm: exclude ioeventfd from counting kvm_io_range limit List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amos Kong Cc: stefanha@redhat.com, kvm@vger.kernel.org, gleb@redhat.com, qemu-devel@nongnu.org On Sat, May 25, 2013 at 06:44:15AM +0800, Amos Kong wrote: > We can easily reach the 1000 limit by start VM with a couple > hundred I/O devices (multifunction=on). The hardcode limit > already been adjusted 3 times (6 ~ 200 ~ 300 ~ 1000). > > In userspace, we already have maximum file descriptor to > limit ioeventfd count. But kvm_io_bus devices also are used > for pit, pic, ioapic, coalesced_mmio. They couldn't be limited > by maximum file descriptor. > > Currently only ioeventfds take too much kvm_io_bus devices, > so just exclude it from counting kvm_io_range limit. > > Also fixed one indent issue in kvm_host.h > > Signed-off-by: Amos Kong > --- > include/linux/kvm_host.h | 3 ++- > virt/kvm/eventfd.c | 2 ++ > virt/kvm/kvm_main.c | 3 ++- > 3 files changed, 6 insertions(+), 2 deletions(-) Reviewed-by: Stefan Hajnoczi