From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:51793)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1SxKkT-0003kQ-OK
	for qemu-devel@nongnu.org; Fri, 03 Aug 2012 12:29:14 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1SxKkS-0006HS-Iw
	for qemu-devel@nongnu.org; Fri, 03 Aug 2012 12:29:13 -0400
Received: from mail-yw0-f45.google.com ([209.85.213.45]:50613)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1SxKkS-0006HO-Eb
	for qemu-devel@nongnu.org; Fri, 03 Aug 2012 12:29:12 -0400
Received: by yhpp34 with SMTP id p34so1025750yhp.4
	for <qemu-devel@nongnu.org>; Fri, 03 Aug 2012 09:29:12 -0700 (PDT)
Sender: Paolo Bonzini <paolo.bonzini@gmail.com>
Message-ID: <501BFC51.5020005@redhat.com>
Date: Fri, 03 Aug 2012 18:29:05 +0200
From: Paolo Bonzini <pbonzini@redhat.com>
MIME-Version: 1.0
References: <D0B75B542AA3C648B838B5A90BE651C72CA7FEE1@nzhmlmbx01.ad.endace.com>
In-Reply-To: <D0B75B542AA3C648B838B5A90BE651C72CA7FEE1@nzhmlmbx01.ad.endace.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [BUG] BSOD on Win2003 Server when 64bit PCI
	resource is present
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexey Korolev <Alexey.Korolev@endace.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>, "kevin@koconnor.net" <kevin@koconnor.net>, "seabios@seabios.org" <seabios@seabios.org>, Gerd Hoffmann <kraxel@redhat.com>, "mst@redhat.com" <mst@redhat.com>

Il 26/07/2012 17:38, Alexey Korolev ha scritto:
> HI,
> 
> Current version of Seabios is causing blue screen on Windows2003 when 64bit PCI resource is present and occupies high memory.
> 
> BSOD Error code is: 0x000000A5 (0x02, 0xFFFFfADF6A446880, 0x1, 0xFFFFFADFAA34690)
> 
> The issue is localized, it is related to presence of 64bit resource in _CRS method.
> 
> If we disable a 64bit region from _CRS  the Win2003 load normally but this doesn't allow Windows to use 64bit resources.
> 
> At the moment I have no idea how to fix this. Please help! 

I wonder if Windows 2003 does not support ConcatenateResTemplate.  The patch below
is obviously wrong, but should let you check if this is the issue:

git diff src/acpi-dsdt.dsl
diff --git a/src/acpi-dsdt.dsl b/src/acpi-dsdt.dsl
index 72dc7d8..11a9c92 100644
--- a/src/acpi-dsdt.dsl
+++ b/src/acpi-dsdt.dsl
@@ -175,9 +175,6 @@ DefinitionBlock (
                     0x00000000,         // Address Translation Offset
                     0x1EC00000,         // Address Length
                     ,, PW32, AddressRangeMemory, TypeStatic)
-            })
-            Name (CR64, ResourceTemplate ()
-            {
                 QWordMemory (ResourceProducer, PosDecode, MinFixed, MaxFixed, Cacheable, ReadWrite,
                     0x00000000,          // Address Space Granularity
                     0x8000000000,        // Address Range Minimum
@@ -221,20 +218,14 @@ DefinitionBlock (
 		Store (P0EL, PE32)
 		Store (P0LL, PL32)
 
-		If (LAnd(LEqual(P1SL, 0x00), LEqual(P1SH, 0x00))) {
-		    Return (CRES)
-		} Else {
 		    /* fixup 64bit pci io window */
-		    CreateQWordField (CR64,\_SB.PCI0.PW64._MIN, PS64)
-		    CreateQWordField (CR64,\_SB.PCI0.PW64._MAX, PE64)
-		    CreateQWordField (CR64,\_SB.PCI0.PW64._LEN, PL64)
+		    CreateQWordField (CRES,\_SB.PCI0.PW64._MIN, PS64)
+		    CreateQWordField (CRES,\_SB.PCI0.PW64._MAX, PE64)
+		    CreateQWordField (CRES,\_SB.PCI0.PW64._LEN, PL64)
 		    Store (P1S, PS64)
 		    Store (P1E, PE64)
 		    Store (P1L, PL64)
-		    /* add window and return result */
-		    ConcatenateResTemplate (CRES, CR64, Local0)
-		    Return (Local0)
-		}
+		    Return (CRES)
             }
         }
     }

Paolo