Moving ntoskrnl configuration defines into ntoskrnl/include/config.h ?

[Gregor Anich]

Hi!
I have written a tool to configure the ReactOS source tree (customize
the build) It builds a GUI interface from an XML file which lists all
the possible options and how they are "realized" (in which file a value
has to be changed to make the build-system do what is wanted, i.e. set
KDBG in config to 1)
I would like to add the possibility of configuring things like
WHOLE_PAGE_ALLOCATIONS or TAG_STATISTICS_TRACKING (from ntoskrnl/mm/npool.c)
It would be good if such defines were moved into a file like
ntoskrnl/include/config.h which is then included by ntoskrnl.h and looks
like this:
/* Enable tracking of statistics about the tagged blocks in the pool */
#undef TAG_STATISTICS_TRACKING
/*
 * Put each block in its own range of pages and position the block at the
 * end of the range so any accesses beyond the end of block are to invalid
 * memory locations.
 */
#undef WHOLE_PAGE_ALLOCATIONS
I don't want to make the tool edit the C files directly.
If anybody is against this please let me know, otherwise I will do it.
(Note: The tool is written in python and will thus not be allowed into
the reactos tree but I will provide it for whomever is interested)
Thanks,
  blight

[Steven Edwards]

Hi Blight,
--- Gregor Anich <blight(a)blight.eu.org> wrote:
>   If anybody is against this please let me know,
> otherwise I will do
>  it.
>  (Note: The tool is written in python and will thus not be allowed
>  into
>  the reactos tree but I will provide it for whomever is interested) 
I like the idea of moving all the defines for the kernel to one place
even if your tool is outside of the tree we can document these options
on the wiki as well and provide a link to your tool.
Thanks
Steven
__________________________________
Do you Yahoo!?
The all-new My Yahoo! - Get yours free!
http://my.yahoo.com

[Casper Hornstrup]

>   -----Original Message-----
>  From: ros-dev-bounces(a)reactos.com
>  [mailto:[e-mail]] On Behalf Of Gregor Anich
>  Sent: 28. november 2004 00:51
>  To: ReactOS Development List
>  Subject: [ros-dev] Moving ntoskrnl configuration defines into
>  ntoskrnl/include/config.h ?
>  Hi!
>  I would like to add the possibility of configuring things
>  like WHOLE_PAGE_ALLOCATIONS or TAG_STATISTICS_TRACKING (from
>  ntoskrnl/mm/npool.c) It would be good if such defines were
>  moved into a file like ntoskrnl/include/config.h which is
>  then included by ntoskrnl.h and looks like this:
>  /* Enable tracking of statistics about the tagged blocks in
>  the pool */ #undef TAG_STATISTICS_TRACKING
>  /*
>   * Put each block in its own range of pages and position the
>  block at the
>   * end of the range so any accesses beyond the end of block
>  are to invalid
>   * memory locations.
>   */
>  #undef WHOLE_PAGE_ALLOCATIONS
>  I don't want to make the tool edit the C files directly. 
WHOLE_PAGE_ALLOCATIONS shouldn't be a compile time define. The
code should be written to retrieve the setting from registry
instead. The reason is to minimize the number of compile time
defines in the code base. The more compile time defines there
is, the harder it is to make sure a particular change does
not break building with one or more of the configurations.
Ususually a developer does not want to do a make clean &
reconfigure & make for each configuration to make sure it does
still build in those configurations. The same goes for KDBG.
Casper

[Anich Gregor]

On Sunday 28 November 2004 11:41, Casper Hornstrup wrote:
> >   -----Original
> > Message-----
> >  From: ros-dev-bounces(a)reactos.com
> >  [mailto:[e-mail]] On Behalf Of Gregor Anich
> >  Sent: 28. november 2004 00:51
> >  To: ReactOS Development List
> >  Subject: [ros-dev] Moving ntoskrnl configuration defines into
> >  ntoskrnl/include/config.h ?
> >  Hi!
> >  I would like to add the possibility of configuring things
> >  like WHOLE_PAGE_ALLOCATIONS or TAG_STATISTICS_TRACKING (from
> >  ntoskrnl/mm/npool.c) It would be good if such defines were
> >  moved into a file like ntoskrnl/include/config.h which is
> >  then included by ntoskrnl.h and looks like this:
> >  /* Enable tracking of statistics about the tagged blocks in
> >  the pool */ #undef TAG_STATISTICS_TRACKING
> >  /*
> >   * Put each block in its own range of pages and position the
> >  block at the
> >   * end of the range so any accesses beyond the end of block
> >  are to invalid
> >   * memory locations.
> >   */
> >  #undef WHOLE_PAGE_ALLOCATIONS
> >  I don't want to make the tool edit the C files directly. 
>  WHOLE_PAGE_ALLOCATIONS shouldn't be a compile time define. The
>  code should be written to retrieve the setting from registry
>  instead. The reason is to minimize the number of compile time
>  defines in the code base. The more compile time defines there
>  is, the harder it is to make sure a particular change does
>  not break building with one or more of the configurations.
>  Ususually a developer does not want to do a make clean &
>  reconfigure & make for each configuration to make sure it does
>  still build in those configurations. The same goes for KDBG.
>  Casper
>  
Hi!
It would be cool if our build system supported to select the target directory
so one can use one source directory, 2 configurations and build each
configuration into a different target directory ;-)
But that wouldn't solve the problem you described either, it would just reduce
diskspace requirements if one wants to have a dbg and non-dbg build for
example (to test building of both before commiting stuff for example)
€0.02 - blight ;)