Re: possible fsync02() xfs slowness regression on power7

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Mar 03, 2013 at 11:23:26PM -0500, CAI Qian wrote:
> 
> > > And this commit in 3.9-rc1:
> > > 
> > > a1e16c2 xfs: limit speculative prealloc size on sparse files
> > > should fix the problem. Please confirm these commits are the cause
> > > and the fix respectively....
> Confirmed this fixed the problem. I'd like to request this to be back-ported
> to stable-3.0, stable-3.4 and stable-3.8. What do you think?

IMO, no, it is not a backport candidate.  The patch has quite a few
dependencies, and at least for 3.0 xfs_bmapi_read() doesn't exist
and hence is not a trivial backport.

Further, it's take 2 years for this to be noticed, and you haven't
explained why the problem exists on your power machine and not any
others that it has been tested on. And there's been very few
complaints about performance of such workloads over the past 2
years, so either the workload is not important or only your power7
machine is having problems.

Hence I don't see any need to back port it - it's not a critical fix
and very few people see the problem so there's no real need to do
the backport.  Maybe someone else has the time and resources to
waste on backporting non-critical fixes to stable kernels, but I
don't....

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux