[5.10][bugfix]修复修改PCIe热插拔点灯顺序导致的问题 · Pull Request !3302 · anolis/cloud-kernel

bugzilla: https://bugzilla.openanolis.cn/show_bug.cgi?id=8740

修复修改PCIe热插拔点灯顺序导致的问题。
（1）该问题发生在pcie按键热拔事件中，由于remove_board函数中先执行pciehp_set_indicators设置电源指示灯熄灭，再执行pciehp_unconfigure_device进行卸载磁盘。考虑到某些外设卸载时间较长（需要5-9秒），因此电源指示灯熄灭后立即拔盘，就会因为OS内卸载流程未完成导致如下问题：
（a）按键热拔时，按键后，电源指示灯闪烁5秒熄灭，此时立即拔盘。系统日志会报错“PCIe Bus Error”，导致OS内卸载进程阻塞在i40e_remove函数，这时候卸载进程还未释放pci互斥锁（mutex）。与此同时，OS内会调用pcie_do_recovery函数尝试恢复，但卸载进程还未释放互斥锁，导致pcie_do_recovery流程中获取互斥锁失败，因此pcie_do_recovery进程也阻塞。
（b）如果继续执行热插操作，同样会因为获取pci互斥锁失败，使插盘初始化进程阻塞在pci_lock_rescan_remove函数，最终导致系统无法加载识别插入的磁盘。

（2）外设卸载耗时的测试数据：
[root@host vtu]# time echo 0 > power
[ 3697.552681] pci 0001:01:00.3: Removing from iommu group 39
[ 3699.780722] pci 0001:01:00.2: Removing from iommu group 37
[ 3701.496712] pci 0001:01:00.1: Removing from iommu group 34
[ 3701.508163] pci 0001:01:00.0: Removed PTP clock
[ 3703.300751] pci 0001:01:00.0: Removing from iommu group 29

real 0m9.044s
user 0m0.000s
sys 0m0.000s

（3）pcie热插拔函数调用流程分析：
a. 先按键热拔操作
pciehp_ist
-->if (events & PCI_EXP_SLTSTA_ABP)
---->pciehp_handle_button_press
-->if (events & DISABLE_SLOT)
---->pciehp_handle_disable_request
------>pciehp_disable_slot
--------> __pciehp_disable_slot
---------->remove_board
------------>pciehp_set_indicators //先设置电源指示灯熄灭
-------------->pciehp_unconfigure_device //然后执行卸载
---------------->pci_lock_rescan_remove
...//---> 拔盘导致pcie bus error, 此时还没有释放pci互斥锁
---------------->pci_unlock_rescan_remove

b. 然后热插操作
pciehp_ist
-->if (events & (PCI_EXP_SLTSTA_PDC | PCI_EXP_SLTSTA_DLLSC))
---->pciehp_handle_presence_or_link_change
------>link_active = pciehp_check_link_active(ctrl)
------>case OFF_STATE
-------->ctrl_info(ctrl, "Slot(%s): Card present\n",slot_name(ctrl));
-------->pciehp_enable_slot
----------> __pciehp_enable_slot
------------>board_added
-------------->pciehp_set_indicators //先设置电源指示灯常亮
-------------->pciehp_configure_device //然后设备初始化
---------------->pci_lock_rescan_remove -----------> 获取pci互斥锁失败
...
---------------->pci_unlock_rescan_remove

(4) 相关补丁对nvme热插拔流程做了优化，更改了remove_board函数中灭灯（pciehp_set_indicators）和卸载（pciehp_unconfigure_device）的顺序，补丁链接为：c58940f
回退上述补丁后，pcie按键热插拔测试验证通过。

Type	Assign personnel	Status
Reviewer	xunlei Guixin Liu	Completed (0/0 )
Tester	小龙 Pass test	Completed (1/1 )

GVPanolis/cloud-kernel

Search

GVP anolis/cloud-kernel