Re: libata: disable LPM for Crucial BX100 SSD drives

[Kai-Heng Feng]

> On 16 Feb 2018, at 6:34 PM, Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
> 
> Hi,
> 
> On 15-02-18 20:00, Kai-Heng Feng wrote:
>> After Laptop Mode Tools starts to use min_power for LPM, a user found
>> out Crucial BX100 SSD can't get mounted.
>> Crucial BX100 SSD drives don't work well with min_power. This also
>> happens to med_power_with_dipm.
>> So let's disable LPM for Crucial BX100 SSD drives.
> 
> First of all thank you for your patch for this.
> 
> Compared to a similar patch which I just submitted for MX100 device
> 2 things stand out:
> 
> 1) The entry my patch adds needed to be above the queued TRIM
> entries because there is a more generic entry matching the SSD
> below, it might be good to group all entries together there.

Sure. Iâll rebase mine on top of yours.
Also the model name âCT500BX100SSD1â doesnât contain âMicronâ, so it
should not match to other quirks...

> 
> 2) Your reporter has a 500GB model and as mentioned in my
> patch I've noticed that all reporters (there are multiple
> for my model) all have the 500GB version, so I think we should
> restrict this to the 500GB version for now ?

My guess is that all of them use the same firmware/controller.
But you are right, for now we should just match this specific
model.

> 
> I also noticed that your reporter reports a firmware
> version of "MU02" and so does mine, where as the
> "queued TRIM" blacklist entries all check for a version
> of "MU01", could it be that the non 500GB and 500GB
> versions have different firmware ?  What about the queued
> trim quirks, do we maybe also need that for the MU02 version?

I donât see this SSD model on other Launchpad bugs, so I donât
have the answer.

The user doesnât report any TRIM issue (yet), so I guess LPM quirk
alone is good for now. We can also match MU02 to be even more
restrictive.

Kai-Heng

> 
> Regards,
> 
> Hans
> 
> 
> 
>> BugLink: https://bugs.launchpad.net/bugs/1726930
>> Signed-off-by: Kai-Heng Feng <kai.heng.feng@xxxxxxxxxxxxx>
>> ---
>>  drivers/ata/libata-core.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
>> index 3c09122bf038..577cf820eed4 100644
>> --- a/drivers/ata/libata-core.c
>> +++ b/drivers/ata/libata-core.c
>> @@ -4592,6 +4592,14 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
>>  	{ "WDC WD3000JD-*",		NULL,	ATA_HORKAGE_WD_BROKEN_LPM },
>>  	{ "WDC WD3200JD-*",		NULL,	ATA_HORKAGE_WD_BROKEN_LPM },
>>  +	/*
>> +	 * Crucial BX100 SSD has broken min_power and med_power_with_dipm
>> +	 * support.
>> +	 *
>> +	 * https://bugs.launchpad.net/bugs/1726930
>> +	 */
>> +	{ "CT*BX100SSD1",		NULL,	ATA_HORKAGE_NOLPM },
>> +
>>  	/* End Marker */
>>  	{ }
>>  };