Patchwork [Quantal,SRU,PULL] fix Marvell SATA controller hang

login
register
mail settings
Submitter Kamal Mostafa
Date March 25, 2013, 6:30 p.m.
Message ID <E1UKCAM-0003U1-68@fourier>
Download mbox
Permalink /patch/230961/
State New
Headers show

Pull-request

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

Comments

Kamal Mostafa - March 25, 2013, 6:30 p.m.
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 7a8612c419ba2bb12e8c4cbfb2c6cebdf183badd:

  brcmsmac: fix mismatch in number of custom regulatory rules (2013-03-20 06:25:07 -0600)

are available in the git repository at:

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

for you to fetch changes up to 399161011d4d9d63c7f0b31e4bc7e730e9d71828:

  UBUNTU: SAUCE: PCI: fix system hang issue of Marvell SATA host controller (2013-03-25 11:18:30 -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(+)
Tim Gardner - March 25, 2013, 6:38 p.m.