[ros-dev] Re: [ros-cvs] CVS Update: reactos

[Eric Kohl]

<ion at cvs.reactos.com> wrote:

> CVSROOT: /CVS/ReactOS
> Module name: reactos
> Repository: reactos/ntoskrnl/mm/
> Changes by: ion at mok.osexperts.com 04/10/04 14:38:49
>
> Modified files:
> ./: config rules.mak
> reactos/include/ntdll/: rtl.h
> reactos/ntoskrnl/: Makefile
> reactos/ntoskrnl/include/internal/i386/: mm.h
> reactos/ntoskrnl/ke/i386/: multiboot.S
> reactos/ntoskrnl/mm/: mminit.c
>
> Log message:
> Un-hardcoded the 0xC0000000 value, and stuck the define in one single
> place both for ASM and C code. This define is 0x80000000 by default, but
> automatically sets itself to 0xC0000000 if 3GB is set to 1 in
> reactos/config. This option is turned on by default for now, because the
> Default heap is at 0xA0000000, which doesn't work on a non 3GB system.
> Lower values seem to crash Win32K but I fill figure out a solution.

I tried a NULL base address in the call to RtlCreateHeap (in
lib/ntdll/ldr/startup.c) and regedit (WIN32 app) works very well. The NULL
base address enables the default bottom-up memory allocation so the heap
will be allocated from the bootom of the process address space just like
Windows does. HEAP_BASE should have been removed ages ago.

Regards,
Eric