From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ipmail06.adl2.internode.on.net ([150.101.137.129]:37351 "EHLO ipmail06.adl2.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933873AbeFUWeJ (ORCPT ); Thu, 21 Jun 2018 18:34:09 -0400 Date: Fri, 22 Jun 2018 08:34:06 +1000 From: Dave Chinner Subject: Re: [PATCH 1/2] xfs: transactionless xfs_bunmapi shouldn't do format conversion Message-ID: <20180621223406.GV19934@dastard> References: <20180619024128.22669-1-david@fromorbit.com> <20180619024128.22669-2-david@fromorbit.com> <20180619045405.GI8128@magnolia> <20180620073142.GA5257@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180620073142.GA5257@infradead.org> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Christoph Hellwig Cc: "Darrick J. Wong" , linux-xfs@vger.kernel.org On Wed, Jun 20, 2018 at 12:31:42AM -0700, Christoph Hellwig wrote: > On Mon, Jun 18, 2018 at 09:54:05PM -0700, Darrick J. Wong wrote: > > ...it's also weird that xfs_bmap_punch_delalloc_range calls xfs_bunmapi > > with no transaction and a xfs_defer that we dump on the ground. > > > > So yes, I think the patch does fix the crash, but it's kinda gross. > > > > Thoughts? > > I've got an alternative solution: > > http://git.infradead.org/users/hch/xfs.git/commitdiff/a1c0685b2085b448cbe02f0f9ff0c8771e3f4496 > > The only bit that is missing is removing the now unused support for > a NULL tp in __xfs_bunmapi.. Ah, I forgot about that patch. Thanks for the reminder, Christoph! Darrick, can we get Christoph's patch in as a standalone bug fix rather than wait for the bufferhead removal to be merged? Cheers, Dave. -- Dave Chinner david@fromorbit.com