Proposal: Unify printing the function name in panic messages()

Kirk McKusick mckusick at mckusick.com
Tue Feb 12 06:15:55 UTC 2013


> To: Kirk McKusick <mckusick at mckusick.com>
> cc: John Baldwin <jhb at freebsd.org>, Adrian Chadd <adrian at freebsd.org>,
>         Christoph Mallon <christoph.mallon at gmx.de>,
>         Andriy Gapon <avg at freebsd.org>, freebsd-arch at freebsd.org
> From: Mike Karels <mike at karels.net>
> Subject: Re: Proposal: Unify printing the function name in panic messages() 
> Date: Tue, 12 Feb 2013 00:01:07 -0600
> 
> I'm not arguing against consistency, nor even agaist the proposal itself
> (as modified for a lower-case panic macro).  However,  I don't think the
> lack of consistency is the real problem.  "panic: watchdog timeout" tells
> me what I need to know, whether or not it includes "watchdog_fire" or the
> line number.  The only problem that has been pointed out is lack of
> uniqueness.  That is a simpler problem to handle, and isn't handled by
> the current proposal as I understand it.
> 
> 		Mike

Though the default for the current proposal gives just the function name,
in its verbose mode it give file, function, and line number. And in its
lean and mean mode, just the error string. This replacing the hodge-podge
that we have now. My main point is that it is a significant improvement
over what we have now.

	Kirk


More information about the freebsd-arch mailing list