mbox

[Precise,SRU,PULL] fix Marvell SATA controller hang

Message ID E1UKCAC-0003Tr-AQ@fourier
State New
Headers show

Pull-request

git://kernel.ubuntu.com/kamal/ubuntu-precise.git marvell-sata

Message

Kamal Mostafa March 25, 2013, 6:30 p.m. UTC 
SRU Justification

Impact:
Ubuntu 12.04.2 causes the system to be hung after running command udevadm info --attribute-walk --path=/sys/devices/pci0000:00/0000:00:01.0/0000:01:00.0/ata1/ata_port/ata1 | sed -n 's/ ATTR{\(.*\)}=="\(.*\)"/A: \1=\2/p'
for sata controller. Sata controller in question is Marvel 9125

Issue can be reproduce by running checkbox certification on Atom cartridge with MArvell sata controller.
There is patch provided by MArvell and submitted upstream but not yet accepted.
Here are the discussions happened so far.

https://patchwork.kernel.org/patch/2232391/
 http://www.spinics.net/lists/linux-pci/msg20596.html

Canonical can fix in kernel once patch has been accepted upstream. Patch details are as follows
http://comments.gmane.org/gmane.linux.kernel.pci/20678

Test Case:
See above.

---------- >8 ------------------------------------------------------

The following changes since commit f50fbb1c473d9242eea0936744e0f629ea4b6245:

  UBUNTU: SAUCE: Revert "[PATCH] UBUNTU: SAUCE: xen/pv-spinlock: Never enable interrupts in xen_spin_lock_slow()" (2013-03-25 08:24:48 -0400)

are available in the git repository at:

  git://kernel.ubuntu.com/kamal/ubuntu-precise.git marvell-sata

for you to fetch changes up to ee9c1e4813586568fe4c02d09203604fc438ee70:

  UBUNTU: SAUCE: PCI: fix system hang issue of Marvell SATA host controller (2013-03-25 11:23:33 -0700)

----------------------------------------------------------------
Xiangliang Yu (2):
      UBUNTU: SAUCE: PCI: define macro for marvell vendor ID
      UBUNTU: SAUCE: PCI: fix system hang issue of Marvell SATA host controller

 drivers/pci/quirks.c    |   15 +++++++++++++++
 include/linux/pci_ids.h |    1 +
 2 files changed, 16 insertions(+)

Comments

Tim Gardner March 25, 2013, 6:36 p.m. UTC | #1
Brad Figg March 25, 2013, 6:37 p.m. UTC | #2 
On 03/25/2013 11:30 AM, Kamal Mostafa wrote:
> SRU Justification
> 
> Impact:
> Ubuntu 12.04.2 causes the system to be hung after running command udevadm info --attribute-walk --path=/sys/devices/pci0000:00/0000:00:01.0/0000:01:00.0/ata1/ata_port/ata1 | sed -n 's/ ATTR{\(.*\)}=="\(.*\)"/A: \1=\2/p'
> for sata controller. Sata controller in question is Marvel 9125
> 
> Issue can be reproduce by running checkbox certification on Atom cartridge with MArvell sata controller.
> There is patch provided by MArvell and submitted upstream but not yet accepted.
> Here are the discussions happened so far.
> 
> https://patchwork.kernel.org/patch/2232391/
>  http://www.spinics.net/lists/linux-pci/msg20596.html
> 
> Canonical can fix in kernel once patch has been accepted upstream. Patch details are as follows
> http://comments.gmane.org/gmane.linux.kernel.pci/20678
> 
> Test Case:
> See above.
> 
> ---------- >8 ------------------------------------------------------
> 
> The following changes since commit f50fbb1c473d9242eea0936744e0f629ea4b6245:
> 
>   UBUNTU: SAUCE: Revert "[PATCH] UBUNTU: SAUCE: xen/pv-spinlock: Never enable interrupts in xen_spin_lock_slow()" (2013-03-25 08:24:48 -0400)
> 
> are available in the git repository at:
> 
>   git://kernel.ubuntu.com/kamal/ubuntu-precise.git marvell-sata
> 
> for you to fetch changes up to ee9c1e4813586568fe4c02d09203604fc438ee70:
> 
>   UBUNTU: SAUCE: PCI: fix system hang issue of Marvell SATA host controller (2013-03-25 11:23:33 -0700)
> 
> ----------------------------------------------------------------
> Xiangliang Yu (2):
>       UBUNTU: SAUCE: PCI: define macro for marvell vendor ID
>       UBUNTU: SAUCE: PCI: fix system hang issue of Marvell SATA host controller
> 
>  drivers/pci/quirks.c    |   15 +++++++++++++++
>  include/linux/pci_ids.h |    1 +
>  2 files changed, 16 insertions(+)
>