Commit 2865c073 authored by Tomas Henzl's avatar Tomas Henzl Committed by James Bottomley

mptsas: fix depth param in scsi_track_queue_full

A user of scsi_track_queue_full should pass to the function a constant value
untill the queue-depth changes, otherwise the internal logic in
scsi_track_queue_full rejects the change.  Other users of this function use a
'sdev->queue_depth - 1' as depth parameter, let's do the same.
Signed-off-by: default avatarTomas Henzl <thenzl@redhat.com>
Acked-by: default avatar"Sreekanth Reddy" <Sreekanth.reddy@avagotech.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Odin.com>
parent 258c3af2
...@@ -4090,7 +4090,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event) ...@@ -4090,7 +4090,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event)
continue; continue;
} }
depth = scsi_track_queue_full(sdev, depth = scsi_track_queue_full(sdev,
current_depth - 1); sdev->queue_depth - 1);
if (depth > 0) if (depth > 0)
sdev_printk(KERN_INFO, sdev, sdev_printk(KERN_INFO, sdev,
"Queue depth reduced to (%d)\n", "Queue depth reduced to (%d)\n",
...@@ -4100,7 +4100,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event) ...@@ -4100,7 +4100,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event)
"Tagged Command Queueing is being " "Tagged Command Queueing is being "
"disabled\n"); "disabled\n");
else if (depth == 0) else if (depth == 0)
sdev_printk(KERN_INFO, sdev, sdev_printk(KERN_DEBUG, sdev,
"Queue depth not changed yet\n"); "Queue depth not changed yet\n");
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment