linux-kernel - Re:Re: [PATCH v2] mediatek/mt7601u: add debugfs exit function

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <15b4f.2aad.1805ece06a1.Coremail.zhaojunkui2008@126.com>
Date:   Mon, 25 Apr 2022 11:40:02 +0800 (CST)
From:   z <zhaojunkui2008@....com>
To:     "Jakub Kicinski" <kubakici@...pl>
Cc:     "Kalle Valo" <kvalo@...nel.org>,
        "David S. Miller" <davem@...emloft.net>,
        "Paolo Abeni" <pabeni@...hat.com>,
        "Matthias Brugger" <matthias.bgg@...il.com>,
        linux-wireless@...r.kernel.org, netdev@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org, linux-kernel@...r.kernel.org,
        bernard@...o.com
Subject: Re:Re: [PATCH v2] mediatek/mt7601u: add debugfs exit function


At 2022-04-23 03:47:04, "Jakub Kicinski" <kubakici@...pl> wrote:
>On Fri, 22 Apr 2022 01:08:54 -0700 Bernard Zhao wrote:
>> When mt7601u loaded, 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.
>> This patch add debugfs exit function and try to cleanup debugfs
>> node when mt7601u loaded fail or unloaded.
>> 
>> Signed-off-by: Bernard Zhao <zhaojunkui2008@....com>
>
>Ah, missed that there was a v2. My point stands, wiphy debugfs dir
>should do the cleanup.
>
>Do you encounter problems in practice or are you sending this patches
>based on reading / static analysis of the code only.

Hi Jakub Kicinski:

The issue here is found by reading code.
I read the drivers/net/wireless code and found that many modules are not cleanup the debugfs. 
I sorted out the modules that were not cleaned up the debugfs:
./ti/wl18xx
./ti/wl12xx
./intel/iwlwifi
./intel/iwlwifi
./mediatek/mt76
I am not sure whether this part is welcome to kernel so I submitted a patch.
If you have any suggestions, welcome to put forward for discussion, thank you！

BR//Bernard