Re: [Qemu-devel] [PATCH] pc: limit 64 bit hole to 2G by default

[Gerd Hoffmann]

On 07/24/13 11:51, Michael S. Tsirkin wrote:
> On Wed, Jul 24, 2013 at 09:01:51AM +0200, Gerd Hoffmann wrote:
>> On 07/24/13 08:01, Michael S. Tsirkin wrote:
>>> It turns out that some 32 bit windows guests crash
>>> if 64 bit PCI hole size is >2G.
>>
>> Ah, *that* is the reason for winxp crashing with a 64bit hole.
>>
>> Current seabios uses a slightly different approach: the 64bit hole is
>> present only in case it is actually used to map bars there, and seabios
>> tries to fit everything into the 32bit hole first.
> 
> Yes. But this doesn't work with device hotplug.

Sure.  This wasn't meant as request to match current seabios behavior.
Just wanted provide some additional background info.

> I intend to send a patch to properties that allows writing
> "size=1G" instead.
> Will this address your comment?

Sounds good.

cheers,
  Gerd