Re: ddb scripts now load by default

I know this is a big step forward in a lot of ways, but it's also
a big step backwards for people used to how things just work.

Can we somehow put a "press a key to abort reboot" option in that
script before acting on it?

-Alfred (who thinks this is really cool, but wants to be able to
debug kernels without having to learn and disable something
new)

* Brooks Davis <brooks_at_freebsd.org> [080421 11:24] wrote:
> I've changed the default value of ddb_enable to YES so we now load ddb
> scripts from ddb.conf by default.  Developers expecting to perform
> active deugging should add ddb_enable=NO or change the script to not
> perform at textdump and reboot the machine.
> 
> This is another step toward hopefully shipping 8.0 with DDB built
> in and generating useful crash dumps.  The security implications of
> doing so need to be carefully considered and in particular the GENERIC
> kernel will probably want to be shipped with at least the options
> DDB_UNATTENDED and SC_DISABLE_KDBKEY.
> 
> The fact that the watchdog driver changes it's behavior based on
> DDB_UNATTENDED is probably a bug that should be fixed.
> 
> -- Brooks
> 
> ----- Forwarded message from Brooks Davis <brooks_at_FreeBSD.ORG> -----
> 
> From: Brooks Davis <brooks_at_FreeBSD.ORG>
> Date: Mon, 21 Apr 2008 18:18:00 +0000 (UTC)
> To: brooks_at_FreeBSD.ORG
> Subject: [src] cvs commit: src/etc/defaults rc.conf
> 
> brooks      2008-04-21 18:17:48 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     etc/defaults         rc.conf 
>   Log:
>   Change the default of ddb_enable to YES so we default to generating textdumps
>   on panic.  This means you get a potentially useful dump even if your system
>   is running X when you panic.
>   
>   X-MFC after:    never
>   
>   Revision  Changes    Path
>   1.332     +1 -1      src/etc/defaults/rc.conf
> _______________________________________________
> cvs-all_at_freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/cvs-all
> To unsubscribe, send any mail to "cvs-all-unsubscribe_at_freebsd.org"
> 
> 
> Index: src/etc/defaults/rc.conf
> diff -u src/etc/defaults/rc.conf:1.331 src/etc/defaults/rc.conf:1.332
> --- src/etc/defaults/rc.conf:1.331	Sun Apr 20 20:37:20 2008
> +++ src/etc/defaults/rc.conf	Mon Apr 21 18:17:48 2008
> _at__at_ -33,7 +33,7 _at__at_
>  apm_enable="NO"		# Set to YES to enable APM BIOS functions (or NO).
>  apmd_enable="NO"	# Run apmd to handle APM event from userland.
>  apmd_flags=""		# Flags to apmd (if enabled).
> -ddb_enable="NO"		# Set to YES to load ddb scripts at boot.
> +ddb_enable="YES"	# Load ddb scripts at boot.
>  ddb_config="/etc/ddb.conf"	# ddb(8) config file.
>  devd_enable="YES" 	# Run devd, to trigger programs on device tree changes.
>  devd_flags=""		# Additional flags for devd(8).
> 
> 
> ----- End forwarded message -----



-- 
- Alfred Perlstein