Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1496473 > unrolled thread

[PATCH 23/54] md/raid1: Replace a seq_printf() call by seq_puts() in raid1_status()

Started bySF Markus Elfring <elfring@users.sourceforge.net>
First post2016-10-06 11:30 +0200
Last post2016-10-06 11:30 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 23/54] md/raid1: Replace a seq_printf() call by seq_puts() in  raid1_status() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-06 11:30 +0200

#1496473 — [PATCH 23/54] md/raid1: Replace a seq_printf() call by seq_puts() in raid1_status()

FromSF Markus Elfring <elfring@users.sourceforge.net>
Date2016-10-06 11:30 +0200
Subject[PATCH 23/54] md/raid1: Replace a seq_printf() call by seq_puts() in raid1_status()
Message-ID<spda1-6DS-3@gated-at.bofh.it>
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Tue, 4 Oct 2016 21:16:13 +0200

The script "checkpatch.pl" pointed information out like the following.

WARNING: Prefer seq_puts to seq_printf

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
 drivers/md/raid1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
index 95f6098..ec6aafb 100644
--- a/drivers/md/raid1.c
+++ b/drivers/md/raid1.c
@@ -1417,7 +1417,7 @@ static void raid1_status(struct seq_file *seq, struct mddev *mddev)
 			   rdev && test_bit(In_sync, &rdev->flags) ? "U" : "_");
 	}
 	rcu_read_unlock();
-	seq_printf(seq, "]");
+	seq_puts(seq, "]");
 }
 
 static void raid1_error(struct mddev *mddev, struct md_rdev *rdev)
-- 
2.10.1

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web