From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MGDsz-0000dU-Bs for qemu-devel@nongnu.org; Mon, 15 Jun 2009 11:14:13 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MGDst-0000bq-9c for qemu-devel@nongnu.org; Mon, 15 Jun 2009 11:14:12 -0400 Received: from [199.232.76.173] (port=59950 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MGDss-0000bd-F7 for qemu-devel@nongnu.org; Mon, 15 Jun 2009 11:14:06 -0400 Received: from mx2.redhat.com ([66.187.237.31]:59565) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MGDsr-00077Q-TD for qemu-devel@nongnu.org; Mon, 15 Jun 2009 11:14:06 -0400 Message-ID: <4A3664A0.6080105@redhat.com> Date: Mon, 15 Jun 2009 18:11:28 +0300 From: Avi Kivity MIME-Version: 1.0 Subject: Re: Configuration vs. compat hints [was Re: [Qemu-devel] [PATCHv3 03/13] qemu: add routines to manage PCI capabilities] References: <20090610150129.GC28601@redhat.com> <200906101624.30659.paul@codesourcery.com> <20090610174301.GC7416@shareable.org> <20090610182227.GN28601@redhat.com> <20090610192702.GH7416@shareable.org> <1244796209.16425.20.camel@blaa> <4A326B5C.5010501@codemonkey.ws> <1244821292.30522.56.camel@blaa> <4A327E4A.7010300@codemonkey.ws> <1244825303.26769.19.camel@blaa> <20090614095016.GA7560@redhat.com> <4A3617D4.5090405@redhat.com> <4A36424D.3080901@codemonkey.ws> <4A364698.1090402@redhat.com> <4A364A9B.9070001@codemonkey.ws> <4A364E32.4010808@redhat.com> <4A365078.5080209@codemonkey.ws> <4A36529D.4060200@redhat.com> <4A3663C6.3070801@codemonkey.ws> In-Reply-To: <4A3663C6.3070801@codemonkey.ws> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Carsten Otte , Rusty Russell , kvm@vger.kernel.org, Mark McLoughlin , Glauber Costa , "Michael S. Tsirkin" , qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org, Blue Swirl , Christian Borntraeger , Paul Brook On 06/15/2009 06:07 PM, Anthony Liguori wrote: > Avi Kivity wrote: >> >> I'd object to any implicit addressing rules. If we have to say >> target=2,lun=7,street=8,city=9,state=99,zip=12345 instead of >> index=8345345235 so be it. > > The next observation is that while we expand the SCSI addressing, the > current propose flattens the PCI hierarchy (i.e. pci_addr=00:01.0). > > An alternative would be to either always expand or always flatten > addressing. I think the later has a lot of merit. Consider: > > -controller type=lsi1234,addr=00:01,name=blah > -controller-disk controller=blah,addr=00:01,name=sda > > -controller type=ide,addr=00.02,name=ide > -controller-disk controller=ide,addr=3,name=hdd > > -drive file=foo.img,controller-disk=sda > -drive file=bar.img,controller-disk=hdd > > This means that addr's format depends on the parent device node which > is a bit less explicit than the previous example. However, it is much > more consistent and easier to implement. Basically, when adding a > device to it's parent, you hand the parent the "addr" field and that > lets you say where you want to sit on the bus. I would prefer explicit names (pci_addr, lun, etc.) but would be okay with generic names too. There's value in sticking to well-understood names and address formats. -- error compiling committee.c: too many arguments to function