[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220719080431.GY2316@kadam>
Date: Tue, 19 Jul 2022 11:04:31 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: Tong Zhang <ztong0001@...il.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Jakub Kicinski <kuba@...nel.org>,
Colin Ian King <colin.king@...el.com>,
Saurav Girepunje <saurav.girepunje@...il.com>,
Nathan Chancellor <nathan@...nel.org>,
Johan Hovold <johan@...nel.org>, linux-kernel@...r.kernel.org,
linux-staging@...ts.linux.dev, Zheyu Ma <zheyuma97@...il.com>
Subject: Re: [PATCH v2 0/3] staging: rtl8192u: fix rmmod warn when wlan0 is
renamed
On Mon, Jul 18, 2022 at 10:50:35PM -0700, Tong Zhang wrote:
> There are 4 debug files created under /proc/net/[Devname]. Devname could
> be wlan0 initially, however it could be renamed later to e.g. enx00e04c000002.
> This will cause problem during debug file teardown since it uses
> netdev->name which is no longer wlan0. To solve this problem, add a
> notifier to handle device renaming.
>
> Also, due to this is purely for debuging as files are created read only,
> move this to debugfs like other NIC drivers do instead of using procfs.
>
> Reported-by: Zheyu Ma <zheyuma97@...il.com>
> Tested-by: Zheyu Ma <zheyuma97@...il.com>
> Signed-off-by: Tong Zhang <ztong0001@...il.com>
>
> v2: break down patch and fix pointer check
>
> Tong Zhang (3):
> staging: rtl8192u: move debug stuff to its own file
> staging: rtl8192u: move debug files to debugfs
> staging: rtl8192u: fix rmmod warn when wlan0 is renamed
Thanks! This is much better.
Reviewed-by: Dan Carpenter <dan.carpenter@...cle.com>
regards,
dan carpenter
Powered by blists - more mailing lists