Re: 3.8.0-rc4+ - Oops on removing WinTV-HVR-1400 expresscard TV Tuner

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Yijing,

On 01/28/13 02:40, Yijing Wang wrote:
Hi Chris,
    Sorry for the delay reply. It seems like my reply last night was missed.

 From the sysinfo you provide, there are no pcie port devices under /sys/bus/pci_express/devices.
Maybe because there are some problems with _OSC in your laptop, so pcie port driver won't create pcie port device
for hotplug, aer and so on.

Maybe you can add boot parameter "pcie_ports=native" and reboot your laptop.
Then use #modprobe pciehp pciehp_force=1 pciehp_debug=1 to load pciehp modules.
After above actions, enter /sys/bus/pci_express/devices/ directory and /sys/bus/pci/slots/
Some slots and pcie port devices should be there now.

Sorry, I've tried your suggestion, but the two directories are still empty.

I verified the test environment as follows:

[chris:~]$ uname -a
Linux laptop 3.7.4 #15 SMP PREEMPT Mon Jan 28 09:43:57 GMT 2013 i686 GNU/Linux
[chris:~]$ grep acpiphp /boot/System.map-3.7.4
[chris:~]$ modinfo acpiphp
modinfo: ERROR: Module acpiphp not found.
[chris:~]$ modinfo pciehp
filename:       /lib/modules/3.7.4/kernel/drivers/pci/hotplug/pciehp.ko
license:        GPL
description:    PCI Express Hot Plug Controller Driver
author: Dan Zink <dan.zink@xxxxxxxxxx>, Greg Kroah-Hartman <greg@xxxxxxxxx>, Dely Sy <dely.l.sy@xxxxxxxxx>
depends:
intree:         Y
vermagic:       3.7.4 SMP preempt mod_unload CORE2
parm:           pciehp_detect_mode:Slot detection mode: pcie, acpi, auto
  pcie          - Use PCIe based slot detection
  acpi          - Use ACPI for slot detection
  auto(default) - Auto select mode. Use acpi option if duplicate
                  slot ids are found. Otherwise, use pcie option
 (charp)
parm:           pciehp_debug:Debugging mode enabled or not (bool)
parm: pciehp_poll_mode:Using polling mechanism for hot-plug events or not (bool) parm: pciehp_poll_time:Polling mechanism frequency, in seconds (int)
parm:           pciehp_force:Force pciehp, even if OSHP is missing (bool)
[chris:~]$ cat /proc/cmdline
root=/dev/sda5 pciehp_ports=native ro resume=/dev/sda6
[chris:~]$ sudo modprobe pciehp pciehp_force=1 pciehp_debug=1
[chris:~]$ lsmod
Module                  Size  Used by
pciehp                 19907  0
[...]

You will notice that the kernel I have used is 3.7.4. I hope that's a suitable kernel for your tests. I've moved away from the 3.8 development kernel onto one that's stable and on which Martin has identified a solution. I see Greg KH released 3.7.5 yesterday and it includes a pciehp change. I'll upgrade to that, run the tests again and report back.

One question - should I include the (acpi) pci_slot driver in the kernel build or does pciehp populate the directories without pci_slot?

Thanks again.

/sys/bus/pci_express/devices:
total 0

/sys/bus/pci_express/drivers:
total 0
drwxr-xr-x 2 root root 0 Jan 27 13:17 pciehp/


On 2013/1/28 6:53, Chris Clayton wrote:
Thanks again, Martin.

Firstly, maybe we should remove the linux-media list from the copy list. I imagine this hotplug stuff is just noise to them.

[snip]
Do you have any other express card around to try if it works at all? Try that always after a cold boot.

Not at the moment, but I ordered at USB3 expresscard yesterday, so I will have one soon.

Posting a diff result of the below procedure might help:

# lspci -vvvxxx > lspci.before_insertion.txt

[plug your card into the slot]

# lspci -vvvxxx > lspci.after_insertion.txt

[ unplug your card]

# lspci -vvvxxx > lspci.after_1st_removal.txt

[re-plug your card into the slot]

# lspci -vvvxxx > lspci.after_1st_re-insertion.txt

[ unplug your card]

# lspci -vvvxxx > lspci.after_2nd_removal.txt


OK, I've been using kernel 3.8.0-rc kernels so far, but given that is still under development, I've switched to 3.7.4, mainly because you are having success with 3.7.x, acpiphp and pcie_aspm=off. I verified the environment as follows:

[chris:~]$ cat /proc/cmdline
root=/dev/sda5 pcie_aspm=off ro resume=/dev/sda6
[chris:~]$ dmesg | grep ASPM
[    0.000000] PCIe ASPM is disabled
[    0.348959]  pci0000:00: ACPI _OSC support notification failed, disabling PCIe ASPM
[chris:~]$ dmesg | grep acpiphp
[    0.400846] acpiphp: ACPI Hot Plug PCI Controller Driver version: 0.5
[chris:~]$ dmesg | grep pciehp
[chris:~]$ uname -a
Linux laptop 3.7.4 #13 SMP PREEMPT Sun Jan 27 18:39:39 GMT 2013 i686 GNU/Linux


Then compare them using diff. These should have no difference:

diff lspci.after_insertion.txt lspci.after_1st_re-insertion.txt
diff lspci.after_1st_removal.txt lspci.after_2nd_removal.txt

Correct, there were no differences.


These may have only little difference, or none:

diff lspci.before_insertion.txt lspci.after_1st_removal.txt

263c263
<               LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <1us, L1 <16us
---
  >               LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
265c265
<               LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk-
---
  >               LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+
267c267
<               LnkSta: Speed 2.5GT/s, Width x0, TrErr- Train- SlotClk+ DLActive- BWMgmt- ABWMgmt-
---
  >               LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive- BWMgmt+ ABWMgmt-
273c273
<                       Changed: MRL- PresDet- LinkState-
---
  >                       Changed: MRL- PresDet- LinkState+
295,296c295,296
< 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 4c 12 04
< 50: 03 00 01 10 60 b2 1c 00 08 00 00 00 00 00 00 00
---
  > 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 3c 12 04
  > 50: 40 00 11 50 60 b2 1c 00 08 00 00 01 00 00 00 00

diff lspci.after_1st_removal.txt lspci.after_2nd_removal.txt

No difference.


Finally, these should confirm whether the PresDet works for you (for me NOT with pciehp but does work with acpiphp).
You should see PresDet- to PresDet+ changes in:

Yes, I do see the PresDet- to PresDet+ changes

diff lspci.before_insertion.txt lspci.after_insertion.txt

263c263
<               LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <1us, L1 <16us
---
  >               LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
265c265
<               LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk-
---
  >               LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+
267c267
<               LnkSta: Speed 2.5GT/s, Width x0, TrErr- Train- SlotClk+ DLActive- BWMgmt- ABWMgmt-
---
  >               LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive+ BWMgmt+ ABWMgmt-
272,273c272,273
<               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock-
<                       Changed: MRL- PresDet- LinkState-
---
  >               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock-
  >                       Changed: MRL- PresDet- LinkState+
295,296c295,296
< 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 4c 12 04
< 50: 03 00 01 10 60 b2 1c 00 08 00 00 00 00 00 00 00
---
  > 40: 10 80 42 01 00 80 00 00 00 00 10 00 12 3c 12 04
  > 50: 40 00 11 70 60 b2 1c 00 08 00 40 01 00 00 00 00

diff lspci.after_1st_removal.txt lspci.after_1st_re-insertion.txt
267c267
<               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-
272c272
<               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock-
---
  >               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock-
296c296
< 50: 40 00 11 50 60 b2 1c 00 08 00 00 01 00 00 00 00
---
  > 50: 40 00 11 70 60 b2 1c 00 08 00 40 01 00 00 00 00


You should see PresDet+ to PresDet- changes in:
Yes, I see those changes too.
diff lspci.after_insertion.txt lspci.after_1st_removal.txt
267c267
<               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-
272c272
<               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock-
---
  >               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock-
296c296
< 50: 40 00 11 70 60 b2 1c 00 08 00 40 01 00 00 00 00
---
  > 50: 40 00 11 50 60 b2 1c 00 08 00 00 01 00 00 00 00

diff lspci.after_1st_re-insertion.txt lspci.after_2nd_removal.txt
267c267
<               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-
272c272
<               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet+ Interlock-
---
  >               SltSta: Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock-
296c296
< 50: 40 00 11 70 60 b2 1c 00 08 00 40 01 00 00 00 00
---
  > 50: 40 00 11 50 60 b2 1c 00 08 00 00 01 00 00 00 00

I did plenty of these with my laptop using 3.3.x and 3.7.1 and the conclusion was
that pciehp got broken since some 3.6? (commit 0d52f54e2ef64c189dedc332e680b2eb4a34590a)
but I can live on 3.7.x with acpiphp and pcie_aspm=off.

I could live with that too, but despite my findings being in line with your predictions, hotplug does not work for me.

The above test could tell you what
works in your case. Of course, you can try separately pciehp and acpiphp. With 3.4
series I lived with pciehp and pcie_aspm=force.
Need sleep now, but tomorrow, I'll build a 3.4 kernel and try pciehp and pcie_aspm=force.

Martin
BTW: Re-post your dmesg output so that we can see if you those OSC errors
(for details see "Re: Dell Vostro 3550: pci_hotplug+acpiphp require 'pcie_aspm=force' on kernel command-line for hotplug to work" thread).

I've attached a file containing the output from dmesg. I does contain errors related to OSC.



--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux