Re: [SeaBIOS] [PATCH v3] DSDT: Fix HPET _CRS Method

[Rene Rebe]

Hi Gabriel,

On 08.11.2012, at 18:35, Gabriel L. Somlo wrote:
> Updated _CRS method for HPET, bringing it in line with the way it is
> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
> Allows it to be detected and utilized from Mac OS X; Also tested OK on
> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
> Signed-off-by: Gabriel Somlo
>
> On Thu, Nov 08, 2012 at 04:12:07PM +0100, Gerd Hoffmann wrote:
> > Looks good to me now.
>
> One more thing, the "IRQNoFlags" line (also present on the real
> hardware I used as examples) makes it possible to start OS X with
> multiple virtual CPUs in qemu (e.g. -smp 4,cores=2). Re-ran tests
> on Fedora16 and Windows7 successfully (also with smp enabled).

What did the Darwin kernel do before with -smp > 1?

I ask, because for me it booted already, just was not stable.
The question is: Is OS X stable for you with -smp > 1 under load?

René
> Thanks,
> --Gabriel
>
> src/acpi-dsdt.dsl |   14 +++++---------
> 1 files changed, 5 insertions(+), 9 deletions(-)
>
> diff --git a/src/acpi-dsdt.dsl b/src/acpi-dsdt.dsl
> index 284d70d..711302e 100644
> --- a/src/acpi-dsdt.dsl
> +++ b/src/acpi-dsdt.dsl
> @@ -269,15 +269,11 @@ DefinitionBlock (
>                     Return (0x0F)
>             }
>             Name(_CRS, ResourceTemplate() {
>            DWordMemory(
>                ResourceConsumer, PosDecode, MinFixed, MaxFixed,
>                NonCacheable, ReadWrite,
>                0x00000000,
>                0xFED00000,
>                0xFED003FF,
>                0x00000000,
>                0x00000400 /* 1K memory: FED00000 - FED003FF */
>            )
>            IRQNoFlags () {2, 8}
>            Memory32Fixed (ReadOnly,
>                0xFED00000,         // Address Base
>                0x00000400,         // Address Length
>                )
>       })
>   }
> }