dm zoned: remove spurious newlines from debugging messages
authorHannes Reinecke <hare@suse.de>
Thu, 14 May 2020 06:09:29 +0000 (08:09 +0200)
committerMike Snitzer <snitzer@redhat.com>
Wed, 20 May 2020 21:09:46 +0000 (17:09 -0400)
DMDEBUG will already add a newline to the logging messages, so we
shouldn't be adding it to the message itself.

Signed-off-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Damien Le Moal <damien.lemoal@wdc.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-zoned-reclaim.c
drivers/md/dm-zoned-target.c

index 3c8847d..7e9b11e 100644 (file)
@@ -403,7 +403,7 @@ out:
 
        ret = dmz_flush_metadata(zrc->metadata);
        if (ret) {
-               DMDEBUG("(%s): Metadata flush for zone %u failed, err %d\n",
+               DMDEBUG("(%s): Metadata flush for zone %u failed, err %d",
                        dmz_metadata_label(zmd), rzone->id, ret);
                return ret;
        }
@@ -491,7 +491,7 @@ static void dmz_reclaim_work(struct work_struct *work)
 
        ret = dmz_do_reclaim(zrc);
        if (ret) {
-               DMDEBUG("(%s): Reclaim error %d\n",
+               DMDEBUG("(%s): Reclaim error %d",
                        dmz_metadata_label(zmd), ret);
                if (!dmz_check_dev(zmd))
                        return;
index ea43f68..a3d572d 100644 (file)
@@ -515,7 +515,7 @@ static void dmz_flush_work(struct work_struct *work)
        /* Flush dirty metadata blocks */
        ret = dmz_flush_metadata(dmz->metadata);
        if (ret)
-               DMDEBUG("(%s): Metadata flush failed, rc=%d\n",
+               DMDEBUG("(%s): Metadata flush failed, rc=%d",
                        dmz_metadata_label(dmz->metadata), ret);
 
        /* Process queued flush requests */
@@ -679,7 +679,7 @@ static int dmz_map(struct dm_target *ti, struct bio *bio)
        /* Now ready to handle this BIO */
        ret = dmz_queue_chunk_work(dmz, bio);
        if (ret) {
-               DMDEBUG("(%s): BIO op %d, can't process chunk %llu, err %i\n",
+               DMDEBUG("(%s): BIO op %d, can't process chunk %llu, err %i",
                        dmz_metadata_label(zmd),
                        bio_op(bio), (u64)dmz_bio_chunk(zmd, bio),
                        ret);