Re: ddb scripts now load by default

On Mon, Apr 21, 2008 at 12:22:57PM -0700, Alfred Perlstein wrote:
> * Brooks Davis <brooks_at_freebsd.org> [080421 12:20] wrote:
> > 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
> 
> See, that's not really what I asked for. :)
> 
> but it's OK.

Actually, what we need to implement what you want is a new ddb command
or possibly an optional countdown time for reset.  I'm just working on
the userland side at this point. :)

My goal here is to hit the right default for most people and figure
developers can adapt a bit, though hopefully not too much.  Ideally we'd
have some good example scripts in ddb.conf for developer use so new
developers could switch to them with some comment changes.  I definitely
encourage those who do use ddb a lot to add useful entries.

-- Brooks

> -Alfred
> 
> > 
> > -- 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"
> > > 
> 
> 
> 
> -- 
> - Alfred Perlstein
>