| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <5105AFDB.9000200@googlemail.com>
Date: Sun, 27 Jan 2013 22:53:15 +0000
From: Chris Clayton <chris2553@...glemail.com>
To: Martin Mokrejs <mmokrejs@...d.natur.cuni.cz>
CC: Yijing Wang <wangyijing0307@...il.com>,
linux-media@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
linux-pci@...r.kernel.org
Subject: Re: 3.8.0-rc4+ - Oops on removing WinTV-HVR-1400 expresscard TV Tuner
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.
View attachment "dmesg.txt" of type "text/plain" (42940 bytes)
Powered by blists - more mailing lists