From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=36631 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PIQnt-00066w-GK for qemu-devel@nongnu.org; Tue, 16 Nov 2010 14:02:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PIQnr-000645-7j for qemu-devel@nongnu.org; Tue, 16 Nov 2010 14:02:53 -0500 Received: from mx1.redhat.com ([209.132.183.28]:48330) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PIQnq-000641-T2 for qemu-devel@nongnu.org; Tue, 16 Nov 2010 14:02:51 -0500 Date: Tue, 16 Nov 2010 21:02:46 +0200 From: Gleb Natapov Message-ID: <20101116190246.GA27851@redhat.com> References: <1289749181-12070-1-git-send-email-gleb@redhat.com> <1289749181-12070-16-git-send-email-gleb@redhat.com> <20101115084242.GG7948@redhat.com> <20101116141112.GS7948@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In-Reply-To: Subject: [Qemu-devel] Re: [PATCHv4 15/15] Pass boot device list to firmware. List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: kvm@vger.kernel.org, mst@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, alex.williamson@redhat.com, kevin@koconnor.net On Tue, Nov 16, 2010 at 06:30:19PM +0000, Blue Swirl wrote: > >> Perhaps the FW path should use device class names if no name is specif= ied. > > What do you mean by "device class name". We can do something like this: > > if (dev->child_bus.lh_first) > > =9A =9A =9A =9Areturn dev->child_bus.lh_first->info->name; > > > > i.e if there is child bus use its bus name as fw name. This will make > > all pci devices to have "pci" as fw name automatically. The problem is > > that theoretically same device can provide different buses. >=20 > I meant PCI class name, like "display" for display controllers, > "network" for NICs etc. >=20 That is what my pci bus related patch is doing already. > >> I'll try Sparc32 to see how this fits there. >=20 > Except bootindex is not implemented for SCSI. Will look into adding it. -- Gleb.