qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: "Chen, Tiejun" <tiejun.chen@intel.com>
Cc: "peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"stefano.stabellini@eu.citrix.com"
	<stefano.stabellini@eu.citrix.com>,
	"mst@redhat.com" <mst@redhat.com>,
	"Kay, Allen M" <allen.m.kay@intel.com>,
	"Kelly.Zytaruk@amd.com" <Kelly.Zytaruk@amd.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"Zhang, Yang Z" <yang.z.zhang@intel.com>,
	"anthony@codemonkey.ws" <anthony@codemonkey.ws>,
	Anthony PERARD <anthony.perard@citrix.com>
Subject: Re: [Qemu-devel] [v2][PATCH 4/8] xen, gfx passthrough: reserve 00:02.0 for INTEL IGD
Date: Thu, 22 May 2014 07:39:36 +0200	[thread overview]
Message-ID: <1400737176.25874.10.camel@nilsson.home.kraxel.org> (raw)
In-Reply-To: <CCCF29FA0F52DC4B8E2D1EF38AE07FF7B3A244@SHSMSX101.ccr.corp.intel.com>

  Hi,

> > According to our discussions, I realize we may have some plans or policies
> > dedicated to how to assign devfn, but to support GFX passthrough for XEN, I
> > think currently it may be a better solution to adopt #1 simply like this:
> > 
> > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c index eaf3e61..500b3c2
> > 100644
> > --- a/hw/i386/pc_piix.c
> > +++ b/hw/i386/pc_piix.c
> > @@ -386,7 +386,7 @@ static void pc_xen_hvm_init(QEMUMachineInitArgs
> > *args)
> > 
> >      bus = pci_find_primary_bus();
> >      if (bus != NULL) {
> > -        pci_create_simple(bus, -1, "xen-platform");
> > +        pci_create_simple(bus, PCI_DEVFN(3,0), "xen-platform");
> >      }
> >  }
> >  #endif
> > 
> > Then we can go out to plan how to assign devfn in common, is this fine?

Somewhere else in the thread someone listed a libxl config file snippet
which is supposed to disable the xen platform device.  Which doesn't
work with upstream qemu for some reason.

Searching ...   ah, here:
http://lists.gnu.org/archive/html/qemu-devel/2014-05/msg03996.html

So, how about making "xen_platform_pci=0" working correctly ?

Another useful thing would be to not create the xen platform device in
case "-nodefaults" was specified on the command line (that switch turns
off a bunch of other devices present by default: vga, nic, cdrom, ...).

cheers,
  Gerd

  reply	other threads:[~2014-05-22  5:40 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-16 10:53 [Qemu-devel] [v2][PATCH 0/8] xen: add Intel IGD passthrough support Tiejun Chen
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 1/8] pci: use bitmap to manage registe/runregister pci device Tiejun Chen
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 2/8] pci: provide a way to reserve some specific devfn Tiejun Chen
2014-05-16 14:07   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19  9:43     ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 3/8] xen, gfx passthrough: basic graphics passthrough support Tiejun Chen
2014-05-16 14:06   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19  9:42     ` Chen, Tiejun
2014-05-19 13:35       ` Konrad Rzeszutek Wilk
2014-05-20  9:32         ` Chen, Tiejun
2014-05-19 12:10     ` Stefano Stabellini
2014-05-20  5:09       ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 4/8] xen, gfx passthrough: reserve 00:02.0 for INTEL IGD Tiejun Chen
2014-05-16 14:08   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19  9:54     ` Chen, Tiejun
2014-05-19  6:44   ` [Qemu-devel] " Gerd Hoffmann
2014-05-19  7:48     ` [Qemu-devel] [Xen-devel] " Fabio Fantoni
2014-05-19  8:15       ` Zhang, Yang Z
2014-05-19  9:34       ` Chen, Tiejun
2014-05-19  9:25     ` [Qemu-devel] " Chen, Tiejun
2014-05-19 10:13       ` Michael S. Tsirkin
2014-05-20  9:34         ` Chen, Tiejun
2014-05-20 11:36           ` Michael S. Tsirkin
2014-05-19 11:22       ` Gerd Hoffmann
2014-05-19 12:04         ` Chen, Tiejun
2014-05-19 13:50           ` Gerd Hoffmann
2014-05-19 14:00             ` Daniel P. Berrange
2014-05-21  7:07             ` Chen, Tiejun
2014-05-22  0:31               ` Chen, Tiejun
2014-05-22  5:39                 ` Gerd Hoffmann [this message]
2014-05-22  6:18                   ` Chen, Tiejun
2014-05-22  6:44                     ` Gerd Hoffmann
2014-05-22  6:49                       ` Michael S. Tsirkin
2014-05-22  7:11                       ` Chen, Tiejun
2014-05-22 10:50                       ` Chen, Tiejun
2014-05-22 11:03                         ` Gonglei (Arei)
2014-05-22 11:22                         ` Gerd Hoffmann
2014-05-23  1:07                           ` Chen, Tiejun
2014-05-22 11:25                         ` Michael S. Tsirkin
2014-05-22 14:20                           ` [Qemu-devel] [Xen-devel] " Igor Mammedov
2014-05-23  1:18                             ` Chen, Tiejun
2014-05-23  7:38                               ` Igor Mammedov
2014-05-23 10:52                                 ` Anthony PERARD
2014-05-23 11:40                                   ` Stefano Stabellini
2014-05-23 11:53                                     ` Gerd Hoffmann
2014-05-23 12:06                                   ` Igor Mammedov
2014-05-23 12:16                                   ` Igor Mammedov
2014-05-22  9:58                   ` Konrad Rzeszutek Wilk
2014-05-20 14:45           ` [Qemu-devel] " Anthony PERARD
2014-05-21  1:25             ` Chen, Tiejun
2014-06-25  2:49         ` Chen, Tiejun
2014-06-25 23:04           ` Slutz, Donald Christopher
2014-06-26  2:00             ` Chen, Tiejun
2014-06-26  8:23               ` Chen, Tiejun
2014-06-26 16:58                 ` Don Slutz
2014-06-30  9:29                   ` Gerd Hoffmann
2014-06-30 10:23                     ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 5/8] xen, gfx passthrough: create intel isa bridge Tiejun Chen
2014-05-16 14:11   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19  9:59     ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 6/8] xen, gfx passthrough: support Intel IGD passthrough with VT-D Tiejun Chen
2014-05-16 14:35   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19  0:58     ` Zhang, Yang Z
2014-05-19 13:34       ` Konrad Rzeszutek Wilk
2014-05-20  5:13         ` Chen, Tiejun
2014-05-20 13:39           ` Konrad Rzeszutek Wilk
2014-05-19 10:02     ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 7/8] xen, gfx passthrough: create host bridge to passthrough Tiejun Chen
2014-05-16 14:37   ` [Qemu-devel] [Xen-devel] " Konrad Rzeszutek Wilk
2014-05-19 10:03     ` Chen, Tiejun
2014-05-16 10:53 ` [Qemu-devel] [v2][PATCH 8/8] xen, gfx passthrough: add opregion mapping Tiejun Chen
2014-05-19 11:53   ` Stefano Stabellini
2014-05-20  9:24     ` Chen, Tiejun
2014-05-20 10:50       ` Stefano Stabellini
2014-05-21  0:57         ` 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=1400737176.25874.10.camel@nilsson.home.kraxel.org \
    --to=kraxel@redhat.com \
    --cc=Kelly.Zytaruk@amd.com \
    --cc=allen.m.kay@intel.com \
    --cc=anthony.perard@citrix.com \
    --cc=anthony@codemonkey.ws \
    --cc=mst@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=tiejun.chen@intel.com \
    --cc=xen-devel@lists.xensource.com \
    --cc=yang.z.zhang@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).