From patchwork Tue Mar 12 01:00:14 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Mokrejs X-Patchwork-Id: 226740 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 81EF22C02B4 for ; Tue, 12 Mar 2013 12:00:24 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754543Ab3CLBAU (ORCPT ); Mon, 11 Mar 2013 21:00:20 -0400 Received: from fold.natur.cuni.cz ([195.113.57.32]:48417 "HELO fold.natur.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754484Ab3CLBAT (ORCPT ); Mon, 11 Mar 2013 21:00:19 -0400 Received: (qmail 10459 invoked from network); 12 Mar 2013 01:00:16 -0000 Received: from unknown (HELO ?192.168.251.6?) (192.168.251.6) by 192.168.251.1 with SMTP; 12 Mar 2013 01:00:16 -0000 Message-ID: <513E7E1E.80508@fold.natur.cuni.cz> Date: Tue, 12 Mar 2013 02:00:14 +0100 From: Martin Mokrejs User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:19.0) Gecko/20100101 Firefox/19.0 SeaMonkey/2.16 MIME-Version: 1.0 To: "linux-pci@vger.kernel.org" , Bjorn Helgaas , "Rafael J. Wysocki" , Yinghai Lu Subject: 3.9-rc1: pciehp and eSATA card SiI 3132, no XHCI X-Enigmail-Version: 1.5 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org Hi, I cold-booted a computer with the card inserted before power-on and collected lspci output and express card slot status every second. Then I tried eject and hotplug inserts and it appearts the card is not detected, actually maybe pciehp is not even suppoosed to work in this case as the slot was occupied during boot. But let me come to it later. I noted the following on the very first eject of the so far cold-plugged card that lspci properly reports PresDet change on the Changed: line. The only time I ever saw it. ;-) However, for any following inserts and ejects the value is not updated but slot status bits are updated. I had BIOS defaults loaded and additionally, disabled MediaCardReader. The slot status bits read by setpci were changing in this path: card in slot and cold boot 0040 -> eject 0100 hotplug insert -> 0140 eject -> 0100 hotplug insert -> 0140 eject -> 0100 Sadly, although the very first eject was notified the sata_sila24 neither did "release" the device nor did it complain that it suddenly disappeared. That was so far always in dmesg output (like for example): [ 251.632928] sata_sil24 0000:11:00.0: PME# disabled [ 251.636498] sata_sil24: IRQ status == 0xffffffff, PCI fault or device removal? I re-plugged the card and ejected several times while the while loops collecting lspci and slot_status via the setpci shell command harvested data. Still, in dmesg there was nothing logged regarding card insert/eject. Eventually, I tried rmmod sata_sil24 (did NOT need 'rmmod -f') and that did log something new. [ 529.571040] ata8.00: disabled [ 529.576365] sd 7:0:0:0: [sdb] Synchronizing SCSI cache [ 529.577202] sd 7:0:0:0: [sdb] [ 529.577204] Result: hostbyte=DID_BAD_TARGET driverbyte=DRIVER_OK [ 529.577205] sd 7:0:0:0: [sdb] Stopping disk [ 529.577251] sd 7:0:0:0: [sdb] START_STOP FAILED [ 529.577253] sd 7:0:0:0: [sdb] [ 529.577254] Result: hostbyte=DID_BAD_TARGET driverbyte=DRIVER_OK [ 529.578309] sata_sil24: IRQ status == 0xffffffff, PCI fault or device removal? [ 605.818905] sata_sil24 0000:11:00.0: version 1.1 [ 605.818923] sata_sil24 0000:11:00.0: enabling device (0000 -> 0003) [ 605.914722] sata_sil24 0000:11:00.0: failed to clear port RST [ 606.014860] sata_sil24 0000:11:00.0: failed to clear port RST [ 606.014882] sata_sil24 0000:11:00.0: enabling bus mastering [ 606.017741] scsi8 : sata_sil24 [ 606.019388] scsi9 : sata_sil24 [ 606.019898] ata9: SATA max UDMA/100 host m128@0xf6c84000 port 0xf6c80000 irq 19 [ 606.019902] ata10: SATA max UDMA/100 host m128@0xf6c84000 port 0xf6c82000 irq 19 [ 606.115003] ata9: controller in dubious state, performing PORT_RST [ 608.327747] ata9: SATA link down (SStatus FFFFFFFF SControl FFFFFFFF) [ 608.427847] ata10: controller in dubious state, performing PORT_RST [ 610.640619] ata10: SATA link down (SStatus FFFFFFFF SControl FFFFFFFF) Finally, I am tempted to say that pciehp does not work at all under 3.9-rc1 if the slot was occupied during boot. It does NOT contain the usual debug info spanning several lines and it only said: [ 2.109629] pci_hotplug: PCI Hot Plug PCI Core version: 0.5 [ 2.110225] pciehp: pcie_port_service_register = 0 [ 2.110226] pciehp: PCI Express Hot Plug Controller Driver version: 0.4 If that is intentional the driver should complain loudly. Please educate me what those "[virtual] Expansion ROM's" were in lspci above and whether those "[disabled]" memory ranges relate to that as well. If the card is not thought to be present or just not recognized ... why is that printed in lspci at all? Similarly, what can you tell me if lspci output shows only: Latency: 0 or even lacks the whole "Latency" line altoghether, is that also a disabled device? I believe in working cases there is "Latency: 0, Cache Line Size: 64 bytes" instead. Details and raw files at: http://195.113.57.32/~mmokrejs/tmp/3.9-rc1_disabled_MediaCard_reader_eSATA_coldplugged.tar.bz2 To inspect the incremental diffs of lspci output I used: prev='/dev/null'; ls -latr lspci_* | awk '{print $9}' | while read cur; do diff -u -w $prev $cur | less; prev=$cur; done Thank you, Martin --- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- lspci_loop.23:58:50 2013-03-11 23:58:50.000000000 +0100 +++ lspci_loop.23:58:51 2013-03-11 23:58:51.000000000 +0100 @@ -439,7 +439,7 @@ I/O behind bridge: 0000c000-0000dfff Memory behind bridge: f6c00000-f7cfffff Prefetchable memory behind bridge: 00000000f0000000-00000000f10fffff - Secondary status: 66MHz- FastB2B- ParErr- DEVSEL=fast >TAbort- TAbort- Reset- FastB2B- PriDiscTmr- SecDiscTmr- DiscTmrStat- DiscTmrSERREn- Capabilities: [40] Express (v2) Root Port (Slot+), MSI 00 @@ -453,13 +453,13 @@ ClockPM- Surprise- LLActRep+ BwNot- LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+ ExtSynch- ClockPM- AutWidDis- BWInt- AutBWInt- - LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive+ BWMgmt+ ABWMgmt- + LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive- BWMgmt+ ABWMgmt- SltCap: AttnBtn- PwrCtrl- MRL- AttnInd- PwrInd- HotPlug+ Surprise+ Slot #7, PowerLimit 10.000W; Interlock- NoCompl+ SltCtl: Enable: AttnBtn- PwrFlt- MRL- PresDet- CmdCplt- HPIrq- LinkChg- Control: AttnInd Unknown, PwrInd Unknown, Power- Interlock- - SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock- - Changed: MRL- PresDet- LinkState- + SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock- + Changed: MRL- PresDet+ LinkState+ RootCtl: ErrCorrectable- ErrNon-Fatal- ErrFatal- PMEIntEna- CRSVisible- RootCap: CRSVisible- RootSta: PME ReqID 0000, PMEStatus- PMEPending- @@ -478,11 +478,11 @@ Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- Kernel driver in use: pcieport 00: 86 80 1e 1c 07 00 10 00 b5 00 04 06 10 00 81 00 -10: 00 00 00 00 00 00 00 00 00 11 16 00 c0 d0 00 00 +10: 00 00 00 00 00 00 00 00 00 11 16 00 c0 d0 00 20 20: c0 f6 c0 f7 01 f0 01 f1 00 00 00 00 00 00 00 00 30: 00 00 00 00 40 00 00 00 00 00 00 00 0a 04 10 00 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 3c 12 08 -50: 40 00 11 70 60 b2 3c 00 00 00 40 00 00 00 00 00 +50: 40 00 11 50 60 b2 3c 00 00 00 08 01 00 00 00 00 60: 00 00 00 00 16 00 00 00 00 00 00 00 00 00 00 00 70: 01 00 01 00 00 00 00 00 00 00 00 00 00 00 00 00 80: 05 90 00 00 00 00 00 00 00 00 00 00 00 00 00 00 @@ -490,7 +490,7 @@ a0: 01 00 02 c8 00 00 00 00 00 00 00 00 00 00 00 00 b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 -d0: 00 00 00 01 02 0b 00 00 02 80 11 c1 00 00 00 00 +d0: 00 00 00 01 02 0b 00 00 02 80 11 c1 00 00 00 40 e0: 00 03 00 00 00 00 00 00 01 00 00 00 00 00 00 00 f0: 00 00 00 00 00 00 00 00 87 0f 05 08 00 00 00 00 @@ -797,55 +797,23 @@ e0: 00 00 40 63 00 00 00 00 00 00 00 00 00 00 00 00 f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 -11:00.0 Mass storage controller: Silicon Image, Inc. SiI 3132 Serial ATA Raid II Controller (rev 01) - Subsystem: Silicon Image, Inc. SiI 3132 Serial ATA Raid II Controller - Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- - Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- TAbort- Reset- FastB2B- PriDiscTmr- SecDiscTmr- DiscTmrStat- DiscTmrSERREn- Capabilities: [40] Express (v2) Root Port (Slot+), MSI 00 @@ -459,7 +459,7 @@ SltCtl: Enable: AttnBtn- PwrFlt- MRL- PresDet- CmdCplt- HPIrq- LinkChg- Control: AttnInd Unknown, PwrInd Unknown, Power- Interlock- SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock- - Changed: MRL- PresDet- LinkState- + Changed: MRL- PresDet- LinkState+ RootCtl: ErrCorrectable- ErrNon-Fatal- ErrFatal- PMEIntEna- CRSVisible- RootCap: CRSVisible- RootSta: PME ReqID 0000, PMEStatus- PMEPending- @@ -478,11 +478,11 @@ Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- Kernel driver in use: pcieport 00: 86 80 1e 1c 07 00 10 00 b5 00 04 06 10 00 81 00 -10: 00 00 00 00 00 00 00 00 00 11 16 00 c0 d0 00 00 +10: 00 00 00 00 00 00 00 00 00 11 16 00 c0 d0 00 20 20: c0 f6 c0 f7 01 f0 01 f1 00 00 00 00 00 00 00 00 30: 00 00 00 00 40 00 00 00 00 00 00 00 0a 04 10 00 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 3c 12 08 -50: 40 00 11 70 60 b2 3c 00 00 00 40 00 00 00 00 00 +50: 40 00 11 70 60 b2 3c 00 00 00 40 01 00 00 00 00 60: 00 00 00 00 16 00 00 00 00 00 00 00 00 00 00 00 70: 01 00 01 00 00 00 00 00 00 00 00 00 00 00 00 00 80: 05 90 00 00 00 00 00 00 00 00 00 00 00 00 00 00 @@ -799,14 +799,13 @@ 11:00.0 Mass storage controller: Silicon Image, Inc. SiI 3132 Serial ATA Raid II Controller (rev 01) Subsystem: Silicon Image, Inc. SiI 3132 Serial ATA Raid II Controller - Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- + Control: I/O- Mem- BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- SERR-