From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 30 Nov 2018 09:43:52 +1100 From: Dave Chinner To: Amir Goldstein Cc: sashal@kernel.org, stable , linux-kernel , Dave Chinner , "Darrick J. Wong" , linux-fsdevel , "Luis R. Chamberlain" Subject: Re: [PATCH AUTOSEL 4.19 52/68] iomap: sub-block dio needs to zeroout beyond EOF Message-ID: <20181129224352.GN19305@dastard> References: <20181129055559.159228-1-sashal@kernel.org> <20181129055559.159228-52-sashal@kernel.org> <20181129121922.GL19305@dastard> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: List-ID: On Thu, Nov 29, 2018 at 02:36:50PM +0200, Amir Goldstein wrote: > > Same again - what's the test plan for these cherry-picked data > > corruption fixes? > > > > Dave, > > Just to check if we are on the same page, if this was the test plan: > https://www.spinics.net/lists/linux-xfs/msg20639.html > > Would "no regressions from baseline" have been sufficient to validate > those specific patches are solid for stable? No, not in this case, because fsx in fstests only does 100k ops at most - it's a smoke test. This isn't sufficient to regression test fixes that, in some cases, took hundreds of millions of fsx ops to expose. Cheers, Dave. -- Dave Chinner david@fromorbit.com