All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Zhijian <lizhijian@fujitsu.com>
To: linux-kernel@vger.kernel.org
Cc: Li Zhijian <lizhijian@fujitsu.com>,
	Michael Cyr <mikecyr@linux.ibm.com>,
	"James E.J. Bottomley" <jejb@linux.ibm.com>,
	"Martin K. Petersen" <martin.petersen@oracle.com>,
	linux-scsi@vger.kernel.org, target-devel@vger.kernel.org
Subject: [PATCH 26/42] drivers/scsi/ibmvscsi_tgt: Convert snprintf to sysfs_emit
Date: Tue, 16 Jan 2024 12:51:35 +0800	[thread overview]
Message-ID: <20240116045151.3940401-24-lizhijian@fujitsu.com> (raw)
In-Reply-To: <20240116045151.3940401-1-lizhijian@fujitsu.com>

Per filesystems/sysfs.rst, show() should only use sysfs_emit()
or sysfs_emit_at() when formatting the value to be returned to user space.

coccinelle complains that there are still a couple of functions that use
snprintf(). Convert them to sysfs_emit().

> ./drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c:3619:8-16: WARNING: please use sysfs_emit
> ./drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c:3625:8-16: WARNING: please use sysfs_emit
> ./drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c:3633:8-16: WARNING: please use sysfs_emit

No functional change intended

CC: Michael Cyr <mikecyr@linux.ibm.com>
CC: "James E.J. Bottomley" <jejb@linux.ibm.com>
CC: "Martin K. Petersen" <martin.petersen@oracle.com>
CC: linux-scsi@vger.kernel.org
CC: target-devel@vger.kernel.org
Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
---
 drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
index 4dc411a58107..071dd3c4dcf8 100644
--- a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
+++ b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
@@ -3616,13 +3616,13 @@ static void ibmvscsis_remove(struct vio_dev *vdev)
 static ssize_t system_id_show(struct device *dev,
 			      struct device_attribute *attr, char *buf)
 {
-	return snprintf(buf, PAGE_SIZE, "%s\n", system_id);
+	return sysfs_emit(buf, "%s\n", system_id);
 }
 
 static ssize_t partition_number_show(struct device *dev,
 				     struct device_attribute *attr, char *buf)
 {
-	return snprintf(buf, PAGE_SIZE, "%x\n", partition_number);
+	return sysfs_emit(buf, "%x\n", partition_number);
 }
 
 static ssize_t unit_address_show(struct device *dev,
@@ -3630,7 +3630,7 @@ static ssize_t unit_address_show(struct device *dev,
 {
 	struct scsi_info *vscsi = container_of(dev, struct scsi_info, dev);
 
-	return snprintf(buf, PAGE_SIZE, "%x\n", vscsi->dma_dev->unit_address);
+	return sysfs_emit(buf, "%x\n", vscsi->dma_dev->unit_address);
 }
 
 static int ibmvscsis_get_system_info(void)
-- 
2.29.2


  parent reply	other threads:[~2024-01-16  4:52 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-16  4:10 [PATCH 00/42] Fix coccicheck warnings Li Zhijian
2024-01-16  4:10 ` [cocci] " Li Zhijian
2024-01-16  4:10 ` Li Zhijian
2024-01-16  4:10 ` Li Zhijian
2024-01-16  4:10 ` [PATCH 01/42] coccinelle: device_attr_show.cocci: update description and warning message Li Zhijian
2024-01-16  4:10   ` [cocci] " Li Zhijian
2024-01-18 20:26   ` Julia Lawall
2024-01-18 20:26     ` Julia Lawall
2024-01-19  2:53     ` Zhijian Li (Fujitsu)
2024-01-19  2:53       ` [cocci] " Zhijian Li (Fujitsu)
2024-01-19  6:25       ` Zhijian Li (Fujitsu)
2024-01-19  6:25         ` [cocci] " Zhijian Li (Fujitsu)
2024-01-16  4:10 ` [PATCH 02/42] arch/arm/mm: Convert snprintf to sysfs_emit Li Zhijian
2024-01-16  4:10   ` Li Zhijian
2024-01-16  4:51 ` [PATCH 03/42] arch/x86/events/amd: " Li Zhijian
2024-01-16  4:51   ` [PATCH 04/42] arch/x86/events/core: " Li Zhijian
2024-01-16  7:01     ` Adrian Hunter
2024-01-16  7:35       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 05/42] arch/x86/events/intel: " Li Zhijian
2024-01-19 22:00     ` Namhyung Kim
2024-01-22  2:59       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 06/42] drivers/bus/fsl-mc: " Li Zhijian
2024-01-16  4:51   ` [PATCH 07/42] drivers/edac: " Li Zhijian
2024-01-16  4:51   ` [PATCH 08/42] drivers/fsi: " Li Zhijian
2024-01-17 14:19     ` Eddie James
2024-01-16  4:51   ` [PATCH 09/42] drivers/hid/hid-lenovo: " Li Zhijian
2024-01-16  4:51   ` [PATCH 10/42] drivers/hid/hid-roccat-*: " Li Zhijian
2024-01-16  4:51   ` [PATCH 11/42] drivers/hid: " Li Zhijian
2024-02-24 16:59     ` Bruno Prémont
2024-02-28  9:56       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 12/42] drivers/message/fusion: " Li Zhijian
2024-01-16  4:51   ` [PATCH 13/42] drivers/most: " Li Zhijian
2024-01-16  4:51   ` [PATCH 14/42] drivers/platform/mellanox: " Li Zhijian
2024-01-24 11:01     ` Ilpo Järvinen
2024-01-16  4:51   ` [PATCH 15/42] drivers/ptp: " Li Zhijian
2024-01-16  5:01     ` Rahul Rameshbabu
2024-01-16  5:52       ` Zhijian Li (Fujitsu)
2024-01-16 15:33     ` Jakub Kicinski
2024-01-17  5:39       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 16/42] drivers/scsi/53c700: " Li Zhijian
2024-01-16  4:51   ` [PATCH 17/42] drivers/scsi/aacraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 18/42] drivers/scsi/aic94xx: " Li Zhijian
2024-01-16  4:51   ` [PATCH 19/42] drivers/scsi/arcmsr: " Li Zhijian
2024-01-16  4:51   ` [PATCH 20/42] drivers/scsi/be2iscsi: " Li Zhijian
2024-01-16  4:51   ` [PATCH 21/42] drivers/scsi/fcoe: " Li Zhijian
2024-01-16  4:51   ` [PATCH 22/42] drivers/scsi/fnic: " Li Zhijian
2024-01-18 23:24     ` Karan Tilak Kumar (kartilak)
2024-01-16  4:51   ` [PATCH 23/42] drivers/scsi/hpsa: " Li Zhijian
2024-01-16  4:51   ` [PATCH 24/42] drivers/scsi/hptiop: " Li Zhijian
2024-01-16  4:51   ` [PATCH 25/42] drivers/scsi/ibmvscsi: " Li Zhijian
2024-01-16  4:51     ` Li Zhijian
2024-01-16 21:38     ` Tyrel Datwyler
2024-01-16 21:38       ` Tyrel Datwyler
2024-01-16  4:51   ` Li Zhijian [this message]
2024-01-16 21:38     ` [PATCH 26/42] drivers/scsi/ibmvscsi_tgt: " Tyrel Datwyler
2024-01-16  4:51   ` [PATCH 27/42] drivers/scsi/isci: " Li Zhijian
2024-01-16  7:24     ` Artur Paszkiewicz
2024-01-16  4:51   ` [PATCH 28/42] drivers/scsi/megaraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 29/42] drivers/scsi/mpt3sas: " Li Zhijian
2024-01-16  4:51   ` [PATCH 30/42] drivers/scsi/myrb: " Li Zhijian
2024-01-16  4:51   ` [PATCH 31/42] drivers/scsi/myrs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 32/42] drivers/scsi/ncr53c8xx: " Li Zhijian
2024-01-16  4:51   ` [PATCH 33/42] drivers/scsi/pcmcia/sym53c500_cs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 34/42] drivers/scsi/pm8001: " Li Zhijian
2024-01-16  5:52     ` Jinpu Wang
2024-01-16  4:51   ` [PATCH 35/42] drivers/scsi/pmcraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 36/42] drivers/scsi/qla4xxx/ql4_attr: " Li Zhijian
2024-01-16  4:51   ` [PATCH 37/42] drivers/scsi/scsi_sysfs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 38/42] drivers/scsi/scsi_transport_fc: " Li Zhijian
2024-01-16  4:51   ` [PATCH 39/42] drivers/scsi/scsi_transport_sas: " Li Zhijian
2024-01-16  4:51   ` [PATCH 40/42] drivers/scsi/snic/snic_attrs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 41/42] drivers/video/fbdev/uvesafb: " Li Zhijian
2024-01-16  4:51     ` Li Zhijian
2024-01-16  4:51   ` [PATCH 42/42] kernel/cpu: " Li Zhijian
2024-01-26 17:32     ` [tip: smp/core] kernel/cpu: Convert snprintf() to sysfs_emit() tip-bot2 for Li Zhijian
2024-02-06  2:07 ` (subset) [PATCH 00/42] Fix coccicheck warnings Martin K. Petersen
2024-02-06  2:07   ` Martin K. Petersen
2024-02-06  2:07   ` [cocci] " Martin K. Petersen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240116045151.3940401-24-lizhijian@fujitsu.com \
    --to=lizhijian@fujitsu.com \
    --cc=jejb@linux.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --cc=mikecyr@linux.ibm.com \
    --cc=target-devel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.