From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33233) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UINkQ-0001fM-1n for qemu-devel@nongnu.org; Wed, 20 Mar 2013 14:28:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UINcL-0002to-Dz for qemu-devel@nongnu.org; Wed, 20 Mar 2013 14:20:09 -0400 Received: from mx1.redhat.com ([209.132.183.28]:6182) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UINaw-0001yy-DC for qemu-devel@nongnu.org; Wed, 20 Mar 2013 14:18:38 -0400 Message-ID: <1363803514.24132.520.camel@bling.home> From: Alex Williamson Date: Wed, 20 Mar 2013 12:18:34 -0600 In-Reply-To: <1363802711.26710.3.camel@mattotaupa> References: <20130320165728.22281.10294.stgit@bling.home> <1363802711.26710.3.camel@mattotaupa> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [SeaBIOS] [PATCH v3] pciinit: Enable default VGA device List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paul Menzel Cc: kevin@koconnor.net, seabios@seabios.org, qemu-devel@nongnu.org, kraxel@redhat.com On Wed, 2013-03-20 at 19:05 +0100, Paul Menzel wrote: > Dear Alex, >=20 >=20 > Am Mittwoch, den 20.03.2013, 10:58 -0600 schrieb Alex Williamson: > > As QEMU gains PCI bridge and PCIe root port support, >=20 > could you give a commit or version for QEMU please. This would be targeted towards QEMU 1.5. Michael recently sent a pull request with this patch to enable VGA routing in bridges: https://git.kernel.org/cgit/virt/kvm/mst/qemu.git/commit/?h=3Dpci&id=3De4= 75aeac3e2e7f6632c3ea7e83f4431c0ba3a467 It's difficult to make use of without an assigned VGA device, which I plan to start posting soon. > > we won't always find the VGA device on the root bus. We therefore > > need to add support to find and enable a VGA device and the path to i= t > > through the VGA Enable support in the PCI bridge control register. >=20 > Just to be sure, did you test this with older QEMU too? Yes, it works with the QEMU 1.2.x shipped in F18. Any existing QEMU configuration is going to have the VGA device on the host bridge and therefore bail out on the first loop, having done nothing other than add a debug print. > > Signed-off-by: Alex Williamson > > --- > >=20 > > v3: use pci_config_maskw() to trim out some code > > v2: move to qemu specific pciinit.c > >=20 > > src/optionroms.c | 2 +- > > src/pciinit.c | 40 ++++++++++++++++++++++++++++++++++++++++ > > src/util.h | 1 + > > 3 files changed, 42 insertions(+), 1 deletion(-) > >=20 > > diff --git a/src/optionroms.c b/src/optionroms.c > > index caa2151..ac92613 100644 > > --- a/src/optionroms.c > > +++ b/src/optionroms.c > > @@ -213,7 +213,7 @@ run_file_roms(const char *prefix, int isvga, u64 = *sources) > > ****************************************************************/ > > =20 > > // Verify device is a vga device with legacy address decoding enable= d. > > -static int > > +int > > is_pci_vga(struct pci_device *pci) > > { > > if (pci->class !=3D PCI_CLASS_DISPLAY_VGA) > > diff --git a/src/pciinit.c b/src/pciinit.c > > index ce0a4cc..bb9355f 100644 > > --- a/src/pciinit.c > > +++ b/src/pciinit.c > > @@ -316,6 +316,44 @@ static void pci_bios_init_devices(void) > > } > > } > > =20 > > +static void pci_enable_default_vga(void) > > +{ > > + struct pci_device *pci; > > + > > + foreachpci(pci) { > > + if (is_pci_vga(pci)) { > > + dprintf(1, "PCI: Using %02x:%02x.%x for primary VGA\n", > > + pci_bdf_to_bus(pci->bdf), pci_bdf_to_dev(pci->bd= f), > > + pci_bdf_to_fn(pci->bdf)); >=20 > As this is used several times, a function returning a string with %02x:= % > 02x.%x would be handy. >=20 > > + return; > > + } > > + } >=20 > [=E2=80=A6] >=20 > Acked-by: Paul Menzel Thanks! Alex