From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754002Ab3LRMj6 (ORCPT ); Wed, 18 Dec 2013 07:39:58 -0500 Received: from ipmail06.adl6.internode.on.net ([150.101.137.145]:56334 "EHLO ipmail06.adl6.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751474Ab3LRMj4 (ORCPT ); Wed, 18 Dec 2013 07:39:56 -0500 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AqYJAL6WsVJ5LHyk/2dsb2JhbABSB4MKgzywZIVQgRoXdIIlAQEBAwE6HCMFCwgDGAklDwUlAyETh3wHyXwXFo40SAeENgSYFZIVgW2BUiiBLCM Date: Wed, 18 Dec 2013 23:39:52 +1100 From: Dave Chinner To: Pavel Machek Cc: Josh Boyer , "Rafael J. Wysocki" , Linux PM list , LKML , Jan Kara , linux-fsdevel@vger.kernel.org, Nigel Cunningham , "Srivatsa S. Bhat" Subject: Re: [RFC][PATCH] PM / Sleep: Freeze filesystems during system suspend/hibernation Message-ID: <20131218123952.GI31386@dastard> References: <201201281445.49377.rjw@sisk.pl> <201202172159.15396.rjw@sisk.pl> <201205252113.50900.rjw@sisk.pl> <20131217230843.GA2911@amd.pavel.ucw.cz> <20131217233152.GC20579@dastard> <20131218000128.GA3737@amd.pavel.ucw.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20131218000128.GA3737@amd.pavel.ucw.cz> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 18, 2013 at 01:01:28AM +0100, Pavel Machek wrote: > On Wed 2013-12-18 10:31:52, Dave Chinner wrote: > > On Wed, Dec 18, 2013 at 12:08:43AM +0100, Pavel Machek wrote: > > > Hi! > > > > > > > >>> Did this patch ever wind up going anywhere? Fedora has it sitting in > > > > >>> our tree with a comment that says "rebase" and I don't see it in the > > > > >>> linux-next tree at all. > > > > >>> > > > > >>> Did if fall through the cracks or was it NAKed somewhere? > > > > >> > > > > >> No, it wasn't in principle. There were some comments I haven't addressed yet. > > > > > > > > > > Dredging up a really old thread, sorry. > > > > > > > > > > We're still carrying this patch along in Fedora. Should we drop it at > > > > > this point, or is it still eventually going to head upstream? > > > > > > I'd say drop. > > > > I disagree - given the problem it is resolving leads to silent > > filesystem corruption, this patch should be considered somewhat of a > > priority to push... > > Umm. Ok, I forgot what it does, really. It ensures that the filesystem is in an quiescent state both in memory and on disk, and it cannot be modified in memory or on disk whilst the suspend image is being generated, or by log recovery after a resume before the suspended image has been restored. > So... for few years now suspend corrupts data on XFS? And Fedora has > the fix but it is not in mainline? That does not sound right... The issues freezing the filesystem before the suspend image is created affect every journalled filesystem linux supports, be it XFS, ext4, reiser, btrfs, etc. Cheers, Dave. -- Dave Chinner david@fromorbit.com