[PATCH] support for PMP used in TT BlackX Duet drive dock

[PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

[-- Attachment #1: Type: Text/Plain, Size: 449 bytes --]

Hi,

Current linux kernel failes to communicate with PMP chip used in Thermaltake 
BlackX Duet (ST0015E) external drive, adding same quirks (some of them may not 
be necessary) as other chips have fixes it (tested on AHCI driver, on JMB363 
and ICH10R)

Please note that this patch is purposefully not in state includable in 
mainline, it is meant as a base for someone with more understanding of libata 
to make a proper fix

Thanks
Pavel Herrmann

[-- Attachment #2: 0001-libata-pmp-add-support-for-Thermaltake-BlackX-Duet-e.patch --]
[-- Type: text/x-patch, Size: 1268 bytes --]

From f1c2e6ec61c978846d6d3dadacc7c1b2997140a6 Mon Sep 17 00:00:00 2001
From: Pavel Herrmann <morpheus.ibis@gmail.com>
Date: Thu, 7 Apr 2011 18:15:26 +0200
Subject: [PATCH] libata-pmp: add support for Thermaltake BlackX Duet esata drive dock

some errors still show up, but the dock works, both drives can be
accessed at the same time

the chip maker and designation is unknown - possibly jmicron JMB350?

Signed-off-by: Pavel Herrmann <morpheus.ibis@gmail.com>
---
 drivers/ata/libata-pmp.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/drivers/ata/libata-pmp.c b/drivers/ata/libata-pmp.c
index 3120596..9ceef00 100644
--- a/drivers/ata/libata-pmp.c
+++ b/drivers/ata/libata-pmp.c
@@ -449,6 +449,14 @@ static void sata_pmp_quirks(struct ata_port *ap)
 		 * otherwise.  Don't try hard to recover it.
 		 */
 		ap->pmp_link[ap->nr_pmp_links - 1].flags |= ATA_LFLAG_NO_RETRY;
+		}
+	} else if (vendor == 0x1095 && devid == 0x4726) {
+		/* chip found in Thermaltake BlackX Duet */
+		ata_for_each_link(link, ap, EDGE) {
+			/* Use same quirks as sil chips	to make it work */
+			link->flags |= ATA_LFLAG_NO_LPM |
+				       ATA_LFLAG_NO_SRST |
+				       ATA_LFLAG_ASSUME_ATA;
 	}
 }
 
-- 
1.7.4.1

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

[-- Attachment #1: Type: Text/Plain, Size: 612 bytes --]

On Thursday 07 of April 2011 18:54:10 I wrote:
> Hi,
> 
> Current linux kernel failes to communicate with PMP chip used in
> Thermaltake BlackX Duet (ST0015E) external drive, adding same quirks (some
> of them may not be necessary) as other chips have fixes it (tested on AHCI
> driver, on JMB363 and ICH10R)
> 
> Please note that this patch is purposefully not in state includable in
> mainline, it is meant as a base for someone with more understanding of
> libata to make a proper fix
> 
> Thanks
> Pavel Herrmann

the patch included in the last mail had a copy/paste error, this one is fixed

Pavel Herrmann

[-- Attachment #2: 0001-libata-pmp-add-support-for-Thermaltake-BlackX-Duet-e.patch --]
[-- Type: text/x-patch, Size: 1268 bytes --]

From 55617331f5d1ef24f7fb60f9640a2363e3649cdf Mon Sep 17 00:00:00 2001
From: Pavel Herrmann <morpheus.ibis@gmail.com>
Date: Thu, 7 Apr 2011 18:15:26 +0200
Subject: [PATCH] libata-pmp: add support for Thermaltake BlackX Duet esata drive dock

some errors still show up, but the dock works, both drives can be
accessed at the same time

the chip maker and designation is unknown - possibly jmicron JMB350?

Signed-off-by: Pavel Herrmann <morpheus.ibis@gmail.com>
---
 drivers/ata/libata-pmp.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/drivers/ata/libata-pmp.c b/drivers/ata/libata-pmp.c
index 3120596..abc8474 100644
--- a/drivers/ata/libata-pmp.c
+++ b/drivers/ata/libata-pmp.c
@@ -449,6 +449,14 @@ static void sata_pmp_quirks(struct ata_port *ap)
 		 * otherwise.  Don't try hard to recover it.
 		 */
 		ap->pmp_link[ap->nr_pmp_links - 1].flags |= ATA_LFLAG_NO_RETRY;
+	} else if (vendor == 0x1095 && devid == 0x4726) {
+		/* chip found in Thermaltake BlackX Duet */
+		ata_for_each_link(link, ap, EDGE) {
+			/* Use same quirks as sil chips	to make it work */
+			link->flags |= ATA_LFLAG_NO_LPM |
+				       ATA_LFLAG_NO_SRST |
+				       ATA_LFLAG_ASSUME_ATA;
+		}
 	}
 }
 
-- 
1.7.4.1

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Tejun Heo]

On Thu, Apr 07, 2011 at 07:08:10PM +0200, Pavel Herrmann wrote:
> +	} else if (vendor == 0x1095 && devid == 0x4726) {
> +		/* chip found in Thermaltake BlackX Duet */
> +		ata_for_each_link(link, ap, EDGE) {
> +			/* Use same quirks as sil chips	to make it work */
> +			link->flags |= ATA_LFLAG_NO_LPM |
> +				       ATA_LFLAG_NO_SRST |
> +				       ATA_LFLAG_ASSUME_ATA;
> +		}

I'm confused.  Against which version is the patch generated?  Upstream
already has quirks for 4726.

Thakns.

-- 
tejun

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

On Thursday 07 of April 2011 21:03:00 Tejun Heo wrote:
> I'm confused.  Against which version is the patch generated?  Upstream
> already has quirks for 4726.
> 
> Thakns.

this is what you get from running and editing a different tree (and being 
lazy), sorry for messing it up.

the correct (tested) code, without beautifications/comments:

        } else if (vendor == 0x197b && devid == 0x2352) {
                ata_for_each_link(link, ap, EDGE) {
                        link->flags |= ATA_LFLAG_NO_LPM |
                                       ATA_LFLAG_NO_SRST |
                                       ATA_LFLAG_ASSUME_ATA;
                }
        }

third time's the charm

Thanks
Pavel Herrmann

PS: just to be on the safe side, this is the device detection from dmesg
ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Jeff Garzik]

On 04/07/2011 03:40 PM, Pavel Herrmann wrote:
> On Thursday 07 of April 2011 21:03:00 Tejun Heo wrote:
>> I'm confused.  Against which version is the patch generated?  Upstream
>> already has quirks for 4726.
>>
>> Thakns.
>
> this is what you get from running and editing a different tree (and being
> lazy), sorry for messing it up.
>
> the correct (tested) code, without beautifications/comments:
>
>          } else if (vendor == 0x197b&&  devid == 0x2352) {
>                  ata_for_each_link(link, ap, EDGE) {
>                          link->flags |= ATA_LFLAG_NO_LPM |
>                                         ATA_LFLAG_NO_SRST |
>                                         ATA_LFLAG_ASSUME_ATA;
>                  }
>          }
>
> third time's the charm
>
> Thanks
> Pavel Herrmann
>
> PS: just to be on the safe side, this is the device detection from dmesg
> ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0

So you're gonna resend this as a real patch, right?  :)

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

[-- Attachment #1: Type: Text/Plain, Size: 353 bytes --]

On Friday 08 of April 2011 06:58:45 Jeff Garzik wrote:
> So you're gonna resend this as a real patch, right?  :)

right :)

this is the tested version (call it V2 if you wish), against 29-rc2 and 28-
gentoo-r1 (no difference in this file), works in both

I still have no idea what those quirks really do, so test before including

thanks
Pavel Herrmann

[-- Attachment #2: 0001-libata-pmp-add-support-for-Thermaltake-BlackX-Duet-e.patch --]
[-- Type: text/x-patch, Size: 1344 bytes --]

From a7b09f275faca338805fd697ab41fd13d7f21d41 Mon Sep 17 00:00:00 2001
From: Pavel Herrmann <morpheus.ibis@gmail.com>
Date: Thu, 7 Apr 2011 18:15:26 +0200
Subject: [PATCH] libata-pmp: add support for Thermaltake BlackX Duet esata drive dock

some errors still show up, but the dock works, both drives can be
accessed at the same time

the chip maker and designation is unknown - possibly jmicron JMB350?

Signed-off-by: Pavel Herrmann <morpheus.ibis@gmail.com>
---
 drivers/ata/libata-pmp.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/drivers/ata/libata-pmp.c b/drivers/ata/libata-pmp.c
index 3120596..e6e409d 100644
--- a/drivers/ata/libata-pmp.c
+++ b/drivers/ata/libata-pmp.c
@@ -449,6 +449,16 @@ static void sata_pmp_quirks(struct ata_port *ap)
 		 * otherwise.  Don't try hard to recover it.
 		 */
 		ap->pmp_link[ap->nr_pmp_links - 1].flags |= ATA_LFLAG_NO_RETRY;
+	} else if (vendor == 0x197b && devid == 0x2352) {
+		/* chip found in Thermaltake BlackX Duet, jmicron JMB350? */
+		ata_for_each_link(link, ap, EDGE) {
+			/* Use same quirks as sil chips	to make it work,
+			 * not everything might be necessary though
+			 */
+			link->flags |= ATA_LFLAG_NO_LPM |
+				       ATA_LFLAG_NO_SRST |
+				       ATA_LFLAG_ASSUME_ATA;
+		}
 	}
 }
 
-- 
1.7.4.1

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Tejun Heo]

Hey,

On Fri, Apr 08, 2011 at 10:05:37AM +0200, Pavel Herrmann wrote:
> +	} else if (vendor == 0x197b && devid == 0x2352) {
> +		/* chip found in Thermaltake BlackX Duet, jmicron JMB350? */
> +		ata_for_each_link(link, ap, EDGE) {
> +			/* Use same quirks as sil chips	to make it work,
> +			 * not everything might be necessary though
> +			 */
> +			link->flags |= ATA_LFLAG_NO_LPM |
> +				       ATA_LFLAG_NO_SRST |
> +				       ATA_LFLAG_ASSUME_ATA;
> +		}

Can you please do the followings?

* Post dmesg w/o the patch.

* Try different combinations of the flags and see which ones are
  essential.

Thanks.

-- 
tejun

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

[-- Attachment #1: Type: Text/Plain, Size: 536 bytes --]

On Friday 08 of April 2011 17:59:18 Tejun Heo wrote:
> Hey,
> 
> Can you please do the followings?
> 
> * Post dmesg w/o the patch.
> 
> * Try different combinations of the flags and see which ones are
>   essential.
> 
> Thanks.

Hi

NO_LPM seems to be unnecessary, but as I don't do link PM on my desktop I 
wouldn't drop it (when all other quirky PMPs need it), unless someone else 
with a laptop (or using LPM on desktop) confirms it.

dmesgs attached (ata8 is JMB363, ata6 is ICH10R), filenames are self-expanatory

Pavel Herrmann

[-- Attachment #2: pmp_ata6_no_patch --]
[-- Type: text/plain, Size: 3721 bytes --]

[    1.228883] ata6: SATA max UDMA/133 abar m2048@0xf7cfc000 port 0xf7cfc380 irq 70
[    5.659250] ata6: SATA link down (SStatus 0 SControl 300)
[  116.532810] ata6: exception Emask 0x10 SAct 0x0 SErr 0x4050000 action 0xe frozen
[  116.532814] ata6: irq_stat 0x00400040, connection status changed
[  116.532819] ata6: hard resetting link
[  117.425467] ata6: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  117.425790] ata6.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0
[  117.425793] ata6.15: Asynchronous notification not supported, hotplug won't
[  117.426805] ata6.00: hard resetting link
[  117.917853] ata6.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[  117.917895] ata6.01: hard resetting link
[  117.925268] ata6.01: failed to read SCR 2 (Emask=0x40)
[  117.925271] ata6.01: COMRESET failed (errno=-5)
[  117.925273] ata6.01: failed to read SCR 0 (Emask=0x40)
[  117.925275] ata6.01: reset failed, giving up
[  117.925279] ata6.15: hard resetting link
[  118.417107] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  122.915605] ata6.00: hard resetting link
[  123.407724] ata6.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[  123.407766] ata6.01: hard resetting link
[  123.415412] ata6.01: failed to read SCR 2 (Emask=0x40)
[  123.415415] ata6.01: COMRESET failed (errno=-5)
[  123.415418] ata6.01: failed to read SCR 0 (Emask=0x40)
[  123.415420] ata6.01: reset failed, giving up
[  123.415423] ata6.15: hard resetting link
[  123.907274] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  128.405776] ata6.00: hard resetting link
[  128.898131] ata6.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[  128.898173] ata6.01: hard resetting link
[  128.905558] ata6.01: failed to read SCR 2 (Emask=0x40)
[  128.905561] ata6.01: COMRESET failed (errno=-5)
[  128.905563] ata6.01: failed to read SCR 0 (Emask=0x40)
[  128.905565] ata6.01: reset failed, giving up
[  128.905567] ata6.01: failed to recover link after 3 tries, disabling
[  128.905571] ata6.15: hard resetting link
[  129.397396] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  129.717399] ata6.01: failed to read SCR 0 (Emask=0x100)
[  129.717402] ata6.01: failed to write SCR 1 (Emask=0x40)
[  129.717405] ata6.01: failed to clear SError.N (errno=-5)
[  129.717408] ata6.15: hard resetting link
[  130.209125] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  130.529123] ata6.01: failed to read SCR 0 (Emask=0x100)
[  130.529127] ata6.01: failed to write SCR 1 (Emask=0x40)
[  130.529129] ata6.01: failed to clear SError.N (errno=-5)
[  130.529132] ata6.15: hard resetting link
[  131.020901] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  131.341005] ata6.01: failed to write SCR 1 (Emask=0x100)
[  131.341008] ata6.01: COMRESET failed (errno=-5)
[  131.341011] ata6.01: failed to write SCR 1 (Emask=0x40)
[  131.341013] ata6.01: failed to clear SError.N (errno=-5)
[  131.341016] ata6.15: hard resetting link
[  131.832599] ata6.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  132.152732] ata6.01: failed to write SCR 1 (Emask=0x100)
[  132.152735] ata6.01: COMRESET failed (errno=-5)
[  132.152738] ata6.01: failed to write SCR 1 (Emask=0x40)
[  132.152740] ata6.01: failed to clear SError.N (errno=-5)
[  132.152742] ata6: failed to recover PMP after 5 tries, giving up
[  132.152744] ata6.15: Port Multiplier detaching
[  132.152773] ata6.00: disabled
[  132.152781] ata6: exception Emask 0x2 SAct 0x0 SErr 0x0 action 0x6 frozen t4
[  132.152783] ata6: irq_stat 0x00800001, incorrect PMP
[  132.152787] ata6: hard resetting link
[  132.644353] ata6: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[  132.644362] ata6: EH complete

[-- Attachment #3: pmp_ata8_no_ASSUME_ATA --]
[-- Type: text/plain, Size: 3497 bytes --]

[    1.225913] ata8: SATA max UDMA/133 abar m8192@0xfbbfe000 port 0xfbbfe180 irq 16
[    1.720762] ata8: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    1.721527] ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0
[    1.721736] ata8.15: Asynchronous notification not supported, hotplug won't
[    1.724861] ata8.00: hard resetting link
[    1.736581] ata8.00: failed to read SCR 2 (Emask=0x40)
[    1.736710] ata8.00: COMRESET failed (errno=-5)
[    1.736835] ata8.00: failed to read SCR 0 (Emask=0x40)
[    1.736960] ata8.00: reset failed, giving up
[    1.737085] ata8.15: hard resetting link
[    2.232427] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    6.722894] ata8.00: hard resetting link
[    7.043180] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[    7.043353] ata8.00: link online but device misclassifed
[    7.043355] ata8.00: link online but 1 devices misclassified, retrying
[    7.043528] ata8.00: reset failed (errno=-11), retrying in 10 secs
[   16.719519] ata8.00: hard resetting link
[   17.040278] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   17.040454] ata8.00: link online but device misclassifed
[   17.040456] ata8.00: link online but 1 devices misclassified, retrying
[   17.040630] ata8.00: reset failed (errno=-11), retrying in 10 secs
[   26.716136] ata8.00: hard resetting link
[   30.035033] ata8.15: qc timeout (cmd 0xe4)
[   30.035169] ata8.00: failed to read SCR 0 (Emask=0x4)
[   30.035295] ata8.00: SATA link down (SStatus 123 SControl 320)
[   30.035425] ata8.00: failed to read SCR 0 (Emask=0x40)
[   30.035551] ata8.00: failed to read SCR 0 (Emask=0x40)
[   30.035676] ata8.00: link status unknown, clearing UNKNOWN to NONE
[   30.035680] ata8.01: hard resetting link
[   30.035802] ata8.01: failed to read SCR 2 (Emask=0x40)
[   30.035928] ata8.01: failed to read SCR 2 (Emask=0x40)
[   30.036054] ata8.01: COMRESET failed (errno=-5)
[   30.036178] ata8.01: failed to read SCR 0 (Emask=0x40)
[   30.036304] ata8.01: reset failed, giving up
[   30.036427] ata8.15: hard resetting link
[   30.694806] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   35.033338] ata8.01: hard resetting link
[   35.353611] ata8.01: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   35.353785] ata8.01: link online but device misclassifed
[   35.353788] ata8.01: link online but 1 devices misclassified, retrying
[   35.353960] ata8.01: reset failed (errno=-11), retrying in 10 secs
[   45.029918] ata8.01: hard resetting link
[   45.350675] ata8.01: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   45.350851] ata8.01: link online but device misclassifed
[   45.350854] ata8.01: link online but 1 devices misclassified, retrying
[   45.351027] ata8.01: reset failed (errno=-11), retrying in 10 secs
[   55.026566] ata8.01: hard resetting link
[   55.347265] ata8.01: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   55.347441] ata8.01: link online but device misclassifed
[   55.347444] ata8.01: link online but 1 devices misclassified, retrying
[   55.347617] ata8.01: reset failed (errno=-11), retrying in 35 secs
[   90.014841] ata8.01: limiting SATA link speed to 1.5 Gbps
[   90.014970] ata8.01: hard resetting link
[   90.335037] ata8.01: SATA link up 1.5 Gbps (SStatus 113 SControl 310)
[   90.335210] ata8.01: link online but device misclassifed
[   90.335213] ata8.01: link online but 1 devices misclassified, device detection might fail
[   90.335468] ata8: EH complete


[-- Attachment #4: pmp_ata8_no_NO_LPM --]
[-- Type: text/plain, Size: 1791 bytes --]

[    1.237343] ata8: SATA max UDMA/133 abar m8192@0xfbbfe000 port 0xfbbfe180 irq 16
[    1.728096] ata8: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    1.728627] ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0
[    1.728834] ata8.15: Asynchronous notification not supported, hotplug won't
[    1.730504] ata8.00: hard resetting link
[    1.740080] ata8.00: failed to read SCR 2 (Emask=0x40)
[    1.740209] ata8.00: COMRESET failed (errno=-5)
[    1.740331] ata8.00: failed to read SCR 0 (Emask=0x40)
[    1.740454] ata8.00: reset failed, giving up
[    1.740576] ata8.15: hard resetting link
[    2.231929] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    6.726329] ata8.00: hard resetting link
[    9.961235] ata8.15: qc timeout (cmd 0xe4)
[    9.961370] ata8.00: failed to read SCR 0 (Emask=0x4)
[    9.961496] ata8.00: COMRESET failed (errno=-5)
[    9.961620] ata8.00: failed to read SCR 0 (Emask=0x40)
[    9.961747] ata8.00: reset failed, giving up
[    9.961871] ata8.15: hard resetting link
[   10.509064] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   11.724586] ata8.00: hard resetting link
[   12.045405] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   12.045583] ata8.01: hard resetting link
[   12.365303] ata8.01: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   12.384258] ata8.00: ATA-8: WDC WD10EAVS-00D7B1, 01.01A01, max UDMA/133
[   12.384398] ata8.00: 1953525168 sectors, multi 0: LBA48 NCQ (depth 31/32), AA
[   12.385475] ata8.00: configured for UDMA/133
[   12.722839] ata8.01: ATA-8: WDC WD20EADS-00R6B0, 01.00A01, max UDMA/133
[   12.722972] ata8.01: 3907029168 sectors, multi 0: LBA48 NCQ (depth 31/32), AA
[   12.726652] ata8.01: configured for UDMA/133
[   12.726886] ata8: EH complete


[-- Attachment #5: pmp_ata8_no_NO_SRST --]
[-- Type: text/plain, Size: 2223 bytes --]

[    1.234190] ata8: SATA max UDMA/133 abar m8192@0xfbbfe000 port 0xfbbfe180 irq 16
[    1.728869] ata8: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    1.729462] ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0
[    1.729671] ata8.15: Asynchronous notification not supported, hotplug won't
[    1.731173] ata8.00: hard resetting link
[    2.225408] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[    2.225587] ata8.01: hard resetting link
[    2.232638] ata8.01: failed to read SCR 2 (Emask=0x40)
[    2.232763] ata8.01: COMRESET failed (errno=-5)
[    2.232885] ata8.01: failed to read SCR 0 (Emask=0x40)
[    2.233008] ata8.01: reset failed, giving up
[    2.233129] ata8.15: hard resetting link
[    2.724606] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    7.223025] ata8.00: hard resetting link
[    7.715089] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[    7.715264] ata8.01: hard resetting link
[    7.722779] ata8.01: failed to read SCR 2 (Emask=0x40)
[    7.722906] ata8.01: COMRESET failed (errno=-5)
[    7.723030] ata8.01: failed to read SCR 0 (Emask=0x40)
[    7.723156] ata8.01: reset failed, giving up
[    7.723280] ata8.15: hard resetting link
[    8.214682] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   12.713146] ata8.00: hard resetting link
[   13.205235] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   13.205411] ata8.01: hard resetting link
[   13.212920] ata8.01: failed to read SCR 2 (Emask=0x40)
[   13.213046] ata8.01: COMRESET failed (errno=-5)
[   13.213170] ata8.01: failed to read SCR 0 (Emask=0x40)
[   13.213296] ata8.01: reset failed, giving up
[   13.213419] ata8.01: failed to recover link after 3 tries, disabling
[   13.213548] ata8.15: hard resetting link
[   13.704771] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   18.203293] ata8.00: hard resetting link
[   18.695379] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   18.704380] ata8.00: ATA-8: WDC WD10EAVS-00D7B1, 01.01A01, max UDMA/133
[   18.704512] ata8.00: 1953525168 sectors, multi 0: LBA48 NCQ (depth 31/32), AA
[   18.705595] ata8.00: configured for UDMA/133
[   18.705765] ata8: EH complete

[-- Attachment #6: pmp_ata8_no_patch --]
[-- Type: text/plain, Size: 2599 bytes --]

[    1.245918] ata8: SATA max UDMA/133 abar m8192@0xfbbfe000 port 0xfbbfe180 irq 16
[    1.736595] ata8: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    1.737173] ata8.15: Port Multiplier 1.1, 0x197b:0x2352 r0, 2 ports, feat 0x0/0x0
[    1.737381] ata8.15: Asynchronous notification not supported, hotplug won't
[    1.738946] ata8.00: hard resetting link
[    1.748673] ata8.00: failed to read SCR 2 (Emask=0x40)
[    1.748805] ata8.00: COMRESET failed (errno=-5)
[    1.748927] ata8.00: failed to read SCR 0 (Emask=0x40)
[    1.749050] ata8.00: reset failed, giving up
[    1.749172] ata8.15: hard resetting link
[    2.240433] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[    6.734885] ata8.00: hard resetting link
[    7.226986] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[    7.227162] ata8.01: hard resetting link
[    7.234675] ata8.01: failed to read SCR 2 (Emask=0x40)
[    7.234801] ata8.01: COMRESET failed (errno=-5)
[    7.234926] ata8.01: failed to read SCR 0 (Emask=0x40)
[    7.235051] ata8.01: reset failed, giving up
[    7.235175] ata8.15: hard resetting link
[    7.726522] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   12.225041] ata8.00: hard resetting link
[   12.717590] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   12.717767] ata8.01: hard resetting link
[   12.724822] ata8.01: failed to read SCR 2 (Emask=0x40)
[   12.724948] ata8.01: COMRESET failed (errno=-5)
[   12.725073] ata8.01: failed to read SCR 0 (Emask=0x40)
[   12.725199] ata8.01: reset failed, giving up
[   12.725322] ata8.15: hard resetting link
[   13.216669] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   17.715187] ata8.00: hard resetting link
[   18.207273] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   18.207449] ata8.01: hard resetting link
[   18.214961] ata8.01: failed to read SCR 2 (Emask=0x40)
[   18.215087] ata8.01: COMRESET failed (errno=-5)
[   18.215211] ata8.01: failed to read SCR 0 (Emask=0x40)
[   18.215337] ata8.01: reset failed, giving up
[   18.215460] ata8.01: failed to recover link after 3 tries, disabling
[   18.215590] ata8.15: hard resetting link
[   18.706812] ata8.15: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[   23.205337] ata8.00: hard resetting link
[   23.697419] ata8.00: SATA link up 3.0 Gbps (SStatus 123 SControl 320)
[   23.698413] ata8.00: ATA-8: WDC WD10EAVS-00D7B1, 01.01A01, max UDMA/133
[   23.698544] ata8.00: 1953525168 sectors, multi 0: LBA48 NCQ (depth 31/32), AA
[   23.699593] ata8.00: configured for UDMA/133
[   23.699765] ata8: EH complete

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Tejun Heo]

Hello,

On Fri, Apr 08, 2011 at 07:13:24PM +0200, Pavel Herrmann wrote:
> On Friday 08 of April 2011 17:59:18 Tejun Heo wrote:
> > Hey,
> > 
> > Can you please do the followings?
> > 
> > * Post dmesg w/o the patch.
> > 
> > * Try different combinations of the flags and see which ones are
> >   essential.
> > 
> > Thanks.
> 
> Hi
> 
> NO_LPM seems to be unnecessary, but as I don't do link PM on my desktop I 
> wouldn't drop it (when all other quirky PMPs need it), unless someone else 
> with a laptop (or using LPM on desktop) confirms it.
> 
> dmesgs attached (ata8 is JMB363, ata6 is ICH10R), filenames are self-expanatory

It's sad to see the PMP to require both quirks.  The SIMG ones were
first-gens so somewhat understandable.  Eh well, please feel free to
add "Acked-by: Tejun Heo <tj@kernel.org>".  Thanks.

-- 
tejun

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Pavel Herrmann]

Hi,

On Sunday 10 of April 2011 23:30:20 Tejun Heo wrote:
> It's sad to see the PMP to require both quirks.  The SIMG ones were
> first-gens so somewhat understandable.  Eh well, please feel free to
> add "Acked-by: Tejun Heo <tj@kernel.org>".  Thanks.

that means the patch is OK and queued for future kernel release, or do I have 
to send/submit it elsewere?

sorry, not that familiar with patch submissions

thanks
Pavel Herrmann

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Tejun Heo]

On Thu, Apr 28, 2011 at 9:29 PM, Pavel Herrmann <morpheus.ibis@gmail.com> wrote:
\> On Sunday 10 of April 2011 23:30:20 Tejun Heo wrote:
>> It's sad to see the PMP to require both quirks.  The SIMG ones were
>> first-gens so somewhat understandable.  Eh well, please feel free to
>> add "Acked-by: Tejun Heo <tj@kernel.org>".  Thanks.
>
> that means the patch is OK and queued for future kernel release, or do I have
> to send/submit it elsewere?
>
> sorry, not that familiar with patch submissions

Jeff, can you please pick this one up?

Thanks.

-- 
tejun

Re: [PATCH] support for PMP used in TT BlackX Duet drive dock

[Tejun Heo]

On Thu, Apr 28, 2011 at 9:29 PM, Pavel Herrmann <morpheus.ibis@gmail.com> wrote:
> that means the patch is OK and queued for future kernel release, or do I have
> to send/submit it elsewere?

Ooh, was too quick with reply.  It seems there's no properly formatted
patch.  Please regenerate the patch with your Signed-off-by and my
Acked-by added and send it to Jeff and cc me and this mailing list.

Thanks.

-- 
tejun