[azure] libata: apply MAX_SEC_1024 to all LITEON EP1 series devices

Message ID 1517487922-19869-1-git-send-email-marcelo.cerri@canonical.com
State New
Headers show
Series
  • [azure] libata: apply MAX_SEC_1024 to all LITEON EP1 series devices
Related show

Commit Message

Marcelo Henrique Cerri Feb. 1, 2018, 12:25 p.m.
From: Xinyu Lin <xinyu0123@gmail.com>

BugLink: http://bugs.launchpad.net/bugs/1743053

LITEON EP1 has the same timeout issues as CX1 series devices.

Revert max_sectors to the value of 1024.

'e0edc8c54646 ("libata: apply MAX_SEC_1024 to all CX1-JB*-HP devices")'

Signed-off-by: Xinyu Lin <xinyu0123@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: stable@vger.kernel.org
(cherry picked from commit db5ff909798ef0099004ad50a0ff5fde92426fd1)
Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri@canonical.com>
---
 drivers/ata/libata-core.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Colin King Feb. 1, 2018, 1:05 p.m. | #1
On 01/02/18 12:25, Marcelo Henrique Cerri wrote:
> From: Xinyu Lin <xinyu0123@gmail.com>
> 
> BugLink: http://bugs.launchpad.net/bugs/1743053
> 
> LITEON EP1 has the same timeout issues as CX1 series devices.
> 
> Revert max_sectors to the value of 1024.
> 
> 'e0edc8c54646 ("libata: apply MAX_SEC_1024 to all CX1-JB*-HP devices")'
> 
> Signed-off-by: Xinyu Lin <xinyu0123@gmail.com>
> Signed-off-by: Tejun Heo <tj@kernel.org>
> Cc: stable@vger.kernel.org
> (cherry picked from commit db5ff909798ef0099004ad50a0ff5fde92426fd1)
> Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri@canonical.com>
> ---
>  drivers/ata/libata-core.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
> index 1945a8ea2099..9cb865b06d78 100644
> --- a/drivers/ata/libata-core.c
> +++ b/drivers/ata/libata-core.c
> @@ -4439,6 +4439,7 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
>  	 * https://bugzilla.kernel.org/show_bug.cgi?id=121671
>  	 */
>  	{ "LITEON CX1-JB*-HP",	NULL,		ATA_HORKAGE_MAX_SEC_1024 },
> +	{ "LITEON EP1-*",	NULL,		ATA_HORKAGE_MAX_SEC_1024 },
>  
>  	/* Devices we expect to fail diagnostics */
>  
> 
Upstream cherry pick, positive test results and limited to specific
device. So..

Acked-by: Colin Ian King <colin.king@canonical.com>
Thadeu Lima de Souza Cascardo Feb. 1, 2018, 1:07 p.m. | #2
Acked-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>

Patch

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index 1945a8ea2099..9cb865b06d78 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4439,6 +4439,7 @@  static const struct ata_blacklist_entry ata_device_blacklist [] = {
 	 * https://bugzilla.kernel.org/show_bug.cgi?id=121671
 	 */
 	{ "LITEON CX1-JB*-HP",	NULL,		ATA_HORKAGE_MAX_SEC_1024 },
+	{ "LITEON EP1-*",	NULL,		ATA_HORKAGE_MAX_SEC_1024 },
 
 	/* Devices we expect to fail diagnostics */