Re: ddb scripts now load by default

On Mon, Apr 21, 2008 at 11:55:57AM -0700, Alfred Perlstein wrote:
> 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?

echo 'ddb_enable="NO" >> /etc/rc.conf

-- Brooks

> -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
> _______________________________________________
> freebsd-arch_at_freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-arch
> To unsubscribe, send any mail to "freebsd-arch-unsubscribe_at_freebsd.org"
>