[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD
From: |
Chen, Tiejun |
Subject: |
Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D |
Date: |
Thu, 29 May 2014 01:15:35 +0000 |
> -----Original Message-----
> From: Stefano Stabellini [mailto:address@hidden
> Sent: Wednesday, May 28, 2014 8:30 PM
> To: Chen, Tiejun
> Cc: Stefano Stabellini; address@hidden; address@hidden;
> address@hidden; address@hidden;
> address@hidden; address@hidden;
> address@hidden; Kay, Allen M; Zhang, Yang Z
> Subject: RE: [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD
> passthrough with VT-D
>
[snip]
> > > > +/*
> > > > + * Currently we just pass this physical host bridge for IGD, 00:02.0.
> > > > + */
> > > > +static int is_igd_passthrough(PCIDevice *pci_dev) {
> > > > + PCIDevice *f = pci_dev->bus->devices[PCI_DEVFN(2, 0)];
> > > > + if (pci_dev->bus->devices[PCI_DEVFN(2, 0)]) {
> > >
> > > Isn't the purpose of this function to check that the *current*
> > > device is the graphic passthrough device?
> >
> > No.
> >
> > > In that case, shouldn't it just be:
> > >
> > > if (pci_dev) {
> > >
> >
> > Here pci_dev is just that host bridge, so here we have to get that real
> passthrough device by that given devfn to further confirm.
>
> I understand now, thanks for the explanation. Maybe you want to expand the
> comment on top of is_igd_passthrough.
Sure, its easy to do :)
Thanks
Tiejun
- Re: [Qemu-devel] [v3][PATCH 1/5] xen, gfx passthrough: basic graphics passthrough support, (continued)
- [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Tiejun Chen, 2014/05/26
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Stefano Stabellini, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Chen, Tiejun, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Stefano Stabellini, 2014/05/28
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D,
Chen, Tiejun <=
[Qemu-devel] [v3][PATCH 5/5] xen, gfx passthrough: add opregion mapping, Tiejun Chen, 2014/05/26
- Re: [Qemu-devel] [v3][PATCH 5/5] xen, gfx passthrough: add opregion mapping, Stefano Stabellini, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 5/5] xen, gfx passthrough: add opregion mapping, Chen, Tiejun, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 5/5] xen, gfx passthrough: add opregion mapping, Stefano Stabellini, 2014/05/28
- Re: [Qemu-devel] [v3][PATCH 5/5] xen, gfx passthrough: add opregion mapping, Chen, Tiejun, 2014/05/28
[Qemu-devel] [v3][PATCH 4/5] xen, gfx passthrough: create host bridge to passthrough, Tiejun Chen, 2014/05/26