Date: Mon, 21 Apr 2008 14:48:31 -0500 From: Brooks Davis <brooks@freebsd.org> To: Alfred Perlstein <alfred@freebsd.org> Cc: arch@freebsd.org, current@freebsd.org Subject: Re: ddb scripts now load by default Message-ID: <20080421194831.GB21205@lor.one-eyed-alien.net> In-Reply-To: <20080421192257.GF95731@elvis.mu.org> References: <20080421182338.GB20417@lor.one-eyed-alien.net> <20080421185557.GB95731@elvis.mu.org> <20080421192059.GA20748@lor.one-eyed-alien.net> <20080421192257.GF95731@elvis.mu.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--CUfgB8w4ZwR/yMy5 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Apr 21, 2008 at 12:22:57PM -0700, Alfred Perlstein wrote: > * Brooks Davis <brooks@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. > > >=20 > > > Can we somehow put a "press a key to abort reboot" option in that > > > script before acting on it? > >=20 > > echo 'ddb_enable=3D"NO" >> /etc/rc.conf >=20 > See, that's not really what I asked for. :) >=20 > 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 >=20 > >=20 > > -- Brooks > >=20 > > > -Alfred (who thinks this is really cool, but wants to be able to > > > debug kernels without having to learn and disable something > > > new) > > >=20 > > > * Brooks Davis <brooks@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=3DNO or change the script to = not > > > > perform at textdump and reboot the machine. > > > >=20 > > > > 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 GENE= RIC > > > > kernel will probably want to be shipped with at least the options > > > > DDB_UNATTENDED and SC_DISABLE_KDBKEY. > > > >=20 > > > > The fact that the watchdog driver changes it's behavior based on > > > > DDB_UNATTENDED is probably a bug that should be fixed. > > > >=20 > > > > -- Brooks > > > >=20 > > > > ----- Forwarded message from Brooks Davis <brooks@FreeBSD.ORG> ----- > > > >=20 > > > > From: Brooks Davis <brooks@FreeBSD.ORG> > > > > Date: Mon, 21 Apr 2008 18:18:00 +0000 (UTC) > > > > To: brooks@FreeBSD.ORG > > > > Subject: [src] cvs commit: src/etc/defaults rc.conf > > > >=20 > > > > brooks 2008-04-21 18:17:48 UTC > > > >=20 > > > > FreeBSD src repository > > > >=20 > > > > Modified files: > > > > etc/defaults rc.conf=20 > > > > Log: > > > > Change the default of ddb_enable to YES so we default to generati= ng textdumps > > > > on panic. This means you get a potentially useful dump even if y= our system > > > > is running X when you panic. > > > > =20 > > > > X-MFC after: never > > > > =20 > > > > Revision Changes Path > > > > 1.332 +1 -1 src/etc/defaults/rc.conf > > > > _______________________________________________ > > > > cvs-all@freebsd.org mailing list > > > > http://lists.freebsd.org/mailman/listinfo/cvs-all > > > > To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org" > > > >=20 > > > >=20 > > > > Index: src/etc/defaults/rc.conf > > > > diff -u src/etc/defaults/rc.conf:1.331 src/etc/defaults/rc.conf:1.3= 32 > > > > --- 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 > > > > @@ -33,7 +33,7 @@ > > > > apm_enable=3D"NO" # Set to YES to enable APM BIOS functions (or N= O). > > > > apmd_enable=3D"NO" # Run apmd to handle APM event from userland. > > > > apmd_flags=3D"" # Flags to apmd (if enabled). > > > > -ddb_enable=3D"NO" # Set to YES to load ddb scripts at boot. > > > > +ddb_enable=3D"YES" # Load ddb scripts at boot. > > > > ddb_config=3D"/etc/ddb.conf" # ddb(8) config file. > > > > devd_enable=3D"YES" # Run devd, to trigger programs on device tre= e changes. > > > > devd_flags=3D"" # Additional flags for devd(8). > > > >=20 > > > >=20 > > > > ----- End forwarded message ----- > > >=20 > > >=20 > > >=20 > > > --=20 > > > - Alfred Perlstein > > > _______________________________________________ > > > freebsd-arch@freebsd.org mailing list > > > http://lists.freebsd.org/mailman/listinfo/freebsd-arch > > > To unsubscribe, send any mail to "freebsd-arch-unsubscribe@freebsd.or= g" > > >=20 >=20 >=20 >=20 > --=20 > - Alfred Perlstein >=20 --CUfgB8w4ZwR/yMy5 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.8 (FreeBSD) iD8DBQFIDO+PXY6L6fI4GtQRAgppAKCYHnKJXc9V3nsWt1fb34MsDFHPLgCgx/se eSa3d4MBE8Fz8M7H6vOJW9s= =O/01 -----END PGP SIGNATURE----- --CUfgB8w4ZwR/yMy5--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080421194831.GB21205>