target-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] scsi: target: tcmu: add a missing newline when printing parameters
@ 2020-09-03 11:29 Xiongfeng Wang
  2020-09-16  0:19 ` Martin K. Petersen
  2020-09-22  3:56 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: Xiongfeng Wang @ 2020-09-03 11:29 UTC (permalink / raw)
  To: martin.petersen; +Cc: linux-scsi, target-devel, linux-kernel, wangxiongfeng2

When I cat module paramter 'global_max_data_area_mb' by sysfs, it
displays as follows. It's better to add a newline for easy reading.

root@(none):/# cat /sys/module/target_core_user/parameters/global_max_data_area_mb
2048noneroot@(none):/#

Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
---
 drivers/target/target_core_user.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/target/target_core_user.c b/drivers/target/target_core_user.c
index 9b75923..9775670 100644
--- a/drivers/target/target_core_user.c
+++ b/drivers/target/target_core_user.c
@@ -242,7 +242,7 @@ static int tcmu_set_global_max_data_area(const char *str,
 static int tcmu_get_global_max_data_area(char *buffer,
 					 const struct kernel_param *kp)
 {
-	return sprintf(buffer, "%d", TCMU_BLOCKS_TO_MBS(tcmu_global_max_blocks));
+	return sprintf(buffer, "%d\n", TCMU_BLOCKS_TO_MBS(tcmu_global_max_blocks));
 }
 
 static const struct kernel_param_ops tcmu_global_max_data_area_op = {
-- 
1.7.12.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: target: tcmu: add a missing newline when printing parameters
  2020-09-03 11:29 [PATCH] scsi: target: tcmu: add a missing newline when printing parameters Xiongfeng Wang
@ 2020-09-16  0:19 ` Martin K. Petersen
  2020-09-22  3:56 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2020-09-16  0:19 UTC (permalink / raw)
  To: Xiongfeng Wang; +Cc: martin.petersen, linux-scsi, target-devel, linux-kernel


Xiongfeng,

> When I cat module paramter 'global_max_data_area_mb' by sysfs, it
> displays as follows. It's better to add a newline for easy reading.

Applied to 5.10/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: target: tcmu: add a missing newline when printing parameters
  2020-09-03 11:29 [PATCH] scsi: target: tcmu: add a missing newline when printing parameters Xiongfeng Wang
  2020-09-16  0:19 ` Martin K. Petersen
@ 2020-09-22  3:56 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2020-09-22  3:56 UTC (permalink / raw)
  To: Xiongfeng Wang
  Cc: Martin K . Petersen, linux-scsi, target-devel, linux-kernel

On Thu, 3 Sep 2020 19:29:33 +0800, Xiongfeng Wang wrote:

> When I cat module paramter 'global_max_data_area_mb' by sysfs, it
> displays as follows. It's better to add a newline for easy reading.
> 
> root@(none):/# cat /sys/module/target_core_user/parameters/global_max_data_area_mb
> 2048noneroot@(none):/#

Applied to 5.10/scsi-queue, thanks!

[1/1] scsi: target: tcmu: Add missing newline when printing parameters
      https://git.kernel.org/mkp/scsi/c/6d70cb343484

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-09-22  3:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-03 11:29 [PATCH] scsi: target: tcmu: add a missing newline when printing parameters Xiongfeng Wang
2020-09-16  0:19 ` Martin K. Petersen
2020-09-22  3:56 ` Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).