On 17-Jul-2002 Bruce Evans wrote: > On Tue, 16 Jul 2002, W Gerald Hicks wrote: > >> Following patch should silence it. >> ... >> Index: src/sys/dev/md/md.c >> =================================================================== >> RCS file: /home/ncvs/src/sys/dev/md/md.c,v >> retrieving revision 1.66 >> diff -u -r1.66 md.c >> --- src/sys/dev/md/md.c 24 Jun 2002 12:07:02 -0000 1.66 >> +++ src/sys/dev/md/md.c 16 Jul 2002 21:32:44 -0000 >> @@ -606,6 +606,7 @@ >> error = mdstart_swap(sc, bp); >> break; >> default: >> + error = -1; >> panic("Impossible md(type)"); >> break; >> } > > This change and the break after the panic() are bogus. panic() "never" > returns, and the compiler knows this. Unfortunately, the RESTARTABLE_PANICS > option subverts the semantics of panic(), so panic() sometimes returns. > This breaks compilation of md.c and 28 other files in NOTES. The fix > shouldn't be to disturb the usual flow of control in those 29 files.
We should probably turn RESTARTABLE_PANICS off in NOTES. > Bruce -- John Baldwin <[EMAIL PROTECTED]> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to [EMAIL PROTECTED] with "unsubscribe freebsd-current" in the body of the message