[SCSI] libfc: discovery retry should clear pending first.
authorJoe Eykholt <jeykholt@cisco.com>
Tue, 25 Aug 2009 21:02:38 +0000 (14:02 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Thu, 10 Sep 2009 17:07:51 +0000 (12:07 -0500)
Currently fc_disc_timeout() restarts discovery only if it is not pending.
When the timer is scheduled, the discovery is left pending, so the
timeout never restarts it.

Fix by not checking for pending in the timeout handler.

If discovery is stopped and restarted in the meantime, the timeout will
be canceled.

Also, when a new discovery is started, the retry count wasn't cleared.

Signed-off-by: Joe Eykholt <jeykholt@cisco.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/libfc/fc_disc.c

index 9b8043bdeddbc63484f5b42fcd55328f2c8ba3a5..3efdbbab9b2bbae967fd80db5004d615db286021 100644 (file)
@@ -233,6 +233,7 @@ static void fc_disc_restart(struct fc_disc *disc)
         * freshly-discovered remote ports.  Avoid wrapping to zero.
         */
        disc->disc_id = (disc->disc_id + 2) | 1;
+       disc->retry_count = 0;
        fc_disc_gpn_ft_req(disc);
 }
 
@@ -563,8 +564,7 @@ static void fc_disc_timeout(struct work_struct *work)
                                            struct fc_disc,
                                            disc_work.work);
        mutex_lock(&disc->disc_mutex);
-       if (disc->requested && !disc->pending)
-               fc_disc_gpn_ft_req(disc);
+       fc_disc_gpn_ft_req(disc);
        mutex_unlock(&disc->disc_mutex);
 }