Commit 01b6b67e authored by Stephen Hemminger's avatar Stephen Hemminger Committed by Jens Axboe

floppy: use warning macros

Convert assertions to use WARN().  There are several error checks in the
code for things that should never happen.  Convert them to standard
warnings so kerneloops.org will see them.
Signed-off-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarJens Axboe <jaxboe@fusionio.com>
parent b862f26f
...@@ -2542,10 +2542,8 @@ static int make_raw_rw_request(void) ...@@ -2542,10 +2542,8 @@ static int make_raw_rw_request(void)
int tracksize; int tracksize;
int ssize; int ssize;
if (max_buffer_sectors == 0) { if (WARN(max_buffer_sectors == 0, "VFS: Block I/O scheduled on unopened device\n"))
pr_info("VFS: Block I/O scheduled on unopened device\n");
return 0; return 0;
}
set_fdc((long)current_req->rq_disk->private_data); set_fdc((long)current_req->rq_disk->private_data);
...@@ -2895,19 +2893,16 @@ static void process_fd_request(void) ...@@ -2895,19 +2893,16 @@ static void process_fd_request(void)
static void do_fd_request(struct request_queue *q) static void do_fd_request(struct request_queue *q)
{ {
if (max_buffer_sectors == 0) { if (WARN(max_buffer_sectors == 0,
pr_info("VFS: %s called on non-open device\n", __func__); "VFS: %s called on non-open device\n", __func__))
return; return;
}
if (atomic_read(&usage_count) == 0) { if (WARN(atomic_read(&usage_count) == 0,
pr_info("warning: usage count=0, current_req=%p exiting\n", "warning: usage count=0, current_req=%p sect=%ld type=%x flags=%x\n",
current_req); current_req, (long)blk_rq_pos(current_req), current_req->cmd_type,
pr_info("sect=%ld type=%x flags=%x\n", current_req->cmd_flags))
(long)blk_rq_pos(current_req), current_req->cmd_type,
current_req->cmd_flags);
return; return;
}
if (test_bit(0, &fdc_busy)) { if (test_bit(0, &fdc_busy)) {
/* fdc busy, this new request will be treated when the /* fdc busy, this new request will be treated when the
current one is done */ current one is done */
...@@ -3817,10 +3812,10 @@ static int floppy_revalidate(struct gendisk *disk) ...@@ -3817,10 +3812,10 @@ static int floppy_revalidate(struct gendisk *disk)
if (test_bit(FD_DISK_CHANGED_BIT, &UDRS->flags) || if (test_bit(FD_DISK_CHANGED_BIT, &UDRS->flags) ||
test_bit(FD_VERIFY_BIT, &UDRS->flags) || test_bit(FD_VERIFY_BIT, &UDRS->flags) ||
test_bit(drive, &fake_change) || NO_GEOM) { test_bit(drive, &fake_change) || NO_GEOM) {
if (atomic_read(&usage_count) == 0) { if (WARN(atomic_read(&usage_count) == 0,
pr_info("VFS: revalidate called on non-open device.\n"); "VFS: revalidate called on non-open device.\n"))
return -EFAULT; return -EFAULT;
}
lock_fdc(drive, false); lock_fdc(drive, false);
cf = (test_bit(FD_DISK_CHANGED_BIT, &UDRS->flags) || cf = (test_bit(FD_DISK_CHANGED_BIT, &UDRS->flags) ||
test_bit(FD_VERIFY_BIT, &UDRS->flags)); test_bit(FD_VERIFY_BIT, &UDRS->flags));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment