From patchwork Mon Apr 10 17:12:54 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sebastian Andrzej Siewior X-Patchwork-Id: 9673653 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id A7E8A60244 for ; Mon, 10 Apr 2017 17:13:11 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8FE1B26247 for ; Mon, 10 Apr 2017 17:13:11 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8453726E76; Mon, 10 Apr 2017 17:13:11 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id E52F326247 for ; Mon, 10 Apr 2017 17:13:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755652AbdDJRNK (ORCPT ); Mon, 10 Apr 2017 13:13:10 -0400 Received: from Galois.linutronix.de ([146.0.238.70]:52129 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755562AbdDJRNH (ORCPT ); Mon, 10 Apr 2017 13:13:07 -0400 Received: from localhost ([127.0.0.1] helo=bazinga.breakpoint.cc) by Galois.linutronix.de with esmtp (Exim 4.80) (envelope-from ) id 1cxcrB-00015Z-Q8; Mon, 10 Apr 2017 19:12:01 +0200 From: Sebastian Andrzej Siewior To: "Martin K . Petersen" , "James E.J. Bottomley" , linux-scsi@vger.kernel.org Cc: rt@linutronix.de, Lee Duncan , Chris Leech , Chad Dupuis , QLogic-Storage-Upstream@qlogic.com, Johannes Thumshirn , Christoph Hellwig , Andrew Morton , Sebastian Andrzej Siewior , fcoe-devel@open-fcoe.org Subject: [PATCH 5/5] scsi: bnx2fc: convert bnx2fc_l2_rcv_thread() to workqueue Date: Mon, 10 Apr 2017 19:12:54 +0200 Message-Id: <20170410171254.30367-6-bigeasy@linutronix.de> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170410171254.30367-1-bigeasy@linutronix.de> References: <20170410171254.30367-1-bigeasy@linutronix.de> MIME-Version: 1.0 Sender: linux-scsi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This is not driven by the hotplug conversation but while I am at it looks like a good candidate. Converting the thread to a workqueue user removes also the kthread member from struct fcoe_percpu_s. This driver uses the struct fcoe_percpu_s but it does not need the crc_eof_page member, only the work item and fcoe_rx_list. So it is removed there. We had one thread so we only use the workqueue on the current CPU. If someone knows how spread this nicely, it would only require the usage of schedule_work_on() instead schedule_work() :) This patch was only compile-tested due to -ENODEV. Cc: QLogic-Storage-Upstream@qlogic.com Cc: Christoph Hellwig Cc: fcoe-devel@open-fcoe.org Signed-off-by: Sebastian Andrzej Siewior Reviewed-by: Christoph Hellwig Reviewed-by: Johannes Thumshirn --- drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 59 ++++++++------------------------------- include/scsi/libfcoe.h | 1 - 2 files changed, 12 insertions(+), 48 deletions(-) diff --git a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c index 2f66c2ea093c..ca1b5908114d 100644 --- a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c +++ b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c @@ -479,7 +479,7 @@ static int bnx2fc_rcv(struct sk_buff *skb, struct net_device *dev, __skb_queue_tail(&bg->fcoe_rx_list, skb); if (bg->fcoe_rx_list.qlen == 1) - wake_up_process(bg->kthread); + schedule_work(&bg->work); spin_unlock(&bg->fcoe_rx_list.lock); @@ -489,26 +489,20 @@ static int bnx2fc_rcv(struct sk_buff *skb, struct net_device *dev, return -1; } -static int bnx2fc_l2_rcv_thread(void *arg) +static void bnx2fc_l2_rcv_work(struct work_struct *work_s) { - struct fcoe_percpu_s *bg = arg; + struct fcoe_percpu_s *bg; struct sk_buff *skb; - set_user_nice(current, MIN_NICE); - set_current_state(TASK_INTERRUPTIBLE); - while (!kthread_should_stop()) { - schedule(); - spin_lock_bh(&bg->fcoe_rx_list.lock); - while ((skb = __skb_dequeue(&bg->fcoe_rx_list)) != NULL) { - spin_unlock_bh(&bg->fcoe_rx_list.lock); - bnx2fc_recv_frame(skb); - spin_lock_bh(&bg->fcoe_rx_list.lock); - } - __set_current_state(TASK_INTERRUPTIBLE); + bg = container_of(work_s, struct fcoe_percpu_s, work); + + spin_lock_bh(&bg->fcoe_rx_list.lock); + while ((skb = __skb_dequeue(&bg->fcoe_rx_list)) != NULL) { spin_unlock_bh(&bg->fcoe_rx_list.lock); + bnx2fc_recv_frame(skb); + spin_lock_bh(&bg->fcoe_rx_list.lock); } - __set_current_state(TASK_RUNNING); - return 0; + spin_unlock_bh(&bg->fcoe_rx_list.lock); } @@ -2564,8 +2558,6 @@ static int bnx2fc_slave_configure(struct scsi_device *sdev) return 0; } -static enum cpuhp_state bnx2fc_online_state; - /** * bnx2fc_mod_init - module init entry point * @@ -2575,7 +2567,6 @@ static enum cpuhp_state bnx2fc_online_state; static int __init bnx2fc_mod_init(void) { struct fcoe_percpu_s *bg; - struct task_struct *l2_thread; int rc = 0; unsigned int cpu = 0; struct bnx2fc_percpu_s *p; @@ -2608,17 +2599,7 @@ static int __init bnx2fc_mod_init(void) bg = &bnx2fc_global; skb_queue_head_init(&bg->fcoe_rx_list); - l2_thread = kthread_create(bnx2fc_l2_rcv_thread, - (void *)bg, - "bnx2fc_l2_thread"); - if (IS_ERR(l2_thread)) { - rc = PTR_ERR(l2_thread); - goto free_wq; - } - wake_up_process(l2_thread); - spin_lock_bh(&bg->fcoe_rx_list.lock); - bg->kthread = l2_thread; - spin_unlock_bh(&bg->fcoe_rx_list.lock); + INIT_WORK(&bg->work, bnx2fc_l2_rcv_work); for_each_possible_cpu(cpu) { p = &per_cpu(bnx2fc_percpu, cpu); @@ -2631,8 +2612,6 @@ static int __init bnx2fc_mod_init(void) return 0; -free_wq: - destroy_workqueue(bnx2fc_wq); release_bt: bnx2fc_release_transport(); detach_ft: @@ -2645,9 +2624,6 @@ static void __exit bnx2fc_mod_exit(void) { LIST_HEAD(to_be_deleted); struct bnx2fc_hba *hba, *next; - struct fcoe_percpu_s *bg; - struct task_struct *l2_thread; - struct sk_buff *skb; unsigned int cpu = 0; /* @@ -2676,18 +2652,7 @@ static void __exit bnx2fc_mod_exit(void) } cnic_unregister_driver(CNIC_ULP_FCOE); - /* Destroy global thread */ - bg = &bnx2fc_global; - spin_lock_bh(&bg->fcoe_rx_list.lock); - l2_thread = bg->kthread; - bg->kthread = NULL; - while ((skb = __skb_dequeue(&bg->fcoe_rx_list)) != NULL) - kfree_skb(skb); - - spin_unlock_bh(&bg->fcoe_rx_list.lock); - - if (l2_thread) - kthread_stop(l2_thread); + flush_work(&bnx2fc_global.work); for_each_possible_cpu(cpu) { struct bnx2fc_percpu_s *p; diff --git a/include/scsi/libfcoe.h b/include/scsi/libfcoe.h index 722d3264d3bf..ca6fb3c0f172 100644 --- a/include/scsi/libfcoe.h +++ b/include/scsi/libfcoe.h @@ -332,7 +332,6 @@ struct fcoe_transport { * memory for a new trailer */ struct fcoe_percpu_s { - struct task_struct *kthread; struct work_struct work; struct sk_buff_head fcoe_rx_list; struct page *crc_eof_page;