]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
mptsas: fix depth param in scsi_track_queue_full
authorTomas Henzl <thenzl@redhat.com>
Wed, 3 Jun 2015 15:37:02 +0000 (17:37 +0200)
committerJames Bottomley <JBottomley@Odin.com>
Sat, 13 Jun 2015 15:40:33 +0000 (08:40 -0700)
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: Tomas Henzl <thenzl@redhat.com>
Acked-by: "Sreekanth Reddy" <Sreekanth.reddy@avagotech.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
drivers/message/fusion/mptsas.c

index 5bdaae15a74257d9c252abed3b15ae5ce5cde69a..005a88b9f44029c42b2c700fa4e275482039c47f 100644 (file)
@@ -4090,7 +4090,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event)
                                        continue;
                                }
                                depth = scsi_track_queue_full(sdev,
-                                   current_depth - 1);
+                                       sdev->queue_depth - 1);
                                if (depth > 0)
                                        sdev_printk(KERN_INFO, sdev,
                                        "Queue depth reduced to (%d)\n",
@@ -4100,7 +4100,7 @@ mptsas_handle_queue_full_event(struct fw_event_work *fw_event)
                                        "Tagged Command Queueing is being "
                                        "disabled\n");
                                else if (depth == 0)
-                                       sdev_printk(KERN_INFO, sdev,
+                                       sdev_printk(KERN_DEBUG, sdev,
                                        "Queue depth not changed yet\n");
                        }
                }