On Thu, Dec 10, 2015 at 09:26:09PM -0700, Ross Zwisler wrote: > On Fri, Dec 11, 2015 at 09:33:33AM +1100, Dave Chinner wrote: > > On Thu, Dec 10, 2015 at 09:54:58AM -0700, Ross Zwisler wrote: > > > On Wed, Dec 02, 2015 at 02:39:32PM -0700, Ross Zwisler wrote: > > > > I've verified that this fixes all three failing xfstests reported in this mail. > > > > Thanks! > > > > > > Hey Dave, > > > > > > Are you planning on pushing this fix for v4.4? > > > > No plans to right now - ENOSPC is a corner case that most users > > won't be anywhere near, especially for experimental functionality on > > hardware nobody actually has.... > > Really? I realize that it may be a case that most users won't actually hit, > but it is a 5 line change that fixes four xfstests regressions between v4.3 and > v4.4 for my DAX testing... > > Is there a strong reason *not* to push it in the v4.4 cycle? I'm trying to > clear up all xfstests differences between DAX and non-DAX, and this would help > quite a bit. Against my better judgement, I committed a largely untested, fundamental change to allocation policy for DAX-on-XFS *inside the 4.4 merge window* justifying it as "it's experimental code" and that "we needed to get it out there". IOWs, I've already stretched the rules and committed stuff I knew wasn't ready or worked 100% correctly just to keep you guys happy, but I'm not going to continue to do so. I'm reverting to usual policy of "soak in for-next, use "cc: stable" tags to get it backported when upstreamed in the next merge window. Indeed, from my point of view it's pretty clear that lots of recent DAX code has been committed prematurely and without sufficient review and/or testing. This has lead to having to revert chunks of code and completely rework algorithms, I really don't care if you're being pushed to "make DAX shit happen fast" - my only concern here is *don't screw up production filesystems*. Hence when it comes to making fundamental changes to allocation behaviour, I've already bent the policy/rules as far as I'm willing to. Changes in this area carry an inherent risk of breaking stuff unrelated to DAX and that is far more important right now than whether DAX (and experimental feature) works correctly or not. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
