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
| ||
|
Date: Fri, 12 Nov 2021 10:59:51 -0700 From: Shuah Khan <skhan@...uxfoundation.org> To: Christophe JAILLET <christophe.jaillet@...adoo.fr>, peter.chen@...nel.org, gregkh@...uxfoundation.org, balbi@...nel.org, valentina.manea.m@...il.com, shuah@...nel.org, johan@...nel.org, zhengyongjun3@...wei.com, colin.king@...el.com, trix@...hat.com, Shuah Khan <skhan@...uxfoundation.org> Cc: linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org, kernel-janitors@...r.kernel.org Subject: Re: [PATCH] usb: Remove redundant 'flush_workqueue()' calls On 11/12/21 9:30 AM, Christophe JAILLET wrote: > 'destroy_workqueue()' already drains the queue before destroying it, so > there is no need to flush it explicitly. > > Remove the redundant 'flush_workqueue()' calls. > > This was generated with coccinelle: > > @@ > expression E; > @@ > - flush_workqueue(E); > destroy_workqueue(E); > > Signed-off-by: Christophe JAILLET <christophe.jaillet@...adoo.fr> > --- > drivers/usb/chipidea/otg.c | 5 ++--- > drivers/usb/gadget/udc/mv_udc_core.c | 4 +--- > drivers/usb/host/u132-hcd.c | 1 - > drivers/usb/phy/phy-mv-usb.c | 5 +---- > drivers/usb/usbip/usbip_event.c | 1 - > 5 files changed, 4 insertions(+), 12 deletions(-) > > diff --git a/drivers/usb/usbip/usbip_event.c b/drivers/usb/usbip/usbip_event.c > index 086ca76dd053..26513540bcdb 100644 > --- a/drivers/usb/usbip/usbip_event.c > +++ b/drivers/usb/usbip/usbip_event.c > @@ -137,7 +137,6 @@ int usbip_init_eh(void) > > void usbip_finish_eh(void) > { > - flush_workqueue(usbip_queue); > destroy_workqueue(usbip_queue); > usbip_queue = NULL; > } > Might be better to generate per driver patches. Looks good to me. For the usbip change in this patch: Acked-by: Shuah Khan <skhan@...uxfoundation.org> thanks, -- Shuah
Powered by blists - more mailing lists