libata: apply MAX_SEC_1024 to all LITEON EP1 series devices
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| linux (Ubuntu) |
Fix Released
|
Medium
|
Joseph Salisbury | ||
| Xenial |
Fix Released
|
Medium
|
Joseph Salisbury | ||
| Artful |
Fix Released
|
Medium
|
Joseph Salisbury | ||
| Bionic |
Fix Released
|
Medium
|
Joseph Salisbury | ||
| linux-azure (Ubuntu) |
Fix Committed
|
Undecided
|
Marcelo Cerri | ||
| Xenial |
Fix Released
|
Undecided
|
Marcelo Cerri | ||
| Bionic |
Fix Committed
|
Undecided
|
Marcelo Cerri | ||
Bug Description
== SRU Justification ==
Some SSD drives made by LiteOn can crash the kernel due to bad drivers. This is a
regression introduced by commit e0edc8c54646. This regression is fixed by mainline
commit db5ff909798, which was added to mailine in v4.15-rc9.
This commit has already been added to Bionic via upstream stable udpates. Xenial
also needs this fix because it recieved the regression introducing commit e0edc8c54646
via the 4.4.48 stable updates(See bug lp1663657).
== Fix ==
commit db5ff909798ef00
Author: Xinyu Lin <email address hidden>
Date: Sun Dec 17 20:13:39 2017 +0800
libata: apply MAX_SEC_1024 to all LITEON EP1 series devices
== Regression Potential ==
Low. This commit was also cc'd to upstream stable to fix a regression, so it
received extra upstream review.
== Test Case ==
A test kernel was built with this patch and tested by the original bug reporter.
The bug reporter states the test kernel resolved the bug.
== Original Bug Description ==
Some SSD drives made by LiteOn can crash the kernel due to bad drivers. A fix for this issue has been submitted upstream: https:/
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 <email address hidden>
Signed-off-by: Tejun Heo <email address hidden>
Cc: <email address hidden>
At a high level the root cause of this problem is that the device only supports commands with a transfer size up to 512KB. This worked fine with older versions of Linux but newer versions are attempting commands with a transfer size of 1280KB which is problematic for this device (LITEON EP1-KB480)
CVE References
| Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote Missing required logs. | #1 |
| Changed in linux (Ubuntu): | |
| status: | New → Incomplete |
| Stephen A. Zarkos (stevez) wrote | #2 |
| Changed in linux (Ubuntu): | |
| status: | Incomplete → Confirmed |
| Changed in linux (Ubuntu): | |
| importance: | Undecided → Medium |
| status: | Confirmed → Triaged |
| Changed in linux (Ubuntu Artful): | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| tags: | added: kernel-da-key |
| Joseph Salisbury (jsalisbury) wrote | #3 |
| Changed in linux (Ubuntu Artful): | |
| status: | Triaged → In Progress |
| Changed in linux (Ubuntu Bionic): | |
| status: | Triaged → In Progress |
| Changed in linux (Ubuntu Artful): | |
| assignee: | nobody → Joseph Salisbury (jsalisbury) |
| Changed in linux (Ubuntu Bionic): | |
| assignee: | nobody → Joseph Salisbury (jsalisbury) |
| Stephen A. Zarkos (stevez) wrote | #4 |
| Joseph Salisbury (jsalisbury) wrote | #5 |
| Stephen A. Zarkos (stevez) wrote | #6 |
| Changed in linux (Ubuntu Xenial): | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Joseph Salisbury (jsalisbury) |
| Changed in linux-azure (Ubuntu Xenial): | |
| assignee: | nobody → Marcelo Cerri (mhcerri) |
| status: | New → In Progress |
| status: | In Progress → Fix Committed |
| Changed in linux (Ubuntu Bionic): | |
| status: | In Progress → Fix Committed |
| Launchpad Janitor (janitor) wrote | #8 |
| Changed in linux-azure (Ubuntu Xenial): | |
| status: | Fix Committed → Fix Released |
| Stephen A. Zarkos (stevez) wrote | #9 |
| Joseph Salisbury (jsalisbury) wrote | #10 |
| Changed in linux (Ubuntu Artful): | |
| status: | In Progress → Fix Committed |
| Changed in linux (Ubuntu Xenial): | |
| status: | In Progress → Fix Committed |
| Kleber Sacilotto de Souza (kleber-souza) wrote | #11 |
| tags: | added: verification-needed-artful |
| tags: | added: verification-needed-xenial |
| Kleber Sacilotto de Souza (kleber-souza) wrote | #12 |
| Launchpad Janitor (janitor) wrote | #13 |
| Changed in linux (Ubuntu Artful): | |
| status: | Fix Committed → Fix Released |
| Launchpad Janitor (janitor) wrote | #14 |
| Changed in linux (Ubuntu Xenial): | |
| status: | Fix Committed → Fix Released |
| tags: |
added: verification-done-artful verification-done-xenial removed: verification-needed-artful verification-needed-xenial |
| Changed in linux-azure (Ubuntu Bionic): | |
| status: | New → Fix Committed |
| assignee: | nobody → Marcelo Cerri (mhcerri) |
| no longer affects: | linux-azure (Ubuntu Artful) |
| Changed in linux-azure (Ubuntu Bionic): | |
| status: | Fix Committed → Fix Released |
| status: | Fix Released → Fix Committed |
| Changed in linux (Ubuntu Bionic): | |
| status: | Fix Committed → Fix Released |
This bug is missing log files that will aid in diagnosing the problem. While running an Ubuntu kernel (not a mainline or third-party kernel) please enter the following command in a terminal window:
apport-collect 1743053
and then change the status of the bug to 'Confirmed'.
If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.
This change has been made by an automated script, maintained by the Ubuntu Kernel Team.