From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3F1F1C38A2A for ; Fri, 8 May 2020 12:40:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 12E9221835 for ; Fri, 8 May 2020 12:40:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1588941633; bh=mxL4GtE1sRQmhqzBUuaGbL7A8/BDaEG/9CkUMtYn2cg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=HZJAVIcRreL8G8d4WksYMwUv2N9uzJ4/qPLC+eEnuXFGDp6AmVzEtdWC8NTrTFFdC yAUIv2RQieaIaYXrrkKo/sTVCF242aFOjPunBmUsLMeMOh6iyRZ8tgk7vi2Ej5kWKp zDAnkl23iJ2q1qk+B75H2OLTmU4gAodF0FnE5suE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728713AbgEHMkb (ORCPT ); Fri, 8 May 2020 08:40:31 -0400 Received: from mail.kernel.org ([198.145.29.99]:60690 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728011AbgEHMj4 (ORCPT ); Fri, 8 May 2020 08:39:56 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 96C0920731; Fri, 8 May 2020 12:39:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1588941596; bh=mxL4GtE1sRQmhqzBUuaGbL7A8/BDaEG/9CkUMtYn2cg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=taPTNvj/0Vs8T11dt7FDMN9TLrukK4sRNQSy9Vh6/qaIWfgWCNH5cRVAA5SCVhX96 NwwTZ6CmXNukdhnFSeVW1CQtNuMosWSQq4xBqj3KsOv/ZJdVI8d4NBy71AKMTAsAt7 XnxnxYkMJ0vhhzL6K2t6WAAFpA/mAY9ZkKkaKfu0= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Jiri Kosina , David Sterba Subject: [PATCH 4.4 094/312] btrfs: cleaner_kthread() doesnt need explicit freeze Date: Fri, 8 May 2020 14:31:25 +0200 Message-Id: <20200508123131.146903503@linuxfoundation.org> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200508123124.574959822@linuxfoundation.org> References: <20200508123124.574959822@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Jiri Kosina commit 838fe1887765f4cc679febea60d87d2a06bd300e upstream. cleaner_kthread() is not marked freezable, and therefore calling try_to_freeze() in its context is a pointless no-op. In addition to that, as has been clearly demonstrated by 80ad623edd2d ("Revert "btrfs: clear PF_NOFREEZE in cleaner_kthread()"), it's perfectly valid / legal for cleaner_kthread() to stay scheduled out in an arbitrary place during suspend (in that particular example that was waiting for reading of extent pages), so there is no need to leave any traces of freezer in this kthread. Fixes: 80ad623edd2d ("Revert "btrfs: clear PF_NOFREEZE in cleaner_kthread()") Fixes: 696249132158 ("btrfs: clear PF_NOFREEZE in cleaner_kthread()") Signed-off-by: Jiri Kosina Signed-off-by: David Sterba Signed-off-by: Greg Kroah-Hartman --- fs/btrfs/disk-io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -1750,7 +1750,7 @@ static int cleaner_kthread(void *arg) */ btrfs_delete_unused_bgs(root->fs_info); sleep: - if (!try_to_freeze() && !again) { + if (!again) { set_current_state(TASK_INTERRUPTIBLE); if (!kthread_should_stop()) schedule();