From: Kevin O'Connor <kevin@koconnor.net>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: seabios@seabios.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [SeaBIOS] [seabios patch 1/5] pci: init all devices
Date: Fri, 4 May 2012 09:15:46 -0400 [thread overview]
Message-ID: <20120504131546.GA668@morn.localdomain> (raw)
In-Reply-To: <1336119687-6295-2-git-send-email-kraxel@redhat.com>
On Fri, May 04, 2012 at 10:21:23AM +0200, Gerd Hoffmann wrote:
> seabios used to initialize root bus devices only, with this patch
> devices behind pci bridges are initialized too. This allows to boot
> from virtio devices behind pci bridges.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
> src/pciinit.c | 3 ---
> 1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/src/pciinit.c b/src/pciinit.c
> index 25b04ac..6a7a0d2 100644
> --- a/src/pciinit.c
> +++ b/src/pciinit.c
> @@ -213,9 +213,6 @@ static void pci_bios_init_devices(void)
> {
> struct pci_device *pci;
> foreachpci(pci) {
> - if (pci_bdf_to_bus(pci->bdf) != 0)
> - // Only init devices on host bus.
> - break;
I think this will then assign an incorrect PCI_INTERRUPT_LINE value to
non root bus devices.
-Kevin
next prev parent reply other threads:[~2012-05-04 13:15 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-04 8:21 [Qemu-devel] [seabios patch 0/5] dynamic pci i/o windows Gerd Hoffmann
2012-05-04 8:21 ` [Qemu-devel] [seabios patch 1/5] pci: init all devices Gerd Hoffmann
2012-05-04 13:15 ` Kevin O'Connor [this message]
2012-05-04 15:35 ` [Qemu-devel] [SeaBIOS] " Gerd Hoffmann
2012-05-05 0:10 ` Kevin O'Connor
2012-05-12 18:16 ` Kevin O'Connor
2012-05-04 8:21 ` [Qemu-devel] [seabios patch 2/5] acpi: add qemu fwcfg driver Gerd Hoffmann
2012-05-04 8:21 ` [Qemu-devel] [seabios patch 3/5] acpi: update pci io windows according to fw_cfg info Gerd Hoffmann
2012-05-04 8:21 ` [Qemu-devel] [seabios patch 4/5] pciinit: make pci ressources configurable Gerd Hoffmann
2012-05-04 8:21 ` [Qemu-devel] [seabios patch 5/5] update src/acpi-dsdt.hex Gerd Hoffmann
2012-05-04 9:04 ` [Qemu-devel] [SeaBIOS] " Gerd Hoffmann
2012-05-04 9:24 ` Michael Tokarev
2012-05-04 13:18 ` [Qemu-devel] [SeaBIOS] [seabios patch 0/5] dynamic pci i/o windows Kevin O'Connor
2012-05-04 14:01 ` Gerd Hoffmann
2012-05-04 14:46 ` Kevin O'Connor
2012-05-04 15:37 ` Kevin O'Connor
2012-05-06 8:50 ` Gleb Natapov
2012-05-07 1:58 ` Alexey Korolev
2012-05-07 2:43 ` Kevin O'Connor
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=20120504131546.GA668@morn.localdomain \
--to=kevin@koconnor.net \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=seabios@seabios.org \
/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).