2022-06-30 03:06:38

by Li Zhijian

[permalink] [raw]
Subject: [PATCH] scsi: Shorten too long warning messages

sdev_printk will only accept messages less than 128 bytes. So shorten
the messages exceeding 128 bytes avoid getting a incomplete sentence like:
[ 475.156955] sd 9:0:0:0: Warning! Received an indication that the LUN assignments on this target have changed. The Linux SCSI layer does not automatical

Suggested-by: Finn Thain <[email protected]>
Signed-off-by: Li Zhijian <[email protected]>
---
V2: update patch as suggested by Finn and upadte the subject correspondingly
---
drivers/scsi/scsi_error.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index 49ef864df581..947d98a0565f 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -463,14 +463,12 @@ static void scsi_report_sense(struct scsi_device *sdev,
evt_type = SDEV_EVT_LUN_CHANGE_REPORTED;
scsi_report_lun_change(sdev);
sdev_printk(KERN_WARNING, sdev,
- "Warning! Received an indication that the "
"LUN assignments on this target have "
"changed. The Linux SCSI layer does not "
"automatically remap LUN assignments.\n");
} else if (sshdr->asc == 0x3f)
sdev_printk(KERN_WARNING, sdev,
- "Warning! Received an indication that the "
- "operating parameters on this target have "
+ "Operating parameters on this target have "
"changed. The Linux SCSI layer does not "
"automatically adjust these parameters.\n");

--
2.31.1


2022-06-30 16:20:56

by Bart Van Assche

[permalink] [raw]
Subject: Re: [PATCH] scsi: Shorten too long warning messages

On 6/29/22 19:38, [email protected] wrote:
> sdev_printk will only accept messages less than 128 bytes. So shorten
> the messages exceeding 128 bytes avoid getting a incomplete sentence like:
> [ 475.156955] sd 9:0:0:0: Warning! Received an indication that the LUN assignments on this target have changed. The Linux SCSI layer does not automatical

Reviewed-by: Bart Van Assche <[email protected]>

2022-07-07 21:05:52

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH] scsi: Shorten too long warning messages


> sdev_printk will only accept messages less than 128 bytes. So shorten
> the messages exceeding 128 bytes avoid getting a incomplete sentence like:
> [ 475.156955] sd 9:0:0:0: Warning! Received an indication that the LUN
> assignments on this target have changed. The Linux SCSI layer does not
> automatical

Applied to 5.20/scsi-staging, thanks!

--
Martin K. Petersen Oracle Linux Engineering

2022-07-14 04:32:33

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH] scsi: Shorten too long warning messages

On Thu, 30 Jun 2022 02:38:11 +0000, [email protected] wrote:

> sdev_printk will only accept messages less than 128 bytes. So shorten
> the messages exceeding 128 bytes avoid getting a incomplete sentence like:
> [ 475.156955] sd 9:0:0:0: Warning! Received an indication that the LUN assignments on this target have changed. The Linux SCSI layer does not automatical
>
>

Applied to 5.20/scsi-queue, thanks!

[1/1] scsi: Shorten too long warning messages
https://git.kernel.org/mkp/scsi/c/a2417db3679c

--
Martin K. Petersen Oracle Linux Engineering