All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luis Henriques <luis.henriques@canonical.com>
To: Ben Hutchings <ben@decadent.org.uk>
Cc: stable <stable@vger.kernel.org>,
	"Martin K. Petersen" <martin.petersen@oracle.com>,
	Tejun Heo <tj@kernel.org>
Subject: Re: [PATCH stable 1/2] libata: Update Crucial/Micron blacklist
Date: Mon, 25 May 2015 13:55:17 +0100	[thread overview]
Message-ID: <20150525125517.GB1887@ares> (raw)
In-Reply-To: <1432248698.12412.43.camel@decadent.org.uk>

On Thu, May 21, 2015 at 11:51:38PM +0100, Ben Hutchings wrote:
> From: "Martin K. Petersen" <martin.petersen@oracle.com>
> Date: Fri, 27 Mar 2015 15:17:20 -0400
> 
> commit ff7f53fb82a7801a778e5902bdbbc5e195ab0de0 upstream.
> 
> Micron has released an updated firmware (MU02) for M510/M550/MX100
> drives to fix the issues with queued TRIM. Queued TRIM remains broken on
> M500 but is working fine on later drives such as M600 and MX200.
> 
> Tweak our blacklist to reflect the above.
> 
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=71371
> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
> Signed-off-by: Tejun Heo <tj@kernel.org>
> [bwh: Backported to 3.16: adjust context and drop ZERO_AFTER_TRIM flags]
> Signed-off-by; Ben Hutchings <ben@decadent.org.uk>
> ---
> This appears to be needed for 3.12-3.18 stable branches, although 3.12.y
> already has the following patch and will therefore need further context
> changes.
>

Thank you, Ben.  I'll queue these 2 patches along with another
backport of commit 9a9324d39696 ("libata: Blacklist queued TRIM on all
Samsung 800-series").

Cheers,
--
Lu�s

> Ben.
> 
>  drivers/ata/libata-core.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> --- a/drivers/ata/libata-core.c
> +++ b/drivers/ata/libata-core.c
> @@ -4225,9 +4225,10 @@ static const struct ata_blacklist_entry
>  
>  	/* devices that don't properly handle queued TRIM commands */
>  	{ "Micron_M500*",		NULL,	ATA_HORKAGE_NO_NCQ_TRIM, },
> -	{ "Crucial_CT???M500SSD*",	NULL,	ATA_HORKAGE_NO_NCQ_TRIM, },
> -	{ "Micron_M550*",		NULL,	ATA_HORKAGE_NO_NCQ_TRIM, },
> -	{ "Crucial_CT*M550SSD*",	NULL,	ATA_HORKAGE_NO_NCQ_TRIM, },
> +	{ "Crucial_CT*M500*",		NULL,	ATA_HORKAGE_NO_NCQ_TRIM, },
> +	{ "Micron_M5[15]0*",		"MU01",	ATA_HORKAGE_NO_NCQ_TRIM, },
> +	{ "Crucial_CT*M550*",		"MU01",	ATA_HORKAGE_NO_NCQ_TRIM, },
> +	{ "Crucial_CT*MX100*",		"MU01",	ATA_HORKAGE_NO_NCQ_TRIM, },
>  
>  	/*
>  	 * Some WD SATA-I drives spin up and down erratically when the link
> 
> -- 
> Ben Hutchings
> I'm not a reverse psychological virus.  Please don't copy me into your sig.



  parent reply	other threads:[~2015-05-25 12:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-21 22:51 [PATCH stable 1/2] libata: Update Crucial/Micron blacklist Ben Hutchings
2015-05-21 22:54 ` [PATCH stable 2/2] libata: Blacklist queued TRIM on Samsung SSD 850 Pro Ben Hutchings
2015-05-26  8:18   ` Jiri Slaby
2015-05-25 12:55 ` Luis Henriques [this message]
2015-05-26  8:18 ` [PATCH stable 1/2] libata: Update Crucial/Micron blacklist Jiri Slaby

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=20150525125517.GB1887@ares \
    --to=luis.henriques@canonical.com \
    --cc=ben@decadent.org.uk \
    --cc=martin.petersen@oracle.com \
    --cc=stable@vger.kernel.org \
    --cc=tj@kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.