diff options
author | Arjan van de Ven <arjan@linux.intel.com> | 2008-07-25 22:45:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-07-26 15:00:07 -0400 |
commit | 12e0036818eed243c8ed6583ebf98261a2554e12 (patch) | |
tree | 2f60ad7d1c869a995faa5b73150b409705d0e8b2 /block | |
parent | 261c40c1191ad8d7a2e49fa2bb5f6a84e3d44b10 (diff) |
Use WARN() in block/
Use WARN() instead of a printk+WARN_ON() pair; this way the message
becomes part of the warning section for better reporting/collection.
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Cc: Jens Axboe <jens.axboe@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'block')
-rw-r--r-- | block/as-iosched.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/block/as-iosched.c b/block/as-iosched.c index 9735acb5b4f5..cf4eb0eefbbf 100644 --- a/block/as-iosched.c +++ b/block/as-iosched.c | |||
@@ -837,8 +837,7 @@ static void as_completed_request(struct request_queue *q, struct request *rq) | |||
837 | WARN_ON(!list_empty(&rq->queuelist)); | 837 | WARN_ON(!list_empty(&rq->queuelist)); |
838 | 838 | ||
839 | if (RQ_STATE(rq) != AS_RQ_REMOVED) { | 839 | if (RQ_STATE(rq) != AS_RQ_REMOVED) { |
840 | printk("rq->state %d\n", RQ_STATE(rq)); | 840 | WARN(1, "rq->state %d\n", RQ_STATE(rq)); |
841 | WARN_ON(1); | ||
842 | goto out; | 841 | goto out; |
843 | } | 842 | } |
844 | 843 | ||