From: Anthony Liguori <anthony@codemonkey.ws>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: glommer@redhat.com, qemu-devel@nongnu.org,
Alexander Graf <agraf@suse.de>,
Kevin O'Connor <kevin@koconnor.net>,
Gerd Hoffmann <kraxel@redhat.com>,
Sebastian Herbszt <herbszt@gmx.de>
Subject: Re: Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?)
Date: Tue, 15 Dec 2009 15:57:30 -0600 [thread overview]
Message-ID: <4B28064A.1070501@codemonkey.ws> (raw)
In-Reply-To: <20091215215244.GI26712@redhat.com>
Michael S. Tsirkin wrote:
>> Heh, this is going to be really broken with my patches :-)
>>
>> We're during qemu_ram_alloc() and we currently don't have a means to
>> associate ram with anything meaningful. This means that if you hot plug
>> on two ends in different orders (even with fixed slots), the returned
>> qemu_ram_alloc() pointers will be different for the same device. This
>> means when you did the live migration of the rom contents, you'd get the
>> wrong roms in the wrong places.
>>
>> I think we need to improve how we do qemu_ram_alloc() such that we can
>> associate some meaningful context with each allocated chunk that we can
>> migrate with the chunk of ram.
>>
>> Regards,
>>
>> Anthony Liguori
>>
>
> Hmm. You think all this is 0.12 material?
>
I think it's stable-0.12 material because it's badly broken right now
but it's clearly not a candidate for 0.12.0 as it still doesn't work
reliably.
I'm going to pull in Gerd's fix for 0.12.0.
Regards,
Anthony Liguori
next prev parent reply other threads:[~2009-12-15 21:57 UTC|newest]
Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20091213200259.GB25615@redhat.com>
[not found] ` <4B260683.8000506@redhat.com>
[not found] ` <20091214093414.GA30459@redhat.com>
[not found] ` <4B26090B.8010707@redhat.com>
[not found] ` <20091214094406.GB32140@redhat.com>
[not found] ` <4B261082.4030806@redhat.com>
2009-12-14 10:59 ` [Qemu-devel] Re: qdev property bug? Michael S. Tsirkin
2009-12-14 11:55 ` Alexander Graf
2009-12-14 13:24 ` Michael S. Tsirkin
2009-12-14 13:35 ` Alexander Graf
2009-12-14 14:01 ` Michael S. Tsirkin
2009-12-14 17:00 ` Glauber Costa
2009-12-14 14:11 ` Anthony Liguori
2009-12-14 14:11 ` Michael S. Tsirkin
2009-12-14 14:13 ` Michael S. Tsirkin
2009-12-14 14:25 ` Anthony Liguori
2009-12-14 16:20 ` Michael S. Tsirkin
2009-12-14 19:12 ` Sebastian Herbszt
2009-12-14 19:20 ` Michael S. Tsirkin
2009-12-14 19:38 ` Sebastian Herbszt
2009-12-14 19:33 ` Anthony Liguori
2009-12-14 19:42 ` Michael S. Tsirkin
2009-12-14 19:59 ` Anthony Liguori
2009-12-14 20:20 ` Michael S. Tsirkin
2009-12-14 20:32 ` Gerd Hoffmann
2009-12-14 20:34 ` Michael S. Tsirkin
2009-12-14 20:36 ` Michael S. Tsirkin
2009-12-14 20:44 ` Anthony Liguori
2009-12-14 20:51 ` Michael S. Tsirkin
2009-12-15 2:37 ` Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?) Anthony Liguori
2009-12-15 3:55 ` Kevin O'Connor
2009-12-15 4:34 ` Kevin O'Connor
2009-12-15 13:15 ` Anthony Liguori
2009-12-15 14:20 ` Anthony Liguori
2009-12-15 14:33 ` [SeaBIOS] " Avi Kivity
2009-12-15 17:35 ` Anthony Liguori
2009-12-15 23:54 ` Kevin OConnor
2009-12-16 0:41 ` Anthony Liguori
2009-12-16 4:20 ` Kevin O'Connor
2009-12-15 9:17 ` Gerd Hoffmann
2009-12-15 10:14 ` Gerd Hoffmann
2009-12-15 19:10 ` Anthony Liguori
2009-12-16 12:03 ` Gerd Hoffmann
2009-12-16 12:39 ` Michael S. Tsirkin
2009-12-16 13:42 ` Gerd Hoffmann
2009-12-16 13:52 ` Michael S. Tsirkin
2009-12-16 14:12 ` Kevin O'Connor
2009-12-16 14:14 ` Michael S. Tsirkin
2009-12-16 14:17 ` Gleb Natapov
2009-12-16 14:15 ` Michael S. Tsirkin
2009-12-16 14:18 ` Gleb Natapov
2009-12-16 14:24 ` Michael S. Tsirkin
2009-12-16 14:28 ` Gleb Natapov
2009-12-16 14:28 ` Michael S. Tsirkin
2009-12-15 19:35 ` Anthony Liguori
2009-12-15 21:17 ` Michael S. Tsirkin
2009-12-15 21:41 ` Anthony Liguori
2009-12-16 0:33 ` Chris Wright
2009-12-16 1:32 ` Anthony Liguori
2009-12-16 4:57 ` Kevin O'Connor
2009-12-16 12:31 ` Gerd Hoffmann
2009-12-16 16:22 ` Gerd Hoffmann
2009-12-17 1:24 ` Kevin O'Connor
2009-12-17 9:45 ` Gerd Hoffmann
2009-12-17 9:50 ` Michael S. Tsirkin
2009-12-17 10:40 ` Gerd Hoffmann
2009-12-16 14:41 ` Gerd Hoffmann
2009-12-16 14:41 ` Michael S. Tsirkin
2009-12-16 14:50 ` [SeaBIOS] " Gleb Natapov
2009-12-15 13:16 ` Anthony Liguori
2009-12-15 21:24 ` Sebastian Herbszt
2009-12-15 21:31 ` Anthony Liguori
2009-12-15 21:38 ` Sebastian Herbszt
2009-12-15 22:00 ` Anthony Liguori
2009-12-17 21:14 ` Sebastian Herbszt
2009-12-16 4:29 ` Kevin O'Connor
2009-12-16 12:01 ` Gerd Hoffmann
2009-12-15 19:21 ` Anthony Liguori
2009-12-15 21:19 ` Michael S. Tsirkin
2009-12-15 21:45 ` Anthony Liguori
2009-12-15 21:52 ` Michael S. Tsirkin
2009-12-15 21:57 ` Anthony Liguori [this message]
2009-12-15 21:59 ` Michael S. Tsirkin
2009-12-15 22:26 ` Anthony Liguori
2009-12-15 22:51 ` Sebastian Herbszt
2009-12-16 14:49 ` Gerd Hoffmann
2009-12-17 20:33 ` Sebastian Herbszt
2009-12-15 10:51 ` Michael S. Tsirkin
2009-12-15 19:05 ` Anthony Liguori
2009-12-15 19:06 ` Anthony Liguori
2009-12-14 20:43 ` [Qemu-devel] Re: qdev property bug? Anthony Liguori
2009-12-14 20:50 ` Michael S. Tsirkin
2009-12-14 21:05 ` Anthony Liguori
2009-12-14 21:36 ` Sebastian Herbszt
2009-12-14 21:52 ` Anthony Liguori
2009-12-14 22:14 ` Sebastian Herbszt
2009-12-15 4:23 ` Kevin O'Connor
2009-12-14 20:47 ` Sebastian Herbszt
2009-12-14 20:56 ` Anthony Liguori
2009-12-14 20:38 ` Sebastian Herbszt
2009-12-14 19:51 ` Sebastian Herbszt
2009-12-14 14:22 ` Anthony Liguori
2009-12-14 17:03 ` Glauber Costa
2009-12-14 17:01 ` Glauber Costa
2009-12-14 14:10 ` Anthony Liguori
2009-12-14 14:52 ` Gerd Hoffmann
2009-12-14 14:54 ` Anthony Liguori
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=4B28064A.1070501@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=agraf@suse.de \
--cc=glommer@redhat.com \
--cc=herbszt@gmx.de \
--cc=kevin@koconnor.net \
--cc=kraxel@redhat.com \
--cc=mst@redhat.com \
--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).