svn commit: r218280 - head/sys/dev/aac
Ed Maste
emaste at FreeBSD.org
Fri Feb 4 15:45:48 UTC 2011
Author: emaste
Date: Fri Feb 4 15:45:48 2011
New Revision: 218280
URL: http://svn.freebsd.org/changeset/base/218280
Log:
We can pass a format string and args to panic(), so instead of using
printf() to output some information before a panic, just include that
information in the panic.
Submitted by: bde
Reviewed by: bde
Modified:
head/sys/dev/aac/aacvar.h
Modified: head/sys/dev/aac/aacvar.h
==============================================================================
--- head/sys/dev/aac/aacvar.h Fri Feb 4 15:28:26 2011 (r218279)
+++ head/sys/dev/aac/aacvar.h Fri Feb 4 15:45:48 2011 (r218280)
@@ -537,9 +537,8 @@ static __inline void \
aac_enqueue_ ## name (struct aac_command *cm) \
{ \
if ((cm->cm_flags & AAC_ON_AACQ_MASK) != 0) { \
- printf("command %p is on another queue, flags = %#x\n", \
- cm, cm->cm_flags); \
- panic("aac: command is on another queue"); \
+ panic("aac: command %p is on another queue, flags = %#x", \
+ cm, cm->cm_flags); \
} \
TAILQ_INSERT_TAIL(&cm->cm_sc->aac_ ## name, cm, cm_link); \
cm->cm_flags |= AAC_ON_ ## index; \
@@ -549,9 +548,8 @@ static __inline void \
aac_requeue_ ## name (struct aac_command *cm) \
{ \
if ((cm->cm_flags & AAC_ON_AACQ_MASK) != 0) { \
- printf("command %p is on another queue, flags = %#x\n", \
- cm, cm->cm_flags); \
- panic("aac: command is on another queue"); \
+ panic("aac: command %p is on another queue, flags = %#x", \
+ cm, cm->cm_flags); \
} \
TAILQ_INSERT_HEAD(&cm->cm_sc->aac_ ## name, cm, cm_link); \
cm->cm_flags |= AAC_ON_ ## index; \
@@ -564,10 +562,8 @@ aac_dequeue_ ## name (struct aac_softc *
\
if ((cm = TAILQ_FIRST(&sc->aac_ ## name)) != NULL) { \
if ((cm->cm_flags & AAC_ON_ ## index) == 0) { \
- printf("command %p not in queue, flags = %#x, " \
- "bit = %#x\n", cm, cm->cm_flags, \
- AAC_ON_ ## index); \
- panic("aac: command not in queue"); \
+ panic("aac: command %p not in queue, flags = %#x, bit = %#x", \
+ cm, cm->cm_flags, AAC_ON_ ## index); \
} \
TAILQ_REMOVE(&sc->aac_ ## name, cm, cm_link); \
cm->cm_flags &= ~AAC_ON_ ## index; \
@@ -579,10 +575,8 @@ static __inline void \
aac_remove_ ## name (struct aac_command *cm) \
{ \
if ((cm->cm_flags & AAC_ON_ ## index) == 0) { \
- printf("command %p not in queue, flags = %#x, " \
- "bit = %#x\n", cm, cm->cm_flags, \
- AAC_ON_ ## index); \
- panic("aac: command not in queue"); \
+ panic("aac: command %p not in queue, flags = %#x, bit = %#x", \
+ cm, cm->cm_flags, AAC_ON_ ## index); \
} \
TAILQ_REMOVE(&cm->cm_sc->aac_ ## name, cm, cm_link); \
cm->cm_flags &= ~AAC_ON_ ## index; \
More information about the svn-src-head
mailing list