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 Received: from lists.sourceforge.net (lists.sourceforge.net [216.105.38.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E6AA9C61DA4 for ; Thu, 16 Feb 2023 07:47:15 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-1.v29.lw.sourceforge.com) by sfs-ml-1.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from ) id 1pSYz4-0001sw-Hr; Thu, 16 Feb 2023 07:47:13 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-1.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1pSYyz-0001so-Go for linux-f2fs-devel@lists.sourceforge.net; Thu, 16 Feb 2023 07:47:08 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=In-Reply-To:Content-Type:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=rdmKKJQsUTWCltXvtKIWk0TEDT0Ln495dnybJdBF6J4=; b=SFBDNIP7/lyUpqQbhUQ+0jxJ4E SPaSfYUuxcBWIv+aqIc9LtCMBwFjPguK4mzOpGK81XM3O28YRst5SsVyDK2DEtjvhckTwGOwSn6Xo kBHnu+433wTH/ZlMS317g+5xQd6QipWT4k/YCjhjCcuJdZ8TrS0jEToR+OKCQqDSBwUQ=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To :From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=rdmKKJQsUTWCltXvtKIWk0TEDT0Ln495dnybJdBF6J4=; b=JHSLi6n7hOWEohd1Q59K5qD9LL gxPdBBBxW1Z84CIUAu98OCmKc35asYqZWJHKqoNtwaZKw7ivk90FWldGK5oHWFxE+LWTb3i7VDRDy A7s2RMviskDq27mztlzzbwOTXZ+eq63bV2uIzfCAWF7RrPScsoMScwA1/4P5J5bG3xNo=; Received: from ams.source.kernel.org ([145.40.68.75]) by sfi-mx-1.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.95) id 1pSYyn-0061MK-1C for linux-f2fs-devel@lists.sourceforge.net; Thu, 16 Feb 2023 07:47:08 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 09C91B82563; Thu, 16 Feb 2023 07:46:49 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 47E71C4339B; Thu, 16 Feb 2023 07:46:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1676533607; bh=1fHVCAebbHEEGzBBOAxpM2zLnFRsw1EJNwYcko4owRM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=VHpQrgDeKRE19/jG0RzUwrvEKA0SZw+/lG7AQ/Kd5qEjSZqkXr8EXuuK2l0qArdIZ qgHKFj9DRz0+A3JT+oeVigYFCF+1UrXArfuB9yFsPEzt0ne8zqdpX88hewzQwuXdXQ xONNtHK/5GsRN4HR6ij65t2H5dHF+Qg3iYZYAHkY= Date: Thu, 16 Feb 2023 08:46:44 +0100 From: "gregkh@linuxfoundation.org" To: Yonggil Song Message-ID: References: <20230216074427epcms2p49a3d71b08d356530b40e34e750cc2366@epcms2p4> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230216074427epcms2p49a3d71b08d356530b40e34e750cc2366@epcms2p4> X-Headers-End: 1pSYyn-0061MK-1C Subject: Re: [f2fs-dev] [PATCH v2] f2fs: fix uninitialized skipped_gc_rwsem X-BeenThere: linux-f2fs-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "daehojeong@google.com" , "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" , "linux-f2fs-devel@lists.sourceforge.net" , "jaegeuk@kernel.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net On Thu, Feb 16, 2023 at 04:44:27PM +0900, Yonggil Song wrote: > When f2fs skipped a gc round during victim migration, there was a bug which > would skip all upcoming gc rounds unconditionally because skipped_gc_rwsem > was not initialized. It fixes the bug by correctly initializing the > skipped_gc_rwsem inside the gc loop. > > Fixes: 3db1de0e582c ("f2fs: change the current atomic write way") > Cc: stable@vger.kernel.org > Signed-off-by: Yonggil Song > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index b22f49a6f128..81d326abaac1 100644 > --- a/fs/f2fs/gc.c > +++ b/fs/f2fs/gc.c > @@ -1786,8 +1786,8 @@ int f2fs_gc(struct f2fs_sb_info *sbi, struct f2fs_gc_control *gc_control) > prefree_segments(sbi)); > > cpc.reason = __get_cp_reason(sbi); > - sbi->skipped_gc_rwsem = 0; > gc_more: > + sbi->skipped_gc_rwsem = 0; > if (unlikely(!(sbi->sb->s_flags & SB_ACTIVE))) { > ret = -EINVAL; > goto stop; > -- > 2.34.1 Hi, This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him a patch that has triggered this response. He used to manually respond to these common problems, but in order to save his sanity (he kept writing the same thing over and over, yet to different people), I was created. Hopefully you will not take offence and will fix the problem in your patch and resubmit it so that it can be accepted into the Linux kernel tree. You are receiving this message because of the following common error(s) as indicated below: - This looks like a new version of a previously submitted patch, but you did not list below the --- line any changes from the previous version. Please read the section entitled "The canonical patch format" in the kernel file, Documentation/process/submitting-patches.rst for what needs to be done here to properly describe this. If you wish to discuss this problem further, or you have questions about how to resolve this issue, please feel free to respond to this email and Greg will reply once he has dug out from the pending patches received from other developers. thanks, greg k-h's patch email bot _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel