From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=45814 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PXxRF-0003up-Ot for qemu-devel@nongnu.org; Wed, 29 Dec 2010 09:55:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PXxRE-0008Gy-At for qemu-devel@nongnu.org; Wed, 29 Dec 2010 09:55:41 -0500 Received: from mx1.redhat.com ([209.132.183.28]:65476) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PXxRE-0008Gn-3u for qemu-devel@nongnu.org; Wed, 29 Dec 2010 09:55:40 -0500 Date: Wed, 29 Dec 2010 16:55:18 +0200 From: "Michael S. Tsirkin" Message-ID: <20101229145518.GA8974@redhat.com> References: <1292879881$2997@local> <20101227140136.GA27055@redhat.com> <5C6885C942B1469F90662F3DC1C2D8A1@FSCPC> <20101228171955.GA2850@redhat.com> <20101228210831.GA4046@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: [Qemu-devel] Re: [PATCH] piix: use pci_config_set_prog_interface() List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sebastian Herbszt Cc: qemu-devel@nongnu.org On Wed, Dec 29, 2010 at 11:26:32AM +0100, Sebastian Herbszt wrote: > Michael S. Tsirkin wrote: > >On Tue, Dec 28, 2010 at 07:48:23PM +0100, Sebastian Herbszt wrote: > >>Michael S. Tsirkin wrote: > >>>On Tue, Dec 28, 2010 at 05:24:06PM +0100, Sebastian Herbszt wrote: > >>>>Michael S. Tsirkin wrote: > >>>>>On Mon, Dec 20, 2010 at 10:18:01PM +0100, Sebastian Herbszt wrote: > >>>>>>Use pci_config_set_prog_interface(). > >>>>>> > >>>>>>Signed-off-by: Sebastian Herbszt > >>>>> > >>>>>Since I was asked explicitly - I don't have a problem > >>>>>with these changes: both class and prog interface. > >>>>>However, they aren't all that useful in themselves. > >>>>> > >>>>>For class, what I would like to see is a system where > >>>>>the device class is put in the qdev info table, > >>>>>and where -device ? > >>>>>(and hopefully the legacy -help, -nic etc as well) > >>>>>use this information. > >>>> > >>>>I am not sure if you mean something like the patch below. > >>> > >>>Not exactly > >>> > >>>- I'd like to keep the pci_config_set_class in the devices, > >>> just make it do an assert. > >> > >>Assert on which condition? > > > >That PCI class matches device type defined in qdev. > >This will serve to verify that all devices are converted properly. > > > >>>- Nics already have DEFINE_NIC_PROPERTIES - can this be used somehow? > >>> Same for other devices ... > >> > >>We got DEFINE_NIC_PROPERTIES and DEFINE_BLOCK_PROPERTIES. > >>Something like DEFINE_PCI_PROPERTIES could be introduced, but i am not sure > > > >This shouldn't have to do anything with PCI. > >We should define the device type as NIC, > >PCI class can be derived from that. > > So you want to introduce a device property like "device_type" and set it to "NIC" and > then translate "NIC" to the correct PCI class code. Are you sure this is unambiguous? > "NIC" could be ethernet or token ring. A generic name like "HBA" could translate to even > more codes (scsi, ide, raid, sata, ...). > > >>which device properties it should hold (vendor_id, device_id, class, etc?). > >>Those will then be user-modifiable with e.g. -device e1000,pci_class=1234. > >> > >>Sebastian > > > >Why is this helpful? > >- tweaking class will just break guests > >- binary representation is unfriendly, let's not require users to read > > pci spec just to run qemu. > > "pci_class" is optional. > I just wanted to show that qdev device properties are > user-modifiable and i didn't find a way to prevent this. Ah, ok. I think properties starting with x- are internal. It's a kind of undocumented thing. -- MST