From: Anthony Liguori <anthony@codemonkey.ws>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Avi Kivity <avi@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] vga-pci and MMIO BAR
Date: Wed, 25 Jul 2012 20:08:55 -0500 [thread overview]
Message-ID: <87boj3wcpj.fsf@codemonkey.ws> (raw)
In-Reply-To: <1343262144.6986.0.camel@pasglop>
Benjamin Herrenschmidt <benh@kernel.crashing.org> writes:
> On Thu, 2012-07-26 at 08:01 +1000, Benjamin Herrenschmidt wrote:
>> On Wed, 2012-07-25 at 16:59 +0300, Avi Kivity wrote:
>> > > Why ? Who cares ? Especially with vga -std which nobody uses in
>> > practice
>> > > on x86...
>> >
>> > We'll find out who cares when we break it.
>>
>> Chances are pretty much close to 0, especially adding a BAR ... do we
>> even use vga "std" on anything ?
>>
>> > We have a system in place to prevent intentional breakage, let's use
>> > it.
>>
>> And that system is never improve a device ? :-) Or am I missing
>> something ?
>
> Allright, had a chat with Anthony who explained the bit I was missing
> about what you meant by -M old... I'll make sure I don't break that.
For the benefit of the list, Ben didn't know that we could just add a
qdev property to control whether the MMIO BAR was visible making it just
a tiny bit of global magic in the old PC machine models.
PPC still doesn't do versioning of machine models. Ya'll probably
should spend some time looking at what's done for the PC machine types
and start doing the same for spapr.
Regards,
Anthony Liguori
>
> Cheers,
> Ben
next prev parent reply other threads:[~2012-07-26 1:09 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-25 3:57 [Qemu-devel] vga-pci and MMIO BAR Benjamin Herrenschmidt
2012-07-25 10:27 ` Avi Kivity
2012-07-25 10:53 ` Benjamin Herrenschmidt
2012-07-25 12:48 ` Avi Kivity
2012-07-25 13:41 ` Benjamin Herrenschmidt
2012-07-25 13:59 ` Avi Kivity
2012-07-25 22:01 ` Benjamin Herrenschmidt
2012-07-26 0:22 ` Benjamin Herrenschmidt
2012-07-26 1:08 ` Anthony Liguori [this message]
2012-07-26 1:18 ` Benjamin Herrenschmidt
2012-08-06 12:03 ` Gerd Hoffmann
2012-08-06 21:07 ` Benjamin Herrenschmidt
2012-08-10 15:27 ` Alon Levy
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=87boj3wcpj.fsf@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=avi@redhat.com \
--cc=benh@kernel.crashing.org \
--cc=qemu-devel@nongnu.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).