public inbox for linux-ide@vger.kernel.org
 help / color / mirror / Atom feed
From: Niklas Cassel <cassel@kernel.org>
To: Yihang Li <liyihang9@h-partners.com>
Cc: Damien Le Moal <dlemoal@kernel.org>, linux-ide@vger.kernel.org
Subject: Re: [PATCH] ata: libata-core: Disable LPM on Silicon Motion MD619HXCLDE3TC
Date: Wed, 26 Nov 2025 13:05:00 +0100	[thread overview]
Message-ID: <aSbs7HOYr_8waEZZ@ryzen> (raw)
In-Reply-To: <5c2c5cbd-b13c-8ba1-86c8-670f0a34fcf3@h-partners.com>

On Wed, Nov 26, 2025 at 04:48:09PM +0800, Yihang Li wrote:
> Hi Niklas,
> 
> On 2025/11/25 20:27, Niklas Cassel wrote:
> > Ok, really nice that we have a patch that fixes your problem.
> > 
> > However, like I wrote in the comment section:
> > 
> > Before we apply this patch, it would be nice if you could try another SSD,
> > that supports LPM (HIPM and DIPM) with your AHCI controller:
> > Huawei Technologies Co., Ltd. HiSilicon AHCI HBA [19e5:a235] (rev 30)
> > 
> > Note: You can see if your drive supports HIPM and DIPM from the
> > ata_dev_print_features() print to dmesg during boot, e.g.:
> > ata6.00: Features: Trust Dev-Sleep HIPM DIPM NCQ-sndrcv NCQ-prio
> > 
> > Just so we can verify that it is the MD619HXCLDE3TC SSD that is not
> > handling LPM correctly, and that it is not the AHCI HBA that is at fault.
> > 
> > E.g. there have been some weird bugs with this AHCI controller before,
> > see e.g. 234e6d2c18f5 ("ata: ahci: Disable SXS for Hisilicon Kunpeng920").
> > 
> > 
> > 
> > Do you have any other drive, other than MD619HXCLDE3TC and MD619GXCLDE3TC,
> > which supports LPM, and you can see that the drive works as intended,
> > with LPM enabled, so we can be certain that it is not the HBA that has
> > broken LPM support?
> > 
> 
> I checked all the drives I own, but unfortunately, none of them support LPM (HIPM and DIPM).
> :(

Well, it should be sufficient that they support either HIPM and DIPM.

Ideally, they should support the same as:
MD619HXCLDE3TC and MD619GXCLDE3TC.
(Not sure if they supported HIPM and DIPM or only one of them).

But the device should work fine to read / write / mount, etc. without showing
the same errors as you got with MD619HXCLDE3TC and MD619GXCLDE3TC.


Kind regards,
Niklas

  reply	other threads:[~2025-11-26 12:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-24 16:34 [PATCH] ata: libata-core: Disable LPM on Silicon Motion MD619HXCLDE3TC Niklas Cassel
2025-11-25 11:31 ` Yihang Li
2025-11-25 12:27   ` Niklas Cassel
2025-11-26  8:48     ` Yihang Li
2025-11-26 12:05       ` Niklas Cassel [this message]
2025-11-28  2:54         ` Yihang Li
2025-11-28  6:01           ` Niklas Cassel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aSbs7HOYr_8waEZZ@ryzen \
    --to=cassel@kernel.org \
    --cc=dlemoal@kernel.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=liyihang9@h-partners.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox