[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 2 Feb 2022 20:53:58 +0800
From: "D. Wythe" <alibuda@...ux.alibaba.com>
To: Karsten Graul <kgraul@...ux.ibm.com>
Cc: kuba@...nel.org, davem@...emloft.net, netdev@...r.kernel.org,
linux-s390@...r.kernel.org, linux-rdma@...r.kernel.org,
matthieu.baerts@...sares.net
Subject: Re: [PATCH v2 net-next 1/3] net/smc: Make smc_tcp_listen_work()
independent
That's right, 'extern' is unnecessary, I'll remove it soon.
Looking forward for more advise.
Thanks.
>>
>> +extern struct workqueue_struct *smc_tcp_ls_wq; /* wq for tcp listen work */
>
> I don't think this extern is needed, the work queue is only used within af_smc.c, right?
> Even the smc_hs_wq would not need to be extern, but this would be a future cleanup.
>
>> extern struct workqueue_struct *smc_hs_wq; /* wq for handshake work */
>> extern struct workqueue_struct *smc_close_wq; /* wq for close work */
>>
Powered by blists - more mailing lists