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

<ion(a)cvs.reactos.com> wrote:
>   CVSROOT: /CVS/ReactOS
>  Module name: reactos
>  Repository: reactos/ntoskrnl/mm/
>  Changes by: ion(a)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