All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Chen, Tiejun" <tiejun.chen@intel.com>
To: mst@redhat.com
Cc: pbonzini@redhat.com, xen-devel@lists.xensource.com,
	qemu-devel@nongnu.org, stefano.stabellini@eu.citrix.com
Subject: Re: [Qemu-devel] [v5][PATCH 0/4] xen: introduce new machine for IGD passthrough
Date: Thu, 14 Aug 2014 08:59:49 +0800	[thread overview]
Message-ID: <53EC0A05.2090906@intel.com> (raw)
In-Reply-To: <1407836957-29098-1-git-send-email-tiejun.chen@intel.com>

Michael,

Any further comments to this revision?

Thanks
Tiejun

On 2014/8/12 17:49, Tiejun Chen wrote:
> v5:
>
> * Simplify to make sure its really inherited from the standard one in patch #3
> * Then drop the original patch #3
>
> v4:
>
> * Rebase on latest tree
> * Drop patch #2
> * Regenerate patches after Michael introduce patch #1
> * We need to use this pci_type as a index to reuse I440FX_PCI_DEVICE()
> * Test: boot with a preinstalled winxp
>    ./i386-softmmu/qemu-system-i386 -hda winxp-32.img -m 2560 -boot c -machine pc
>
> v3:
>
> * Drop patch #4
> * Add one patch #1 from Michael
> * Rebase
> * In./i386-softmmu/qemu-system-i386 -hda test.img -m 2560 -boot c -machine pc
>
> 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.
>
> ----------------------------------------------------------------
> Michael S. Tsirkin (1):
>        i440fx: make types configurable at run-time
>
> Tiejun Chen (3):
>        pc_init1: pass parameters just with types
>        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    | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-----
>   hw/pci-host/piix.c   | 24 +++++++++++++++++++-----
>   include/hw/i386/pc.h |  8 +++++++-
>   3 files changed, 81 insertions(+), 11 deletions(-)
>
> Thanks
> Tiejun
>
>
>

WARNING: multiple messages have this Message-ID (diff)
From: "Chen, Tiejun" <tiejun.chen@intel.com>
To: mst@redhat.com
Cc: pbonzini@redhat.com, xen-devel@lists.xensource.com,
	qemu-devel@nongnu.org, stefano.stabellini@eu.citrix.com
Subject: Re: [v5][PATCH 0/4] xen: introduce new machine for IGD passthrough
Date: Thu, 14 Aug 2014 08:59:49 +0800	[thread overview]
Message-ID: <53EC0A05.2090906@intel.com> (raw)
In-Reply-To: <1407836957-29098-1-git-send-email-tiejun.chen@intel.com>

Michael,

Any further comments to this revision?

Thanks
Tiejun

On 2014/8/12 17:49, Tiejun Chen wrote:
> v5:
>
> * Simplify to make sure its really inherited from the standard one in patch #3
> * Then drop the original patch #3
>
> v4:
>
> * Rebase on latest tree
> * Drop patch #2
> * Regenerate patches after Michael introduce patch #1
> * We need to use this pci_type as a index to reuse I440FX_PCI_DEVICE()
> * Test: boot with a preinstalled winxp
>    ./i386-softmmu/qemu-system-i386 -hda winxp-32.img -m 2560 -boot c -machine pc
>
> v3:
>
> * Drop patch #4
> * Add one patch #1 from Michael
> * Rebase
> * In./i386-softmmu/qemu-system-i386 -hda test.img -m 2560 -boot c -machine pc
>
> 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.
>
> ----------------------------------------------------------------
> Michael S. Tsirkin (1):
>        i440fx: make types configurable at run-time
>
> Tiejun Chen (3):
>        pc_init1: pass parameters just with types
>        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    | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-----
>   hw/pci-host/piix.c   | 24 +++++++++++++++++++-----
>   include/hw/i386/pc.h |  8 +++++++-
>   3 files changed, 81 insertions(+), 11 deletions(-)
>
> Thanks
> Tiejun
>
>
>

  parent reply	other threads:[~2014-08-14  1:00 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-12  9:49 [Qemu-devel] [v5][PATCH 0/4] xen: introduce new machine for IGD passthrough Tiejun Chen
2014-08-12  9:49 ` Tiejun Chen
2014-08-12  9:49 ` [Qemu-devel] [v5][PATCH 1/4] i440fx: make types configurable at run-time Tiejun Chen
2014-08-12  9:49   ` Tiejun Chen
2014-08-12  9:49 ` [Qemu-devel] [v5][PATCH 2/4] pc_init1: pass parameters just with types Tiejun Chen
2014-08-12  9:49   ` Tiejun Chen
2014-08-12  9:49 ` [Qemu-devel] [v5][PATCH 3/4] xen:hw:pci-host:piix: create host bridge to passthrough Tiejun Chen
2014-08-12  9:49   ` Tiejun Chen
2014-08-12  9:49 ` [Qemu-devel] [v5][PATCH 4/4] xen:hw:i386:pc_piix: introduce new machine for IGD passthrough Tiejun Chen
2014-08-12  9:49   ` Tiejun Chen
2014-08-14 16:03   ` [Qemu-devel] " Michael S. Tsirkin
2014-08-14 16:03     ` Michael S. Tsirkin
2014-08-15  5:46     ` [Qemu-devel] " Chen, Tiejun
2014-08-15  5:46       ` Chen, Tiejun
2014-08-14  0:59 ` Chen, Tiejun [this message]
2014-08-14  0:59   ` [v5][PATCH 0/4] xen: " Chen, Tiejun
2014-08-14  7:09 ` [Qemu-devel] " Michael S. Tsirkin
2014-08-14  7:09   ` Michael S. Tsirkin
2014-08-14  7:23   ` [Qemu-devel] " Chen, Tiejun
2014-08-14  7:23     ` Chen, Tiejun
2014-11-23 10:10 ` [Qemu-devel] " Michael S. Tsirkin
2014-11-23 10:10   ` Michael S. Tsirkin
2014-11-24  0:40   ` [Qemu-devel] " Chen, Tiejun
2014-11-24  0:40     ` Chen, Tiejun

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53EC0A05.2090906@intel.com \
    --to=tiejun.chen@intel.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.