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@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
