From patchwork Fri Nov 11 16:10:33 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: ChenXiaoSong X-Patchwork-Id: 18878 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp795348wru; Fri, 11 Nov 2022 07:09:20 -0800 (PST) X-Google-Smtp-Source: AA0mqf7RHa+fh8j8COKfePKoWZgzhhHBokf6kCVfDbbfa5XGWZhdw/gmsaLkPBVKQ8CjEWdSv7Ah X-Received: by 2002:a17:90b:b08:b0:212:d796:d30f with SMTP id bf8-20020a17090b0b0800b00212d796d30fmr2442345pjb.9.1668179360511; Fri, 11 Nov 2022 07:09:20 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668179360; cv=none; d=google.com; s=arc-20160816; b=a/hShMYsKdGgbp/ufpqRwb8Yhh/gMmqnpI1mErITyRR0n9OYoSTvaXyzFjd5oZAdpe 3Ty8n30K88y8qHCA3TGChf8RPgt6b/qskjhijKpxW6s1RgprJ7pjSjVxzax6pFd5o/lm Vx1Xrz1h48RLRzOtXjzvsN8EbY8mm7oH47baELKaENjnlH3VDWYea24FmwCd4mmBy9Tw xMhE0tOWi+eYswejha0fMFcj1IPJi+giSQVAEp2TrWZ/WSSY9SXooAJ18Gf1sjJZ9HKY q9sZUXK4k8q/rbrfjFOXMrC2cY1tIR4/RONSyZa76sKQmUEA1BfFS9F2/4Q9JPUH236A jDIQ== 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=KbckR146Er6voHU/WL5A5PH4uH5c58mz54qXKikv7/g=; b=EcMXZw5pW1KGwshdcwaTY+J7Wm+ZZL6kpAFbxYVy5jUJnW6hFJAaGDaBzHR5/74y5A FLDWHkBDM2AiSaMN0jHfFqqOFfHGcbGEqGD/IX3qJ1YsRvZ/gtEPdxNWfTTD7nniT/zh dLGbzZviAROAPfCjJpjHjQBhHQqP/IKaTd1ZFN3j1QNKXXv5pgl/maxBFl5iPu8MTE9a yl+KcXJB6i0eL/9I0tfGq4dlWCMCtG76R0Ze90vsiKR249ko1EengcN6OIcBs6Uvbu/+ koPYISc2idqJ6Mo+0kEpg2OSX6K2e6xLyZ2s89Od4hPf55yBbp+kjhegd2n/fS3SJ73F maLw== 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 b25-20020a631b19000000b0046af665ed92si2822908pgb.480.2022.11.11.07.09.03; Fri, 11 Nov 2022 07:09:20 -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 S234975AbiKKPIz (ORCPT + 99 others); Fri, 11 Nov 2022 10:08:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35226 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234864AbiKKPH6 (ORCPT ); Fri, 11 Nov 2022 10:07:58 -0500 Received: from szxga08-in.huawei.com (szxga08-in.huawei.com [45.249.212.255]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59E8276FBC; Fri, 11 Nov 2022 07:05:50 -0800 (PST) Received: from dggemv704-chm.china.huawei.com (unknown [172.30.72.53]) by szxga08-in.huawei.com (SkyGuard) with ESMTP id 4N827v4msrz15MVY; Fri, 11 Nov 2022 23:05:31 +0800 (CST) Received: from kwepemm600015.china.huawei.com (7.193.23.52) by dggemv704-chm.china.huawei.com (10.3.19.47) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Fri, 11 Nov 2022 23:05:46 +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; Fri, 11 Nov 2022 23:05:45 +0800 From: ChenXiaoSong To: , CC: , , , , Subject: [PATCH] NFSv4.1: handle memory allocation failure in nfs4_schedule_state_manager() Date: Sat, 12 Nov 2022 00:10:33 +0800 Message-ID: <20221111161033.899541-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: dggems701-chm.china.huawei.com (10.3.19.178) 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?1749212841222528130?= X-GMAIL-MSGID: =?utf-8?q?1749212841222528130?= 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 | 15 ++++++++++----- 2 files changed, 11 insertions(+), 6 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..127027f777c8 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,7 @@ 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; + goto out; } set_bit(NFS4CLNT_MANAGER_RUNNING, &clp->cl_state); __module_get(THIS_MODULE); @@ -1228,13 +1230,16 @@ 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)) { + ret = PTR_ERR(task); printk(KERN_ERR "%s: kthread_run: %ld\n", - __func__, PTR_ERR(task)); + __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; } /*