libata: fix last_reset timestamp handling
authorTejun Heo <tj@kernel.org>
Tue, 4 Nov 2008 08:08:40 +0000 (17:08 +0900)
committerJeff Garzik <jgarzik@redhat.com>
Tue, 11 Nov 2008 08:01:21 +0000 (03:01 -0500)
commit19b723218bde79c60a394a3caee9eb156ac2d356
tree5ea111eb99068ae04ec5bc5ab2089a288362b5c1
parent44901a96847b9967c057832b185e2f34ee6a14e5
libata: fix last_reset timestamp handling

ehc->last_reset is used to ensure that resets are not issued too
close to each other.  It's initialized to jiffies minus one minute
on EH entry.  However, when new links are initialized after PMP is
probed, new links have zero for this timestamp resulting in long wait
depending on the current jiffies.

This patch makes last_set considered iff ATA_EHI_DID_RESET is set, in
which case last_reset is always initialized.  As an added precaution,
WARN_ON() is added so that warning is printed if last_reset is
in future.

This problem is spotted and debugged by Shane Huang.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Shane Huang <Shane.Huang@amd.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/ata/libata-eh.c