qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Apfelbaum <marcel@redhat.com>
To: Gerd Hoffmann <kraxel@gmail.com>, Kevin O'Connor <kevin@koconnor.net>
Cc: seabios@seabios.org, lersek@redhat.com, qemu-devel@nongnu.org,
	mst@redhat.com
Subject: Re: [Qemu-devel] [PATCH V2] pci: fixes to allow booting from extra root pci buses.
Date: Mon, 15 Jun 2015 12:02:23 +0300	[thread overview]
Message-ID: <557E949F.8070002@redhat.com> (raw)
In-Reply-To: <1434351024.31654.25.camel@gmail.com>

On 06/15/2015 09:50 AM, Gerd Hoffmann wrote:
>    Hi,
>
>> I'm wondering whenever things become easier if we add config registers
>> to the pxb, where the firmware can program the bus number range and we
>> can use the config register base as a way to specify which pxb we are
>> referring to ?
>
> ... and, while thinking about ben's reply elsewhere in this thread,
> maybe even decouple the whole thing from the primary root bus?  You
> can't program the devices via 0x0cf8 then, but we could add a mmconfig
> bar to the host bridge device ...

Hi Gerd,

I thought about it of course, but it seemed to me to be over-kill and
no real reason to do it. I would hove done it if:
- there would be a "generic" spec for such a host-bridge specifying
   at least the registers for the bus number. I didn't find anything =>
   => Seabios should be aware of a special QEMU device and look for it... ugly
- I would have seen how Seabios/coreboot program the above bus number.
   Again, didn't find the host-bridge programming code.
At last, I followed a 'real' PXB device, some old Intel snooping host bridge.

As always, I am open to ideas, but, the latest patches from Laszlo follows
Seabios way, and if OVMF can also handle it maybe we can stop:
"The use of logical bus number instead of bus index" discussion and
start arguing on something else. :)

I just want to say thank you to everybody involved,
a lot of information and good guidelines surfaced during this talk.

Until the next patch...
Thanks,
Marcel

>
> cheers,
>    Gerd
>
>

  reply	other threads:[~2015-06-15  9:02 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-11 13:37 [Qemu-devel] [PATCH V2] pci: fixes to allow booting from extra root pci buses Marcel Apfelbaum
2015-06-11 13:57 ` Laszlo Ersek
2015-06-11 13:58 ` Kevin O'Connor
2015-06-11 14:12   ` Marcel Apfelbaum
2015-06-11 14:24     ` Kevin O'Connor
2015-06-11 14:36       ` Marcel Apfelbaum
2015-06-11 15:00         ` Laszlo Ersek
2015-06-11 16:54         ` Kevin O'Connor
2015-06-11 17:46           ` Marcel Apfelbaum
2015-06-11 18:34             ` Laszlo Ersek
2015-06-11 19:24               ` Kevin O'Connor
2015-06-12  9:25                 ` Laszlo Ersek
2015-06-12 13:03                   ` Kevin O'Connor
2015-06-12 15:45                     ` Laszlo Ersek
2015-06-12 18:40                       ` Kevin O'Connor
2015-06-12 20:13                         ` Laszlo Ersek
2015-06-14 12:05                         ` Michael S. Tsirkin
2015-06-14 14:50                           ` Kevin O'Connor
2015-06-14 18:06                             ` Michael S. Tsirkin
2015-06-14 18:21                               ` Kevin O'Connor
2015-06-14 21:39                               ` Benjamin Herrenschmidt
2015-06-14 21:59                                 ` Kevin O'Connor
2015-06-15  2:50                                   ` Benjamin Herrenschmidt
2015-06-15  8:22                                     ` Michael S. Tsirkin
2015-06-11 19:10             ` Kevin O'Connor
2015-06-12  6:00               ` Gerd Hoffmann
2015-06-12 12:17                 ` Marcel Apfelbaum
2015-06-12 13:23                   ` Kevin O'Connor
2015-06-15  6:01                     ` Gerd Hoffmann
2015-06-15  6:50                       ` Gerd Hoffmann
2015-06-15  9:02                         ` Marcel Apfelbaum [this message]
2015-06-15  9:43                       ` Michael S. Tsirkin
2015-06-15 10:18                         ` Gerd Hoffmann
2015-06-15 10:26                           ` Michael S. Tsirkin
2015-06-11 14:35   ` Laszlo Ersek
2015-06-11 16:48     ` Kevin O'Connor
2015-06-11 18:38       ` [Qemu-devel] [SeaBIOS] " Laszlo Ersek
2015-06-14 12:10       ` [Qemu-devel] " Michael S. Tsirkin
2015-06-14 13:47         ` 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=557E949F.8070002@redhat.com \
    --to=marcel@redhat.com \
    --cc=kevin@koconnor.net \
    --cc=kraxel@gmail.com \
    --cc=lersek@redhat.com \
    --cc=mst@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).