[PATCH] net/wireless: add debugfs exit function
z
zhaojunkui2008 at 126.com
Thu Apr 21 23:37:55 PDT 2022
Hi Kalle Valo:
>-----邮件原件-----
>发件人: Kalle Valo <kvalo at kernel.org>
>发送时间: 2022年4月22日 13:57
>收件人: Bernard Zhao <zhaojunkui2008 at 126.com>
>抄送: Jakub Kicinski <kubakici at wp.pl>; David S. Miller <davem at davemloft.net>; Paolo Abeni <pabeni at redhat.com>; Matthias Brugger <matthias.bgg at gmail.com>; linux-wireless at vger.kernel.org; netdev at vger.kernel.org; >linux-arm-kernel at lists.infradead.org; linux-mediatek at lists.infradead.org; linux-kernel at vger.kernel.org; 赵军奎 <bernard at vivo.com>
>主题: Re: [PATCH] net/wireless: add debugfs exit function
>Bernard Zhao <zhaojunkui2008 at 126.com> writes:
>> This patch add exit debugfs function to mt7601u.
>> Debugfs need to be cleanup when module is unloaded or load fail.
>"load fail"? Please be more specific, are you saying that the second module load fails or what?
For this part, there are two cases:
First when mt7601u is loaded, in function mt7601u_probe, if function mt7601u_probe run into error lable err_hw, mt7601u_cleanup didn`t cleanup the debugfs node.
Second when the module disconnect, in function mt7601u_disconnect, mt7601u_cleanup didn`t cleanup the debugfs node.
I think these are the mt7601u unloaded or load fail cases, but both with no debugfs cleanup work.
>> drivers/net/wireless/mediatek/mt7601u/debugfs.c | 9 +++++++--
>> drivers/net/wireless/mediatek/mt7601u/init.c | 1 +
>> drivers/net/wireless/mediatek/mt7601u/mt7601u.h | 1 +
>The title should be:
>mt7601u: add debugfs exit function
Got it, thanks!
>> --- a/drivers/net/wireless/mediatek/mt7601u/debugfs.c
>> +++ b/drivers/net/wireless/mediatek/mt7601u/debugfs.c
>> @@ -9,6 +9,8 @@
>> #include "mt7601u.h"
>> #include "eeprom.h"
>>
>> +static struct dentry *dir;
>How will this work when there are multiple mt7601u devices? Because of that, avoid using non-const static variables.
Sorry for missing this part, I understand that the better way is to manage it in the struct of the matched device, I would fix this in the next patch.
Thank you very much!
BR//Bernard
>--
>https://patchwork.kernel.org/project/linux-wireless/list/
>https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
More information about the Linux-mediatek
mailing list