Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1499587 > unrolled thread

[PATCH 3.4 031/125] scsi_sysfs: Fix queue_ramp_up_period return code

Started bylizf@kernel.org
First post2016-10-12 15:00 +0200
Last post2016-10-12 15:00 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 3.4 031/125] scsi_sysfs: Fix queue_ramp_up_period return code lizf@kernel.org - 2016-10-12 15:00 +0200

#1499587 — [PATCH 3.4 031/125] scsi_sysfs: Fix queue_ramp_up_period return code

Fromlizf@kernel.org
Date2016-10-12 15:00 +0200
Subject[PATCH 3.4 031/125] scsi_sysfs: Fix queue_ramp_up_period return code
Message-ID<srriy-4vs-27@gated-at.bofh.it>
From: Peter Oberparleiter <oberpar@linux.vnet.ibm.com>

3.4.113-rc1 review patch.  If anyone has any objections, please let me know.

------------------


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(-)

diff --git a/drivers/scsi/scsi_sysfs.c b/drivers/scsi/scsi_sysfs.c
index 05c99af..ca6ddab 100644
--- a/drivers/scsi/scsi_sysfs.c
+++ b/drivers/scsi/scsi_sysfs.c
@@ -793,7 +793,7 @@ sdev_store_queue_ramp_up_period(struct device *dev,
 		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 =
-- 
1.9.1

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web