Received-SPF: softfail (google.com: domain of transitioning linux-ext4-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
MIME-Version: 1.0
References: <20220304180105.409765-1-hch@lst.de> <20220304180105.409765-9-hch@lst.de>
In-Reply-To: <20220304180105.409765-9-hch@lst.de>
From:   Song Liu <song@kernel.org>
Date:   Fri, 4 Mar 2022 11:14:57 -0800
Message-ID: <CAPhsuW5DNs6Kp42VmBi5J4qB8_3orBHhdA82RJuzg7kmQO_2aA@mail.gmail.com>
Subject: Re: [PATCH 08/10] raid5-ppl: stop using bio_devname
To:     Christoph Hellwig <hch@lst.de>
Cc:     Jens Axboe <axboe@kernel.dk>, "Theodore Ts'o" <tytso@mit.edu>,
        linux-block@vger.kernel.org, dm-devel@redhat.com,
        linux-raid <linux-raid@vger.kernel.org>,
        linux-ext4@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, Mar 4, 2022 at 10:01 AM Christoph Hellwig <hch@lst.de> wrote:
>
> 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>

> ---
>  drivers/md/raid5-ppl.c | 13 ++++---------
>  1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/md/raid5-ppl.c b/drivers/md/raid5-ppl.c
> index 93d9364a930e3..845db0ba7c17f 100644
> --- a/drivers/md/raid5-ppl.c
> +++ b/drivers/md/raid5-ppl.c
> @@ -416,12 +416,10 @@ static void ppl_log_endio(struct bio *bio)
>
>  static void ppl_submit_iounit_bio(struct ppl_io_unit *io, struct bio *bio)
>  {
> -       char b[BDEVNAME_SIZE];
> -
> -       pr_debug("%s: seq: %llu size: %u sector: %llu dev: %s\n",
> +       pr_debug("%s: seq: %llu size: %u sector: %llu dev: %pg\n",
>                  __func__, io->seq, bio->bi_iter.bi_size,
>                  (unsigned long long)bio->bi_iter.bi_sector,
> -                bio_devname(bio, b));
> +                bio->bi_bdev);
>
>         submit_bio(bio);
>  }
> @@ -589,9 +587,8 @@ static void ppl_flush_endio(struct bio *bio)
>         struct ppl_log *log = io->log;
>         struct ppl_conf *ppl_conf = log->ppl_conf;
>         struct r5conf *conf = ppl_conf->mddev->private;
> -       char b[BDEVNAME_SIZE];
>
> -       pr_debug("%s: dev: %s\n", __func__, bio_devname(bio, b));
> +       pr_debug("%s: dev: %pg\n", __func__, bio->bi_bdev);
>
>         if (bio->bi_status) {
>                 struct md_rdev *rdev;
> @@ -634,7 +631,6 @@ static void ppl_do_flush(struct ppl_io_unit *io)
>
>                 if (bdev) {
>                         struct bio *bio;
> -                       char b[BDEVNAME_SIZE];
>
>                         bio = bio_alloc_bioset(bdev, 0, GFP_NOIO,
>                                                REQ_OP_WRITE | REQ_PREFLUSH,
> @@ -642,8 +638,7 @@ static void ppl_do_flush(struct ppl_io_unit *io)
>                         bio->bi_private = io;
>                         bio->bi_end_io = ppl_flush_endio;
>
> -                       pr_debug("%s: dev: %s\n", __func__,
> -                                bio_devname(bio, b));
> +                       pr_debug("%s: dev: %ps\n", __func__, bio->bi_bdev);
>
>                         submit_bio(bio);
>                         flushed_disks++;
> --
> 2.30.2
>