raid1: stop using bio_devname
authorChristoph Hellwig <hch@lst.de>
Fri, 4 Mar 2022 18:01:02 +0000 (19:01 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 7 Mar 2022 13:42:33 +0000 (06:42 -0700)
Use the %pg format specifier to save on stack consuption and code size.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Acked-by: Song Liu <song@kernel.org>
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Link: https://lore.kernel.org/r/20220304180105.409765-8-hch@lst.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/md/raid1.c

index c3288d4..03477e9 100644 (file)
@@ -2072,15 +2072,14 @@ static int fix_sync_read_error(struct r1bio *r1_bio)
                } while (!success && d != r1_bio->read_disk);
 
                if (!success) {
-                       char b[BDEVNAME_SIZE];
                        int abort = 0;
                        /* Cannot read from anywhere, this block is lost.
                         * Record a bad block on each device.  If that doesn't
                         * work just disable and interrupt the recovery.
                         * Don't fail devices as that won't really help.
                         */
-                       pr_crit_ratelimited("md/raid1:%s: %s: unrecoverable I/O read error for block %llu\n",
-                                           mdname(mddev), bio_devname(bio, b),
+                       pr_crit_ratelimited("md/raid1:%s: %pg: unrecoverable I/O read error for block %llu\n",
+                                           mdname(mddev), bio->bi_bdev,
                                            (unsigned long long)r1_bio->sector);
                        for (d = 0; d < conf->raid_disks * 2; d++) {
                                rdev = conf->mirrors[d].rdev;