From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [Qemu-devel] [PATCH 3/3] Check pci slot number in parse_pci_devfn Date: Tue, 24 Aug 2010 15:04:44 +0300 Message-ID: <4C73B55C.3050800@redhat.com> References: <20100824063307.GA10815@kt> <20100824064927.GB11495@kt> <20100824113521.GA7201@valinux.co.jp> <4C73B01A.1050908@redhat.com> <20100824120745.GC7201@valinux.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Ken CC , kvm@vger.kernel.org, qemu-devel@nongnu.org, "Michael S. Tsirkin" To: Isaku Yamahata Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57659 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751561Ab0HXMEu (ORCPT ); Tue, 24 Aug 2010 08:04:50 -0400 In-Reply-To: <20100824120745.GC7201@valinux.co.jp> Sender: kvm-owner@vger.kernel.org List-ID: On 08/24/2010 03:07 PM, Isaku Yamahata wrote: > On Tue, Aug 24, 2010 at 02:42:18PM +0300, Avi Kivity wrote: >> On 08/24/2010 02:35 PM, Isaku Yamahata wrote: >>> Add Cc: mst@redhat.com. >>> >>> MAX_PCI_SLOTS should be in pci.h instead of qdev.h? >>> And the name should be start with PCI_ prefix for consistency? >>> >>> Except that, the patches look okay. >>> >> These aren't slots, are they? They are functions. > The function checks if given $slot.$fn (or $slot) is valid. > So it's slots. max 32. + assert(devfn< PCIBUS_MAX_DEVICES); Looks like we're comparing a function number to PCIBUS_MAX_DEVICES. -- error compiling committee.c: too many arguments to function