From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1756878AbdKNVZv (ORCPT <rfc822;w@1wt.eu>);
        Tue, 14 Nov 2017 16:25:51 -0500
Received: from ipmail07.adl2.internode.on.net ([150.101.137.131]:1203 "EHLO
        ipmail07.adl2.internode.on.net" rhost-flags-OK-OK-OK-OK)
        by vger.kernel.org with ESMTP id S1754496AbdKNVZm (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 14 Nov 2017 16:25:42 -0500
Date: Wed, 15 Nov 2017 08:25:38 +1100
From: Dave Chinner <david@fromorbit.com>
To: "Luis R. Rodriguez" <mcgrof@kernel.org>
Cc: Dan Williams <dan.j.williams@intel.com>, Yu Chen <yu.c.chen@intel.com>,
        Michal Hocko <mhocko@suse.com>,
        Hendrik Woltersdorf <hendrikw@arcor.de>,
        Dave Chinner <dchinner@redhat.com>,
        "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
        "Darrick J. Wong" <darrick.wong@oracle.com>,
        Jiri Kosina <jkosina@suse.cz>, Len Brown <len.brown@intel.com>,
        Rui Zhang <rui.zhang@intel.com>, Yu Chen <yu.chen.surf@gmail.com>,
        linux-xfs@vger.kernel.org,
        Linux-pm mailing list <linux-pm@vger.kernel.org>,
        linux-kernel@vger.kernel.org, Thorsten Leemhuis <linux@leemhuis.info>,
        oleg.b.antonyan@gmail.com, Oliver Neukum <oneukum@suse.com>
Subject: Re: [Regression/XFS/PM] Freeze tasks failed in xfsaild
Message-ID: <20171114212538.GC4094@dastard>
References: <20171113103139.GA18936@yu-chen.sh.intel.com>
 <20171113201414.GD16026@wotan.suse.de>
 <CAPcyv4hbHpoMe20Sjpq4Qov6TKx9A+t6Y9GJ68PT2tsY3Jh7kA@mail.gmail.com>
 <20171114201915.GI729@wotan.suse.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20171114201915.GI729@wotan.suse.de>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Nov 14, 2017 at 09:19:15PM +0100, Luis R. Rodriguez wrote:
> This is another way to say suspend has been busted on XFS for a very long time,
> but I would not blame XFS -- this is a kernel issue to get proper filesystem
> suspend working right, and the way we currently deal with kthreads is just
> a sloppy goo mess which has created this situation.

Yes, and I've been telling people that suspend on journalling
filesystems has been broken for a long time (i.e since I first
realised the scope of the problem back in 2005). However, only XFS
triggers those conditions regularly because it is the most
asynchronous of the "freezable" journalling filesytems and has the
most reliance on co-ordination of kernel threads and workqueues to
function correctly.

IOWs, suspend of filesystems has been broken forever, and we've been
slapping bandaids on it in XFS forever. Now we've got to a
catch-22 situation that bandaids can't fix. We need structural
fixes, like I said we needed to do more than 10 years ago.

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com