From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay3.corp.sgi.com [198.149.34.15]) by oss.sgi.com (Postfix) with ESMTP id DB0F47F3F for ; Mon, 26 Oct 2015 09:08:51 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay3.corp.sgi.com (Postfix) with ESMTP id 3D219AC003 for ; Mon, 26 Oct 2015 07:08:51 -0700 (PDT) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id BqqnY93yZHiwBiYX (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Mon, 26 Oct 2015 07:08:49 -0700 (PDT) Date: Mon, 26 Oct 2015 10:08:47 -0400 From: Brian Foster Subject: Re: [PATCH] xfs: clear PF_NOFREEZE for xfsaild kthread Message-ID: <20151026140847.GD59738@bfoster.bfoster> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Jiri Kosina Cc: xfs@oss.sgi.com, Christoph Hellwig , linux-kernel@vger.kernel.org On Mon, Oct 26, 2015 at 03:53:40PM +0900, Jiri Kosina wrote: > From: Jiri Kosina > > Since xfsaild has been converted to kthread in 0030807c, it calls > try_to_freeze() during every AIL push iteration. It however doesn't set > itself as freezable, and therefore this try_to_freeze() will never do > anything. > > Before (hopefully eventually) kthread freezing gets converted to fileystem > freezing, we'd rather mark xfsaild freezable (as it can generate I/O > during suspend). > > Signed-off-by: Jiri Kosina > --- Looks fine to me: Reviewed-by: Brian Foster > fs/xfs/xfs_trans_ail.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c > index 1098cf4..06d1a29 100644 > --- a/fs/xfs/xfs_trans_ail.c > +++ b/fs/xfs/xfs_trans_ail.c > @@ -497,6 +497,7 @@ xfsaild( > long tout = 0; /* milliseconds */ > > current->flags |= PF_MEMALLOC; > + set_freezable(); > > while (!kthread_should_stop()) { > if (tout && tout <= 20) > -- > Jiri Kosina > SUSE Labs > > _______________________________________________ > xfs mailing list > xfs@oss.sgi.com > http://oss.sgi.com/mailman/listinfo/xfs _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs