| From 863e02d0e173bb9d8cea6861be22820b25c076cc Mon Sep 17 00:00:00 2001 |
| From: Peter Oberparleiter <oberpar@linux.vnet.ibm.com> |
| Date: Tue, 27 Oct 2015 10:49:54 +0100 |
| Subject: scsi_sysfs: Fix queue_ramp_up_period return code |
| |
| commit 863e02d0e173bb9d8cea6861be22820b25c076cc upstream. |
| |
| Writing a number to /sys/bus/scsi/devices/<sdev>/queue_ramp_up_period |
| returns the value of that number instead of the number of bytes written. |
| This behavior can confuse programs expecting POSIX write() semantics. |
| Fix this by returning the number of bytes written instead. |
| |
| Signed-off-by: Peter Oberparleiter <oberpar@linux.vnet.ibm.com> |
| Reviewed-by: Hannes Reinecke <hare@suse.de> |
| Reviewed-by: Matthew R. Ochs <mrochs@linux.vnet.ibm.com> |
| Reviewed-by: Ewan D. Milne <emilne@redhat.com> |
| Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> |
| Signed-off-by: Zefan Li <lizefan@huawei.com> |
| --- |
| drivers/scsi/scsi_sysfs.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/scsi/scsi_sysfs.c |
| +++ b/drivers/scsi/scsi_sysfs.c |
| @@ -793,7 +793,7 @@ sdev_store_queue_ramp_up_period(struct d |
| return -EINVAL; |
| |
| sdev->queue_ramp_up_period = msecs_to_jiffies(period); |
| - return period; |
| + return count; |
| } |
| |
| static struct device_attribute sdev_attr_queue_ramp_up_period = |
