cvs commit: src/sys/amd64/conf DEFAULTS src/sys/i386/conf DEFAULTS src/sys/ia64/conf DEFAULTS src/sys/pc98/conf DEFAULTS src/sys/powerpc/conf DEFAULTS src/sys/sparc64/conf DEFAULTS src/sys/sparc64/sparc64 machdep.c src/sys/sun4v/conf DEFAULTS

[Simon L. Nielsen]

On 2006.11.04 04:58:10 +0000, John Birrell wrote:
> jb          2006-11-04 04:58:10 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/amd64/conf       DEFAULTS 
>     sys/i386/conf        DEFAULTS 
>     sys/ia64/conf        DEFAULTS 
>     sys/pc98/conf        DEFAULTS 
>     sys/powerpc/conf     DEFAULTS 
>     sys/sparc64/conf     DEFAULTS 
>     sys/sparc64/sparc64  machdep.c 
>     sys/sun4v/conf       DEFAULTS 
>   Log:
>   Build in kernel support for loading DTrace modules by default. This
>   adds the hooks that DTrace modules register with, and adds a few functions
>   which have the dtrace_ prefix to allow the DTrace FBT (function boundary
>   trace) provider to avoid tracing because they are called from the DTtrace
>   probe context.

Why isn't it just in GENERIC so people can easily disable it if they
like?  I thought DEFAULTS was just meant for things which caused users
pain (and people answering support mails...) if they accidentally
missed the option?  I don't really see DTrace falling in that
category.

On a side note, I think it's really great with the DTrace support and
I look forward to trying it out.  Thanks for working on it!

-- 
Simon L. Nielsen