> -----Original Message----- > From: Michael S. Tsirkin [mailto:m...@redhat.com] > Sent: Tuesday, June 03, 2014 4:37 AM > To: Chen, Tiejun > Cc: anthony.per...@citrix.com; stefano.stabell...@eu.citrix.com; > kelly.zyta...@amd.com; qemu-devel@nongnu.org; > xen-de...@lists.xensource.com; peter.mayd...@linaro.org; > anth...@codemonkey.ws; Kay, Allen M; Zhang, Yang Z > Subject: Re: [v4][PATCH 4/5] xen, gfx passthrough: create host bridge to > passthrough > > On Fri, May 30, 2014 at 04:59:28PM +0800, Tiejun Chen wrote: > > Implement that pci host bridge to specific to passthrough. Actually > > this just inherit the standard one. > > > > Signed-off-by: Tiejun Chen <tiejun.c...@intel.com> > > Can this actually inherit TYPE_I440FX_PCI_DEVICE?
Yes, but > The parent is TYPE_PCI_DEVICE, but if it was TYPE_I440FX_PCI_DEVICE then > you could re-use i440fx_initfn. > And you could do *pi440fx_state = I440FX_PCI_DEVICE(d) unconditionally. > one of reasons to implement our own pci host bridge which just inherit that standard one, please refer to some comments from v1: http://patchwork.ozlabs.org/patch/322443/ And I also think this separation may be flexible to extend more in the future. Thanks Tiejun