Re: Can you please test the attached patch for nv hardreset problem?

[Tejun Heo <tj@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 720 bytes --]

gettinther wrote:
> The log simply ignores the drive altogether.
> 
> from the sound coming from the drive, it does seem that the drive
> initially starts at boot, then stops.  After that stage the device
> detection takes place.  as the item is a pata to sata converter
> (emulated sata), it does not support hot plugging.
> 
> I also need to mention that this behavior (start-stop during boot) did
> not happen on the 2.6.24 and I need to check the 2.6.27.  However the
> device is detected by both of these kernels.

Hmmm...

> Oct 14 21:18:12 localhost kernel: ata2: SATA link up 1.5 Gbps (SStatus
> 113 SControl 300)

Hmmm... Can you please apply the attached patch on top of it and
report the boot log?

-- 
tejun

[-- Attachment #2: debug.patch --]
[-- Type: text/x-patch, Size: 922 bytes --]

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index e97afce..1fc4f80 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -1160,6 +1160,8 @@ unsigned int ata_dev_classify(const struct ata_taskfile *tf)
 	 * We follow the current spec and consider that 0x69/0x96
 	 * identifies a port multiplier and 0x3c/0xc3 a SEMB device.
 	 */
+	printk("XXX classify %02x/%02x\n", tf->lbam, tf->lbah);
+
 	if ((tf->lbam == 0) && (tf->lbah == 0)) {
 		DPRINTK("found ATA device by sig\n");
 		return ATA_DEV_ATA;
@@ -5855,7 +5857,7 @@ int ata_host_register(struct ata_host *host, struct scsi_host_template *sht)
 
 			ehi->probe_mask |= ATA_ALL_DEVICES;
 			ehi->action |= ATA_EH_RESET | ATA_EH_LPM;
-			ehi->flags |= ATA_EHI_NO_AUTOPSY | ATA_EHI_QUIET;
+			ehi->flags |= ATA_EHI_NO_AUTOPSY/* | ATA_EHI_QUIET*/;
 
 			ap->pflags &= ~ATA_PFLAG_INITIALIZING;
 			ap->pflags |= ATA_PFLAG_LOADING;