【7】net phy流程和Oops定位方式

一时失言乱红尘 2022-04-02 11:08 283阅读 0赞

1、内核Oops了  
遇到一个Oops的问题 ，发现phy\_attached\_print发送了Oops，死的原因是访问了NULL指针偏移0x88的地方

Oops的原始日志：

![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70]  
2、反汇编vmlinux

反汇编kernel后发现phy\_attached\_print+0x90的地址对应ffffff80084ad2b8，这句汇编的意思就是把X1+136(0x88)的值赋值给X2

从上面call trace也可以看到X1确实为NULL的，导致Oops的地址是0x88，和X1+0x88是对应起来的，那么现在的问题就是定位为什么X1变成了NULL  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 1]  
对应的C代码，函数phy\_attached\_print，从上下文分析phydev肯定不是空的，那么phy->drv的嫌疑就比较大了，至于是不是phy->drv为NULL，可以通过分析偏移确定。  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 2]  
3、gdb确认偏移  
使用GDB工具print一下phydev->drv->name，看到name对应的偏移就是0x88，可以确认phy->drv是NULL  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 3]  
4、分析流程  
看了看日志上下文也没有什么异常，为什么phydev→drv会为NULL，下面是net phy的driver的流程，可以看到

只要get\_phy\_id从reg获取的phy\_id和调用 phy\_drivers\_register 注册的driver中的phy\_id不匹配，换句话说就是reg中读取的phy\_id没有对应的驱动注册，那么phy\_attached\_print就会Oops

(眨眼)net phy的流程如下  
CSDN上传附件非常不方便，见我的下载里面  
![在这里插入图片描述][20181226144919344.png]

5、打桩复现  
既然流程分析清楚了，我们把broadcom BCM50612E phy\_driver的phyid改成一个不合法的值，人为制造reg获取的phy\_id和调用 phy\_drivers\_register 注册的driver中的phy\_id不匹配的场景（从寄存器获取的phyid非法不好构造，但是我们可以修改代码中期望的phyid，只要两者不同就可以了）。

果然Oops了，并且Oops的地址和call trace的打印和前面的一致。说明某种情况下，比如收到干扰或者链路linkdown，如果从寄存器读取的phyid和代码中的phyid不匹配，就会出现Oops  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 4]  
6、kernel主线怎么修复？  
既然这么明显一个bug，其他人应该早就遇到了，看了下最新的内核代码，果然这个bug已经修复了，修复方法竟然是判NULL，而不是reg获取的phy\_id和调用 phy\_drivers\_register 注册的driver中的phy\_id不匹配认为probe失败

内核的patch如下

patch 1、修复空指针

https://lore.kernel.org/patchwork/patch/824257/

patch 2、删除deadcode

https://lore.kernel.org/patchwork/patch/825029/  
![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 5]

[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70]: /images/20220402/77039b1649e74079a2c866fd318a3881.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 1]: /images/20220402/dcce47ca78734fe8b5e7fb33702284c6.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 2]: /images/20220402/fce7d72ff86f4d658e7a0af8ab2cfb2c.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 3]: /images/20220402/ddbaed40bcfd4d4b8e3ea48a46cd5f45.png
[20181226144919344.png]: /images/20220402/46564b29a6c3481886e0f011cde7daea.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 4]: /images/20220402/484a34a8f55f43edacb33b61af9fc1ad.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L2xpbmppYXNlbg_size_16_color_FFFFFF_t_70 5]: /images/20220402/39c65b0b2de54a0fad342c96e51bd733.png