Re: [Qemu-devel] [v3][PATCH 0/5] xen: introduce new machine for IGD passthrough

[Michael S. Tsirkin]

On Thu, Jul 31, 2014 at 08:09:30PM +0800, Tiejun Chen wrote:
> v3:
> 
> * Drop patch #4
> * Add one patch #1 from Michael
> * Rebase

You added my patch but don't use it, so most of
my comment weren't addressed.
Do you plan to send v4 to address them?


> v2:
> 
> * Fix some coding style
> * New patch to separate i440fx_init
> * Just add prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough
> * Based on patch #2 to regenerate
> * Unify prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough like patch #3
> * Test: boot with a preinstalled ubuntu 14.04
>   ./i386-softmmu/qemu-system-i386 -hda test.img -m 2560 -boot c -machine pc
> 
> As we discussed we need to create a separate machine to support current
> IGD passthrough.
> 
> ----------------------------------------------------------------
> Tiejun Chen (4):
>       i440fx: make types configurable at run-time
>       hw:i386:pc_piix: split pc_init1()
>       xen:hw:pci-host:piix: create host bridge to passthrough
>       xen:hw:i386:pc_piix: introduce new machine for IGD passthrough
> 
>  hw/i386/pc_piix.c    | 227 
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------
>  hw/pci-host/piix.c   |  50 ++++++++++++++++++++++++++++++++++----
>  include/hw/i386/pc.h |   8 ++++++-
>  3 files changed, 254 insertions(+), 31 deletions(-)
> 
> Thanks
> Tiejun