Linux-ide Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
@ 2019-07-14 17:42 Roman Mamedov
  2019-07-15 17:30 ` Martin K. Petersen
  0 siblings, 1 reply; 7+ messages in thread
From: Roman Mamedov @ 2019-07-14 17:42 UTC (permalink / raw)
  To: linux-ide; +Cc: Jens Axboe, stable

My Samsung 860 EVO mSATA 500GB SSD lockups for 20-30 seconds on fstrim, while
dmesg is repeatedly flooded with:

[  332.792044] ata14.00: exception Emask 0x0 SAct 0x3fffe SErr 0x0 action 0x6 frozen
[  332.798271] ata14.00: failed command: SEND FPDMA QUEUED
[  332.804499] ata14.00: cmd 64/01:08:00:00:00/00:00:00:00:00/a0 tag 1 ncq dma 512 out
                        res 40/00:00:00:00:00/00:00:00:00:00/00 Emask 0x4 (timeout)
[  332.817145] ata14.00: status: { DRDY }

Disabling queued TRIM for it, as already done for the 850 series models,
solves the issue completely.

Cc: Jens Axboe <axboe@kernel.dk>
Cc: stable@vger.kernel.org
BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=203475
Signed-off-by: Roman Mamedov <rm@romanrm.net>
---
 drivers/ata/libata-core.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index cbb162b683b6..1fe50b8fe00d 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4566,6 +4566,8 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
 						ATA_HORKAGE_ZERO_AFTER_TRIM, },
 	{ "Samsung SSD 850*",		NULL,	ATA_HORKAGE_NO_NCQ_TRIM |
 						ATA_HORKAGE_ZERO_AFTER_TRIM, },
+	{ "Samsung SSD 860*",		NULL,	ATA_HORKAGE_NO_NCQ_TRIM |
+						ATA_HORKAGE_ZERO_AFTER_TRIM, },
 	{ "FCCT*M500*",			NULL,	ATA_HORKAGE_NO_NCQ_TRIM |
 						ATA_HORKAGE_ZERO_AFTER_TRIM, },
 
-- 
2.11.0

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-14 17:42 [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs Roman Mamedov
@ 2019-07-15 17:30 ` Martin K. Petersen
  2019-07-15 17:42   ` Roman Mamedov
  0 siblings, 1 reply; 7+ messages in thread
From: Martin K. Petersen @ 2019-07-15 17:30 UTC (permalink / raw)
  To: Roman Mamedov; +Cc: linux-ide, Jens Axboe, stable


Roman,

> My Samsung 860 EVO mSATA 500GB SSD lockups for 20-30 seconds on
> fstrim, while dmesg is repeatedly flooded with:

Is that specific to the mSATA model?

FWIW, queued TRIM works fine on the 2.5" form factor. So it would be
best if we could limit the blacklist entry to the mSATA version (or a
particular firmware rev).

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-15 17:30 ` Martin K. Petersen
@ 2019-07-15 17:42   ` Roman Mamedov
  2019-07-16 14:36     ` Martin K. Petersen
  2019-07-19  3:00     ` Martin K. Petersen
  0 siblings, 2 replies; 7+ messages in thread
From: Roman Mamedov @ 2019-07-15 17:42 UTC (permalink / raw)
  To: Martin K. Petersen; +Cc: linux-ide, Jens Axboe, stable

On Mon, 15 Jul 2019 13:30:26 -0400
"Martin K. Petersen" <martin.petersen@oracle.com> wrote:

> 
> Roman,
> 
> > My Samsung 860 EVO mSATA 500GB SSD lockups for 20-30 seconds on
> > fstrim, while dmesg is repeatedly flooded with:
> 
> Is that specific to the mSATA model?
> 
> FWIW, queued TRIM works fine on the 2.5" form factor. So it would be
> best if we could limit the blacklist entry to the mSATA version (or a
> particular firmware rev).

Hello,

I do not have other Samsung (m)SATA models to verify. On the bugreport someone
confirmed this to be an issue for them too. Let's try asking if they have the
mSATA model too, and what firmware revision. Mine is RVT42B6Q and there were
no updates available last time I checked.

-- 
With respect,
Roman

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-15 17:42   ` Roman Mamedov
@ 2019-07-16 14:36     ` Martin K. Petersen
  2019-07-19  3:00     ` Martin K. Petersen
  1 sibling, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2019-07-16 14:36 UTC (permalink / raw)
  To: Roman Mamedov; +Cc: Martin K. Petersen, linux-ide, Jens Axboe, stable


Roman,

> I do not have other Samsung (m)SATA models to verify. On the bugreport
> someone confirmed this to be an issue for them too. Let's try asking
> if they have the mSATA model too, and what firmware revision. Mine is
> RVT42B6Q and there were no updates available last time I checked.

I have an mSATA drive arriving today. I'll see if I can come up with a
suitable heuristic to distinguish between mSATA and the other form
factors.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-15 17:42   ` Roman Mamedov
  2019-07-16 14:36     ` Martin K. Petersen
@ 2019-07-19  3:00     ` Martin K. Petersen
  2019-07-19  5:37       ` Roman Mamedov
  1 sibling, 1 reply; 7+ messages in thread
From: Martin K. Petersen @ 2019-07-19  3:00 UTC (permalink / raw)
  To: Roman Mamedov; +Cc: Martin K. Petersen, linux-ide, Jens Axboe, stable


Roman,

> I do not have other Samsung (m)SATA models to verify. On the bugreport
> someone confirmed this to be an issue for them too. Let's try asking
> if they have the mSATA model too, and what firmware revision. Mine is
> RVT42B6Q and there were no updates available last time I checked.

I have tested two mSATA 860s on two different systems, both with Intel
AHCI controllers, and queued trim works fine for me.

I'll try a few more things tomorrow.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-19  3:00     ` Martin K. Petersen
@ 2019-07-19  5:37       ` Roman Mamedov
  2019-07-22 20:29         ` Martin K. Petersen
  0 siblings, 1 reply; 7+ messages in thread
From: Roman Mamedov @ 2019-07-19  5:37 UTC (permalink / raw)
  To: Martin K. Petersen; +Cc: linux-ide, Jens Axboe, stable

On Thu, 18 Jul 2019 23:00:31 -0400
"Martin K. Petersen" <martin.petersen@oracle.com> wrote:

> I have tested two mSATA 860s on two different systems, both with Intel
> AHCI controllers, and queued trim works fine for me.

What is the firmware version?

Also, do you have an ASMedia ASM1062 controller to try (often seen on
motherboards for additional SATA ports)? That's the one I tested with.

Before tried with AMD chipset ones, but on those the 860s are known[1] to
have serious NCQ issues in general, not just TRIM, so they are not useful for
this test.

With ASMedia only queued TRIM fails and everything else works fine. So I
wonder if 850's queued TRIM issue in 860's case remains only on some SATA
controllers.

[1] https://bugzilla.kernel.org/show_bug.cgi?id=201693

-- 
With respect,
Roman

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs
  2019-07-19  5:37       ` Roman Mamedov
@ 2019-07-22 20:29         ` Martin K. Petersen
  0 siblings, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2019-07-22 20:29 UTC (permalink / raw)
  To: Roman Mamedov; +Cc: Martin K. Petersen, linux-ide, Jens Axboe, stable


Roman,

> What is the firmware version?

RVT41B6Q

> Also, do you have an ASMedia ASM1062 controller to try (often seen on
> motherboards for additional SATA ports)? That's the one I tested with.

I'm afraid not.

> With ASMedia only queued TRIM fails and everything else works fine. So I
> wonder if 850's queued TRIM issue in 860's case remains only on some SATA
> controllers.

Looks likely. In that case we will have to extend the existing trim
heuristic to match on controller as well.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-14 17:42 [PATCH] libata: Disable queued TRIM for Samsung 860 series SSDs Roman Mamedov
2019-07-15 17:30 ` Martin K. Petersen
2019-07-15 17:42   ` Roman Mamedov
2019-07-16 14:36     ` Martin K. Petersen
2019-07-19  3:00     ` Martin K. Petersen
2019-07-19  5:37       ` Roman Mamedov
2019-07-22 20:29         ` Martin K. Petersen

Linux-ide Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-ide/0 linux-ide/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-ide linux-ide/ https://lore.kernel.org/linux-ide \
		linux-ide@vger.kernel.org linux-ide@archiver.kernel.org
	public-inbox-index linux-ide


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-ide


AGPL code for this site: git clone https://public-inbox.org/ public-inbox