Re: [PATCH] fs: sync: fixed performance regression

2013-07-15 Thread Jan Kara
On Fri 12-07-13 09:59:00, Paul Taysom wrote: > `On Fri, Jul 12, 2013 at 8:43 AM, Jan Kara wrote: > > On Thu 11-07-13 13:58:32, Jan Kara wrote: > >> On Thu 11-07-13 12:53:46, Jan Kara wrote: > >> > On Wed 10-07-13 16:12:36, Paul Taysom wrote: > >> > > The following commit introduced a 10x regressio

Re: [PATCH] fs: sync: fixed performance regression

2013-07-12 Thread Paul Taysom
`On Fri, Jul 12, 2013 at 8:43 AM, Jan Kara wrote: > On Thu 11-07-13 13:58:32, Jan Kara wrote: >> On Thu 11-07-13 12:53:46, Jan Kara wrote: >> > On Wed 10-07-13 16:12:36, Paul Taysom wrote: >> > > The following commit introduced a 10x regression for >> > > syncing inodes in ext4 with relatime enabl

Re: [PATCH] fs: sync: fixed performance regression

2013-07-12 Thread Jan Kara
On Thu 11-07-13 13:58:32, Jan Kara wrote: > On Thu 11-07-13 12:53:46, Jan Kara wrote: > > On Wed 10-07-13 16:12:36, Paul Taysom wrote: > > > The following commit introduced a 10x regression for > > > syncing inodes in ext4 with relatime enabled where just > > > the atime had been modified. > > > >

Re: [PATCH] fs: sync: fixed performance regression

2013-07-11 Thread Paul Taysom
On Thu, Jul 11, 2013 at 4:58 AM, Jan Kara wrote: > On Thu 11-07-13 12:53:46, Jan Kara wrote: >> On Wed 10-07-13 16:12:36, Paul Taysom wrote: >> > The following commit introduced a 10x regression for >> > syncing inodes in ext4 with relatime enabled where just >> > the atime had been modified. >> >

Re: [PATCH] fs: sync: fixed performance regression

2013-07-11 Thread Jan Kara
On Thu 11-07-13 12:53:46, Jan Kara wrote: > On Wed 10-07-13 16:12:36, Paul Taysom wrote: > > The following commit introduced a 10x regression for > > syncing inodes in ext4 with relatime enabled where just > > the atime had been modified. > > > > commit 4ea425b63a3dfeb7707fc7cc7161c11a51e871ed

Re: [PATCH] fs: sync: fixed performance regression

2013-07-11 Thread Jan Kara
On Wed 10-07-13 16:12:36, Paul Taysom wrote: > The following commit introduced a 10x regression for > syncing inodes in ext4 with relatime enabled where just > the atime had been modified. > > commit 4ea425b63a3dfeb7707fc7cc7161c11a51e871ed > Author: Jan Kara > Date: Tue Jul 3 16:45

Re: [PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Dave Chinner
On Wed, Jul 10, 2013 at 04:12:36PM -0700, Paul Taysom wrote: > The following commit introduced a 10x regression for > syncing inodes in ext4 with relatime enabled where just > the atime had been modified. > > commit 4ea425b63a3dfeb7707fc7cc7161c11a51e871ed > Author: Jan Kara > Date:

Re: [PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Paul Taysom
Third time is the charm crbug.com/240411 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Paul Taysom
On Wed, Jul 10, 2013 at 4:56 PM, Dave Jones wrote: > On Wed, Jul 10, 2013 at 04:12:36PM -0700, Paul Taysom wrote: > > > Note, when running the test, the slow down doesn't always happen > > but most of the tests will show a slow down. > > > > In response to crbug.com/240422 > > Just in case thi

Re: [PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Dave Jones
On Wed, Jul 10, 2013 at 04:12:36PM -0700, Paul Taysom wrote: > Note, when running the test, the slow down doesn't always happen > but most of the tests will show a slow down. > > In response to crbug.com/240422 Just in case this ends up as the actual commit msg, that url seems to be the wro

Re: [PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Paul Taysom
On Wed, Jul 10, 2013 at 4:12 PM, Paul Taysom wrote: > The following commit introduced a 10x regression for > syncing inodes in ext4 with relatime enabled where just > the atime had been modified. > > commit 4ea425b63a3dfeb7707fc7cc7161c11a51e871ed > Author: Jan Kara > Date: Tue Jul

[PATCH] fs: sync: fixed performance regression

2013-07-10 Thread Paul Taysom
The following commit introduced a 10x regression for syncing inodes in ext4 with relatime enabled where just the atime had been modified. commit 4ea425b63a3dfeb7707fc7cc7161c11a51e871ed Author: Jan Kara Date: Tue Jul 3 16:45:34 2012 +0200 vfs: Avoid unnecessary WB_SYNC_NONE writ