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

2013-07-25 Thread Michael S. Tsirkin
ongnu.org, "Isaku > > Yamahata" , "Alex Williamson" > > , "Gerd Hoffmann" > > , "Laszlo Ersek" , "Andreas Färber" > > , "David Gibson" > > , "Paolo Bonzini" > > Sent: Thu

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

2013-07-25 Thread Igor Mammedov
fmann" > , "Laszlo Ersek" , "Andreas Färber" > , "David Gibson" > , "Paolo Bonzini" > Sent: Thursday, July 25, 2013 5:23:21 PM > Subject: Re: [Qemu-devel] [PATCH] pc: limit 64 bit hole to 2G by default > > On Thu, Jul 25, 2013 at 11

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

2013-07-25 Thread Michael S. Tsirkin
ongnu.org, "Isaku > > Yamahata" , "Alex Williamson" > > , "Gerd Hoffmann" > > , "Paolo Bonzini" , "Laszlo Ersek" > > , "Andreas Färber" > > , "David Gibson" > > Sent: Thursday, July 25, 2013

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

2013-07-25 Thread Igor Mammedov
fmann" > , "Paolo Bonzini" , "Laszlo Ersek" > , "Andreas Färber" > , "David Gibson" > Sent: Thursday, July 25, 2013 3:40:05 PM > Subject: Re: [Qemu-devel] [PATCH] pc: limit 64 bit hole to 2G by default > > On Wed, 24 Jul 2013 09:01:04

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

2013-07-25 Thread Michael S. Tsirkin
On Thu, Jul 25, 2013 at 03:40:05PM +0200, Igor Mammedov wrote: > On Wed, 24 Jul 2013 09:01:04 +0300 > "Michael S. Tsirkin" wrote: > > > It turns out that some 32 bit windows guests crash > > if 64 bit PCI hole size is >2G. > > Limit it to 2G for piix and q35 by default, > > add properties to let

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

2013-07-25 Thread Igor Mammedov
On Wed, 24 Jul 2013 09:01:04 +0300 "Michael S. Tsirkin" wrote: > It turns out that some 32 bit windows guests crash > if 64 bit PCI hole size is >2G. > Limit it to 2G for piix and q35 by default, > add properties to let management override the hole size. > > Examples: > -global i440FX-pcihost.pc

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

2013-07-24 Thread Michael S. Tsirkin
On Wed, Jul 24, 2013 at 03:18:01PM +0200, Paolo Bonzini wrote: > Il 24/07/2013 15:14, Michael S. Tsirkin ha scritto: > > On Wed, Jul 24, 2013 at 02:59:18PM +0200, Paolo Bonzini wrote: > >> Il 24/07/2013 11:51, Michael S. Tsirkin ha scritto: > Current seabios uses a slightly different approach:

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

2013-07-24 Thread Paolo Bonzini
Il 24/07/2013 15:14, Michael S. Tsirkin ha scritto: > On Wed, Jul 24, 2013 at 02:59:18PM +0200, Paolo Bonzini wrote: >> Il 24/07/2013 11:51, Michael S. Tsirkin ha scritto: Current seabios uses a slightly different approach: the 64bit hole is present only in case it is actually used to map

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

2013-07-24 Thread Michael S. Tsirkin
On Wed, Jul 24, 2013 at 02:59:18PM +0200, Paolo Bonzini wrote: > Il 24/07/2013 11:51, Michael S. Tsirkin ha scritto: > > > 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 everythin

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

2013-07-24 Thread 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

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

2013-07-24 Thread Paolo Bonzini
Il 24/07/2013 11:51, Michael S. Tsirkin ha scritto: > > 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 devic

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

2013-07-24 Thread Igor Mammedov
On Wed, 24 Jul 2013 12:51:16 +0300 "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 th

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

2013-07-24 Thread Michael S. Tsirkin
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 sl

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

2013-07-24 Thread Gerd Hoffmann
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 actua

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

2013-07-23 Thread Andreas Färber
Hi, Am 24.07.2013 08:01, schrieb Michael S. Tsirkin: > It turns out that some 32 bit windows guests crash > if 64 bit PCI hole size is >2G. > Limit it to 2G for piix and q35 by default, > add properties to let management override the hole size. > > Examples: > -global i440FX-pcihost.pci_hole64_si

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

2013-07-23 Thread Michael S. Tsirkin
It turns out that some 32 bit windows guests crash if 64 bit PCI hole size is >2G. Limit it to 2G for piix and q35 by default, add properties to let management override the hole size. Examples: -global i440FX-pcihost.pci_hole64_size=137438953472 -global q35-pcihost.pci_hole64_size=137438953472 R