On Tue, 2013-11-12 at 09:14 +0200, Or Gerlitz wrote: > On 11/11/2013 23:02, Nicholas A. Bellinger wrote: > > Just wondering, is not FMR still required for T10 PI regardless for > > small IOs here..? > > > > Should this be depending upon if a protection memory region is in use, > > or do you plan to send that as a separate patch..? > > Nic, there will be changes to support T10, sure, still, this patch is > needed for 3.13 to make sure we don't havelower IOPS rate b/c of doing > FRWRs. The T10 support will come on top and involve more changes, way > beyond this one, but kernel development is done incrementally... The point is that if FMR is only required for future T10 PI support, why not just disable it by default, instead of just avoiding it for the small block count == 1 case..? --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html