qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fabio Fantoni <fabio.fantoni@m2r.biz>
To: Gerd Hoffmann <kraxel@redhat.com>, Tiejun Chen <tiejun.chen@intel.com>
Cc: peter.maydell@linaro.org, xen-devel@lists.xensource.com,
	stefano.stabellini@eu.citrix.com, qemu-devel@nongnu.org,
	weidong.han@intel.com, mst@redhat.com, allen.m.kay@intel.com,
	Kelly.Zytaruk@amd.com, jean.guyader@eu.citrix.com,
	anthony.perard@citrix.com, anthony@codemonkey.ws,
	yang.z.zhang@intel.com
Subject: Re: [Qemu-devel] [Xen-devel]  [v2][PATCH 4/8] xen, gfx passthrough: reserve 00:02.0 for INTEL IGD
Date: Mon, 19 May 2014 09:48:02 +0200	[thread overview]
Message-ID: <5379B732.80806@m2r.biz> (raw)
In-Reply-To: <1400481887.32155.34.camel@nilsson.home.kraxel.org>

Il 19/05/2014 08:44, Gerd Hoffmann ha scritto:
>    Hi,
>
>> +    /*
>> +     * Some video bioses and gfx drivers will assume the bdf of IGD is 00:02.0.
>> +     * So user need to set it to 00:02.0 in Xen configure file explicitly,
>> +     * otherwise IGD will fail to work.
>> +     */
>> +    pci_reserve_pci_devfn(b, PCI_DEVFN(2, 0));
> That is asking for trouble.  Slot 2 is used by the qemu vga cards by
> default, and for quite a while (before memory api was merged) it even
> was impossible to change it.  libvirt still places the vga card at slot
> 2 for that reason -> boom.  I wouldn't be surprised if you find that
> assumption in other management libs / apps too.
>
> Why do you need that patch in the first place?  It should be possible to
> configure qemu to not occupy slot 2 if you need it that way.  Just pass
> '-vga none' to qemu.  Which you probably want anyway if you pass-through
> a vga to the guest.  And explicitly configure a slot (via addr=
> property) for all your pci devices.  Doing it only for the IGD works too
> if you list the device before any other pci device on the qemu command
> line.
>
> cheers,
>    Gerd

I already added vga none support on libxl, useful also for this cases:
http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=2e5738ff47b9d2e1948024100f87b1a25fcf004a

This patch is already tested and working, for now I not tested and I 
can't test with intel gpu passthrough, someone can test it with intel 
gpu passthrough?

Thanks for any reply and sorry for my bad english.

>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

  reply	other threads:[~2014-05-19  7:48 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     ` Fabio Fantoni [this message]
2014-05-19  8:15       ` [Qemu-devel] [Xen-devel] " 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
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=5379B732.80806@m2r.biz \
    --to=fabio.fantoni@m2r.biz \
    --cc=Kelly.Zytaruk@amd.com \
    --cc=allen.m.kay@intel.com \
    --cc=anthony.perard@citrix.com \
    --cc=anthony@codemonkey.ws \
    --cc=jean.guyader@eu.citrix.com \
    --cc=kraxel@redhat.com \
    --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=weidong.han@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).