floppy: cleanup: expand macro DRWE
authorWilly Tarreau <w@1wt.eu>
Mon, 24 Feb 2020 21:23:50 +0000 (22:23 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 16 Mar 2020 14:26:57 +0000 (08:26 -0600)
This macro doesn't bring much value and only slightly obfuscates the
code by silently using global variable "current_drive", let's expand it.

Link: https://lore.kernel.org/r/20200224212352.8640-9-w@1wt.eu
Signed-off-by: Willy Tarreau <w@1wt.eu>
Signed-off-by: Denis Efremov <efremov@linux.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/floppy.c

index 6d4a2e14799a6e6607733081876bc73b95dc049b..d771579df57e575f952d82d01d08c45e0bc40bac 100644 (file)
@@ -306,8 +306,6 @@ static bool initialized;
        /* reverse mapping from unit and fdc to drive */
 #define REVDRIVE(fdc, unit) ((unit) + ((fdc) << 2))
 
-#define DRWE   (&write_errors[current_drive])
-
 #define PH_HEAD(floppy, head) (((((floppy)->stretch & 2) >> 1) ^ head) << 2)
 #define STRETCH(floppy)        ((floppy)->stretch & FD_STRETCH)
 
@@ -2069,7 +2067,7 @@ static void bad_flp_intr(void)
                        return;
        }
        err_count = ++(*errors);
-       INFBOUND(DRWE->badness, err_count);
+       INFBOUND(write_errors[current_drive].badness, err_count);
        if (err_count > drive_params[current_drive].max_errors.abort)
                cont->done(0);
        if (err_count > drive_params[current_drive].max_errors.reset)
@@ -2274,13 +2272,13 @@ static void request_done(int uptodate)
        } else {
                if (rq_data_dir(req) == WRITE) {
                        /* record write error information */
-                       DRWE->write_errors++;
-                       if (DRWE->write_errors == 1) {
-                               DRWE->first_error_sector = blk_rq_pos(req);
-                               DRWE->first_error_generation = drive_state[current_drive].generation;
+                       write_errors[current_drive].write_errors++;
+                       if (write_errors[current_drive].write_errors == 1) {
+                               write_errors[current_drive].first_error_sector = blk_rq_pos(req);
+                               write_errors[current_drive].first_error_generation = drive_state[current_drive].generation;
                        }
-                       DRWE->last_error_sector = blk_rq_pos(req);
-                       DRWE->last_error_generation = drive_state[current_drive].generation;
+                       write_errors[current_drive].last_error_sector = blk_rq_pos(req);
+                       write_errors[current_drive].last_error_generation = drive_state[current_drive].generation;
                }
                floppy_end_request(req, BLK_STS_IOERR);
        }