scsi: target: core: Add line break to status show

maurizio-lombardi · maurizio-lombardi · commit b242bbc4384b · 2025-08-29T15:31:15.000+02:00
JIRA: https://issues.redhat.com/browse/RHEL-111938 To ensure the output is not tangled with the shell prompt, add a line break to clearly display the status. Signed-off-by: Guixin Liu <kanie@linux.alibaba.com> Link: https://lore.kernel.org/r/20250114025041.97301-1-kanie@linux.alibaba.com Reviewed-by: Mike Christie <michael.christie@oracle.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> (cherry picked from commit c9d2782) Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
diff --git a/drivers/target/target_core_stat.c b/drivers/target/target_core_stat.c
@@ -117,9 +117,9 @@ static ssize_t target_stat_tgt_status_show(struct config_item *item,
 		char *page)
 {
 	if (to_stat_tgt_dev(item)->export_count)
-		return snprintf(page, PAGE_SIZE, "activated");
+		return snprintf(page, PAGE_SIZE, "activated\n");
 	else
-		return snprintf(page, PAGE_SIZE, "deactivated");
+		return snprintf(page, PAGE_SIZE, "deactivated\n");
 }
 
 static ssize_t target_stat_tgt_non_access_lus_show(struct config_item *item,

Original file line number	Diff line number	Diff line change
`@@ -117,9 +117,9 @@ static ssize_t target_stat_tgt_status_show(struct config_item *item,`
`117`	`117`	`char *page)`
`118`	`118`	`{`
`119`	`119`	`if (to_stat_tgt_dev(item)->export_count)`
`120`		`- return snprintf(page, PAGE_SIZE, "activated");`
	`120`	`+ return snprintf(page, PAGE_SIZE, "activated\n");`
`121`	`121`	`else`
`122`		`- return snprintf(page, PAGE_SIZE, "deactivated");`
	`122`	`+ return snprintf(page, PAGE_SIZE, "deactivated\n");`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`static ssize_t target_stat_tgt_non_access_lus_show(struct config_item *item,`