* [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
@ 2022-08-25 13:37 Yang Yingliang
2022-08-26 0:44 ` Ping-Ke Shih
0 siblings, 1 reply; 4+ messages in thread
From: Yang Yingliang @ 2022-08-25 13:37 UTC (permalink / raw)
To: linux-kernel, netdev, linux-wireless; +Cc: tony0620emma, kvalo, phhuang
Add the missing destroy_workqueue() before return from rtw_core_init()
in error path.
Fixes: fe101716c7c9 ("rtw88: replace tx tasklet with work queue")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
drivers/net/wireless/realtek/rtw88/main.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/realtek/rtw88/main.c b/drivers/net/wireless/realtek/rtw88/main.c
index 790dcfed1125..557213e52761 100644
--- a/drivers/net/wireless/realtek/rtw88/main.c
+++ b/drivers/net/wireless/realtek/rtw88/main.c
@@ -2094,7 +2094,7 @@ int rtw_core_init(struct rtw_dev *rtwdev)
ret = rtw_load_firmware(rtwdev, RTW_NORMAL_FW);
if (ret) {
rtw_warn(rtwdev, "no firmware loaded\n");
- return ret;
+ goto destroy_workqueue;
}
if (chip->wow_fw_name) {
@@ -2104,11 +2104,15 @@ int rtw_core_init(struct rtw_dev *rtwdev)
wait_for_completion(&rtwdev->fw.completion);
if (rtwdev->fw.firmware)
release_firmware(rtwdev->fw.firmware);
- return ret;
+ goto destroy_workqueue;
}
}
return 0;
+
+destroy_workqueue:
+ destroy_workqueue(rtwdev->tx_wq);
+ return ret;
}
EXPORT_SYMBOL(rtw_core_init);
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
2022-08-25 13:37 [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init() Yang Yingliang
@ 2022-08-26 0:44 ` Ping-Ke Shih
2022-08-26 1:56 ` Yang Yingliang
0 siblings, 1 reply; 4+ messages in thread
From: Ping-Ke Shih @ 2022-08-26 0:44 UTC (permalink / raw)
To: Yang Yingliang, linux-kernel, netdev, linux-wireless
Cc: tony0620emma, kvalo, Bernie Huang
> -----Original Message-----
> From: Yang Yingliang <yangyingliang@huawei.com>
> Sent: Thursday, August 25, 2022 9:38 PM
> To: linux-kernel@vger.kernel.org; netdev@vger.kernel.org; linux-wireless@vger.kernel.org
> Cc: tony0620emma@gmail.com; kvalo@kernel.org; Bernie Huang <phhuang@realtek.com>
> Subject: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
>
> Add the missing destroy_workqueue() before return from rtw_core_init()
> in error path.
>
> Fixes: fe101716c7c9 ("rtw88: replace tx tasklet with work queue")
> Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
> ---
> drivers/net/wireless/realtek/rtw88/main.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/realtek/rtw88/main.c b/drivers/net/wireless/realtek/rtw88/main.c
> index 790dcfed1125..557213e52761 100644
> --- a/drivers/net/wireless/realtek/rtw88/main.c
> +++ b/drivers/net/wireless/realtek/rtw88/main.c
> @@ -2094,7 +2094,7 @@ int rtw_core_init(struct rtw_dev *rtwdev)
> ret = rtw_load_firmware(rtwdev, RTW_NORMAL_FW);
> if (ret) {
> rtw_warn(rtwdev, "no firmware loaded\n");
> - return ret;
> + goto destroy_workqueue;
> }
>
> if (chip->wow_fw_name) {
> @@ -2104,11 +2104,15 @@ int rtw_core_init(struct rtw_dev *rtwdev)
> wait_for_completion(&rtwdev->fw.completion);
> if (rtwdev->fw.firmware)
> release_firmware(rtwdev->fw.firmware);
> - return ret;
> + goto destroy_workqueue;
> }
> }
>
> return 0;
> +
> +destroy_workqueue:
It's not so good that the label 'destroy_workqueue' is the same as function name.
I suggest to just use 'out' instead.
> + destroy_workqueue(rtwdev->tx_wq);
> + return ret;
> }
> EXPORT_SYMBOL(rtw_core_init);
>
> --
> 2.25.1
>
>
> ------Please consider the environment before printing this e-mail.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
2022-08-26 0:44 ` Ping-Ke Shih
@ 2022-08-26 1:56 ` Yang Yingliang
2022-08-26 2:01 ` Ping-Ke Shih
0 siblings, 1 reply; 4+ messages in thread
From: Yang Yingliang @ 2022-08-26 1:56 UTC (permalink / raw)
To: Ping-Ke Shih, linux-kernel, netdev, linux-wireless
Cc: tony0620emma, kvalo, Bernie Huang
Hi,
On 2022/8/26 8:44, Ping-Ke Shih wrote:
>> -----Original Message-----
>> From: Yang Yingliang <yangyingliang@huawei.com>
>> Sent: Thursday, August 25, 2022 9:38 PM
>> To: linux-kernel@vger.kernel.org; netdev@vger.kernel.org; linux-wireless@vger.kernel.org
>> Cc: tony0620emma@gmail.com; kvalo@kernel.org; Bernie Huang <phhuang@realtek.com>
>> Subject: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
>>
>> Add the missing destroy_workqueue() before return from rtw_core_init()
>> in error path.
>>
>> Fixes: fe101716c7c9 ("rtw88: replace tx tasklet with work queue")
>> Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
>> ---
>> drivers/net/wireless/realtek/rtw88/main.c | 8 ++++++--
>> 1 file changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/wireless/realtek/rtw88/main.c b/drivers/net/wireless/realtek/rtw88/main.c
>> index 790dcfed1125..557213e52761 100644
>> --- a/drivers/net/wireless/realtek/rtw88/main.c
>> +++ b/drivers/net/wireless/realtek/rtw88/main.c
>> @@ -2094,7 +2094,7 @@ int rtw_core_init(struct rtw_dev *rtwdev)
>> ret = rtw_load_firmware(rtwdev, RTW_NORMAL_FW);
>> if (ret) {
>> rtw_warn(rtwdev, "no firmware loaded\n");
>> - return ret;
>> + goto destroy_workqueue;
>> }
>>
>> if (chip->wow_fw_name) {
>> @@ -2104,11 +2104,15 @@ int rtw_core_init(struct rtw_dev *rtwdev)
>> wait_for_completion(&rtwdev->fw.completion);
>> if (rtwdev->fw.firmware)
>> release_firmware(rtwdev->fw.firmware);
>> - return ret;
>> + goto destroy_workqueue;
>> }
>> }
>>
>> return 0;
>> +
>> +destroy_workqueue:
> It's not so good that the label 'destroy_workqueue' is the same as function name.
> I suggest to just use 'out' instead.
How about 'out_destory_workqueue' ?
Thanks,
Yang
>
>> + destroy_workqueue(rtwdev->tx_wq);
>> + return ret;
>> }
>> EXPORT_SYMBOL(rtw_core_init);
>>
>> --
>> 2.25.1
>>
>>
>> ------Please consider the environment before printing this e-mail.
> .
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
2022-08-26 1:56 ` Yang Yingliang
@ 2022-08-26 2:01 ` Ping-Ke Shih
0 siblings, 0 replies; 4+ messages in thread
From: Ping-Ke Shih @ 2022-08-26 2:01 UTC (permalink / raw)
To: Yang Yingliang, linux-kernel, netdev, linux-wireless
Cc: tony0620emma, kvalo, Bernie Huang
> -----Original Message-----
> From: Yang Yingliang <yangyingliang@huawei.com>
> Sent: Friday, August 26, 2022 9:57 AM
> To: Ping-Ke Shih <pkshih@realtek.com>; linux-kernel@vger.kernel.org; netdev@vger.kernel.org;
> linux-wireless@vger.kernel.org
> Cc: tony0620emma@gmail.com; kvalo@kernel.org; Bernie Huang <phhuang@realtek.com>
> Subject: Re: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
>
> Hi,
>
> On 2022/8/26 8:44, Ping-Ke Shih wrote:
> >> -----Original Message-----
> >> From: Yang Yingliang <yangyingliang@huawei.com>
> >> Sent: Thursday, August 25, 2022 9:38 PM
> >> To: linux-kernel@vger.kernel.org; netdev@vger.kernel.org; linux-wireless@vger.kernel.org
> >> Cc: tony0620emma@gmail.com; kvalo@kernel.org; Bernie Huang <phhuang@realtek.com>
> >> Subject: [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
> >>
> >> Add the missing destroy_workqueue() before return from rtw_core_init()
> >> in error path.
> >>
> >> Fixes: fe101716c7c9 ("rtw88: replace tx tasklet with work queue")
> >> Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
> >> ---
> >> drivers/net/wireless/realtek/rtw88/main.c | 8 ++++++--
> >> 1 file changed, 6 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/drivers/net/wireless/realtek/rtw88/main.c b/drivers/net/wireless/realtek/rtw88/main.c
> >> index 790dcfed1125..557213e52761 100644
> >> --- a/drivers/net/wireless/realtek/rtw88/main.c
> >> +++ b/drivers/net/wireless/realtek/rtw88/main.c
> >> @@ -2094,7 +2094,7 @@ int rtw_core_init(struct rtw_dev *rtwdev)
> >> ret = rtw_load_firmware(rtwdev, RTW_NORMAL_FW);
> >> if (ret) {
> >> rtw_warn(rtwdev, "no firmware loaded\n");
> >> - return ret;
> >> + goto destroy_workqueue;
> >> }
> >>
> >> if (chip->wow_fw_name) {
> >> @@ -2104,11 +2104,15 @@ int rtw_core_init(struct rtw_dev *rtwdev)
> >> wait_for_completion(&rtwdev->fw.completion);
> >> if (rtwdev->fw.firmware)
> >> release_firmware(rtwdev->fw.firmware);
> >> - return ret;
> >> + goto destroy_workqueue;
> >> }
> >> }
> >>
> >> return 0;
> >> +
> >> +destroy_workqueue:
> > It's not so good that the label 'destroy_workqueue' is the same as function name.
> > I suggest to just use 'out' instead.
> How about 'out_destory_workqueue' ?
>
Since there is only single one error case we need to handle, using 'out'
isn't ambiguous.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-08-26 2:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-25 13:37 [PATCH -next] wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init() Yang Yingliang
2022-08-26 0:44 ` Ping-Ke Shih
2022-08-26 1:56 ` Yang Yingliang
2022-08-26 2:01 ` Ping-Ke Shih
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).