| Message ID | bf5648ef0933536661e42cc73aa06722522d5862.1633872027.git.christophe.jaillet@wanadoo.fr |
|---|---|
| State | New |
| Headers | show |
| Series | ksmbd: Remove redundant 'flush_workqueue()' calls | expand |
2021-10-10 22:21 GMT+09:00, Christophe JAILLET <christophe.jaillet@wanadoo.fr>: > '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@wanadoo.fr> Acked-by: Namjae Jeon <linkinjeon@kernel.org> Thanks!
Acked-by: Hyunchul Lee <hyc.lee@gmail.com> 2021년 10월 10일 (일) 오후 10:29, Christophe JAILLET <christophe.jaillet@wanadoo.fr>님이 작성: > > '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@wanadoo.fr> > --- > fs/ksmbd/ksmbd_work.c | 1 - > fs/ksmbd/transport_rdma.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/fs/ksmbd/ksmbd_work.c b/fs/ksmbd/ksmbd_work.c > index fd58eb4809f6..14b9caebf7a4 100644 > --- a/fs/ksmbd/ksmbd_work.c > +++ b/fs/ksmbd/ksmbd_work.c > @@ -69,7 +69,6 @@ int ksmbd_workqueue_init(void) > > void ksmbd_workqueue_destroy(void) > { > - flush_workqueue(ksmbd_wq); > destroy_workqueue(ksmbd_wq); > ksmbd_wq = NULL; > } > diff --git a/fs/ksmbd/transport_rdma.c b/fs/ksmbd/transport_rdma.c > index 3a7fa23ba850..0fa7b9c17af3 100644 > --- a/fs/ksmbd/transport_rdma.c > +++ b/fs/ksmbd/transport_rdma.c > @@ -2026,7 +2026,6 @@ int ksmbd_rdma_destroy(void) > smb_direct_listener.cm_id = NULL; > > if (smb_direct_wq) { > - flush_workqueue(smb_direct_wq); > destroy_workqueue(smb_direct_wq); > smb_direct_wq = NULL; > } > -- > 2.30.2 >
diff --git a/fs/ksmbd/ksmbd_work.c b/fs/ksmbd/ksmbd_work.c index fd58eb4809f6..14b9caebf7a4 100644 --- a/fs/ksmbd/ksmbd_work.c +++ b/fs/ksmbd/ksmbd_work.c @@ -69,7 +69,6 @@ int ksmbd_workqueue_init(void) void ksmbd_workqueue_destroy(void) { - flush_workqueue(ksmbd_wq); destroy_workqueue(ksmbd_wq); ksmbd_wq = NULL; } diff --git a/fs/ksmbd/transport_rdma.c b/fs/ksmbd/transport_rdma.c index 3a7fa23ba850..0fa7b9c17af3 100644 --- a/fs/ksmbd/transport_rdma.c +++ b/fs/ksmbd/transport_rdma.c @@ -2026,7 +2026,6 @@ int ksmbd_rdma_destroy(void) smb_direct_listener.cm_id = NULL; if (smb_direct_wq) { - flush_workqueue(smb_direct_wq); destroy_workqueue(smb_direct_wq); smb_direct_wq = NULL; }
'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@wanadoo.fr> --- fs/ksmbd/ksmbd_work.c | 1 - fs/ksmbd/transport_rdma.c | 1 - 2 files changed, 2 deletions(-)