From patchwork Sat Nov 12 07:30:55 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: ChenXiaoSong X-Patchwork-Id: 19117 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp1147574wru; Fri, 11 Nov 2022 23:01:57 -0800 (PST) X-Google-Smtp-Source: AA0mqf6Yx2LpBaLLLMKPm2Nt8pUONnFBbAw2krwe4WzJi4Q70J1+5IE0UtdRZMU30xoD/BBZpnDF X-Received: by 2002:a17:902:6b89:b0:188:5688:828b with SMTP id p9-20020a1709026b8900b001885688828bmr5910326plk.129.1668236516769; Fri, 11 Nov 2022 23:01:56 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668236516; cv=none; d=google.com; s=arc-20160816; b=0XrOBdIMCzt+7RgrQyxi1pIQv17gKtwL6c5cA7XheI+ltlmOGEuJf6g9sgORDl0eKd HdEG6fbLqEdem1nshuQEwpzseT+7AinBOhDnAeWZktz02JuEnefRwiKbWu+3IC0OK65w QKWsq05IRGeYkijazSGTVWaIu5WK2GMtTVCYwrWyZo3ozkbP5N8f6Lzx4rQBFexHCShv oI99EGDbVGmVsAHKM1Yxqu0D/yHfo/rOJI0h7YmGxyzvS8+DvLz107HZuKCd9UypoUVm c4q0X7B4H7YHqZFu3sQWIAB5hCeUBcNv1u9RveklLzK5eBJly/0BamyJRBJXIwOs6+V/ 2kWw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=io+9YHtc5d/HQpsrRq7TUAeq0iTu0tUJFCJ12Jhs2gg=; b=zYLXAnTKTmdZb4G2lZzff8SE8AdGf2o/OtJSWHKQfHjurg0AMZiv3ZYhXkjlxukIfz FsWw3LdMU5Sx8qpkfvAYvETg2+peJze6xRCcfYJg3LPAwhHBEU8UsEzTFWiSNCCt2EIH vqFBVIaq+tn2a0C0Aym35SCReUVT0ObJRTApKMoppia0ptn/JaXlCTl7ZCl/egvlfiM3 TGd8Oyhepv16f/mmTN3jIOmOakwT8qVNbBIw6zFCmsl1Lq7HvnIMIiYR0jEP3icdQUdR EgWt7fNvTg06MrckDdARjlT7HRx1IqUxkgN/AY0JItKIYwH4SRnLILPsSXY5uUCJFGre Ys4A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w21-20020a056a0014d500b005627d896458si5091863pfu.142.2022.11.11.23.01.25; Fri, 11 Nov 2022 23:01:56 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234762AbiKLG0T (ORCPT + 99 others); Sat, 12 Nov 2022 01:26:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45306 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233223AbiKLG0S (ORCPT ); Sat, 12 Nov 2022 01:26:18 -0500 Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B0D75BCBE; Fri, 11 Nov 2022 22:26:16 -0800 (PST) Received: from dggemv711-chm.china.huawei.com (unknown [172.30.72.55]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4N8QYy0RZDzmVnT; Sat, 12 Nov 2022 14:25:58 +0800 (CST) Received: from kwepemm600015.china.huawei.com (7.193.23.52) by dggemv711-chm.china.huawei.com (10.1.198.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Sat, 12 Nov 2022 14:26:14 +0800 Received: from huawei.com (10.175.101.6) by kwepemm600015.china.huawei.com (7.193.23.52) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Sat, 12 Nov 2022 14:26:13 +0800 From: ChenXiaoSong To: , CC: , , , , , Subject: [PATCH v2] NFSv4.1: handle memory allocation failure in nfs4_schedule_state_manager() Date: Sat, 12 Nov 2022 15:30:55 +0800 Message-ID: <20221112073055.1024799-1-chenxiaosong2@huawei.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 X-Originating-IP: [10.175.101.6] X-ClientProxiedBy: dggems704-chm.china.huawei.com (10.3.19.181) To kwepemm600015.china.huawei.com (7.193.23.52) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749272773473467392?= X-GMAIL-MSGID: =?utf-8?q?1749272773473467392?= If memory allocation fail in nfs4_schedule_state_manager() when mount NFSv4.1/NFSv4.2, nfs4_run_state_manager() will not be called, and current construction state will never be marked as ready or failed, nfs_wait_client_init_complete() will wait forever, as shown below: syscall(mount) ... nfs4_init_client nfs4_discover_server_trunking nfs41_discover_server_trunking nfs4_schedule_state_manager kthread_run /* nfs4_run_state_manager() will not be called */ kthread_create kthread_create_on_node __kthread_create_on_node create = kmalloc() = NULL return ERR_PTR(-ENOMEM) nfs_wait_client_init_complete /* wait forever */ Fix this by checking return value of nfs4_schedule_state_manager() which can indicate whether kernel thread is created successful. Signed-off-by: ChenXiaoSong --- fs/nfs/nfs4_fs.h | 2 +- fs/nfs/nfs4state.c | 17 +++++++++++------ 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/fs/nfs/nfs4_fs.h b/fs/nfs/nfs4_fs.h index cfef738d765e..74c6d1504010 100644 --- a/fs/nfs/nfs4_fs.h +++ b/fs/nfs/nfs4_fs.h @@ -502,7 +502,7 @@ extern int nfs4_state_mark_reclaim_nograce(struct nfs_client *, struct nfs4_stat extern void nfs4_schedule_lease_recovery(struct nfs_client *); extern int nfs4_wait_clnt_recover(struct nfs_client *clp); extern int nfs4_client_recover_expired_lease(struct nfs_client *clp); -extern void nfs4_schedule_state_manager(struct nfs_client *); +extern int nfs4_schedule_state_manager(struct nfs_client *); extern void nfs4_schedule_path_down_recovery(struct nfs_client *clp); extern int nfs4_schedule_stateid_recovery(const struct nfs_server *, struct nfs4_state *); extern int nfs4_schedule_migration_recovery(const struct nfs_server *); diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c index a2d2d5d1b088..e32739fb9151 100644 --- a/fs/nfs/nfs4state.c +++ b/fs/nfs/nfs4state.c @@ -376,8 +376,9 @@ int nfs41_discover_server_trunking(struct nfs_client *clp, else set_bit(NFS4CLNT_LEASE_CONFIRM, &clp->cl_state); } - nfs4_schedule_state_manager(clp); - status = nfs_wait_client_init_complete(clp); + status = nfs4_schedule_state_manager(clp); + if (!status) + status = nfs_wait_client_init_complete(clp); if (status < 0) nfs_put_client(clp); return status; @@ -1201,11 +1202,12 @@ static void nfs4_clear_state_manager_bit(struct nfs_client *clp) /* * Schedule the nfs_client asynchronous state management routine */ -void nfs4_schedule_state_manager(struct nfs_client *clp) +int nfs4_schedule_state_manager(struct nfs_client *clp) { struct task_struct *task; char buf[INET6_ADDRSTRLEN + sizeof("-manager") + 1]; struct rpc_clnt *cl = clp->cl_rpcclient; + int ret = 0; while (cl != cl->cl_parent) cl = cl->cl_parent; @@ -1213,7 +1215,8 @@ void nfs4_schedule_state_manager(struct nfs_client *clp) set_bit(NFS4CLNT_RUN_MANAGER, &clp->cl_state); if (test_and_set_bit(NFS4CLNT_MANAGER_AVAILABLE, &clp->cl_state) != 0) { wake_up_var(&clp->cl_state); - return; + ret = -EIO; + goto out; } set_bit(NFS4CLNT_MANAGER_RUNNING, &clp->cl_state); __module_get(THIS_MODULE); @@ -1228,13 +1231,15 @@ void nfs4_schedule_state_manager(struct nfs_client *clp) rcu_read_unlock(); task = kthread_run(nfs4_run_state_manager, clp, "%s", buf); if (IS_ERR(task)) { - printk(KERN_ERR "%s: kthread_run: %ld\n", - __func__, PTR_ERR(task)); + ret = PTR_ERR(task); + printk(KERN_ERR "%s: kthread_run: %d\n", __func__, ret); nfs4_clear_state_manager_bit(clp); clear_bit(NFS4CLNT_MANAGER_AVAILABLE, &clp->cl_state); nfs_put_client(clp); module_put(THIS_MODULE); } +out: + return ret; } /*