From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:55009 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751453AbcFVLnA (ORCPT ); Wed, 22 Jun 2016 07:43:00 -0400 Date: Wed, 22 Jun 2016 07:42:00 -0400 From: Brian Foster To: Holger =?iso-8859-1?Q?Hoffst=E4tte?= Cc: Jan Kara , linux-fsdevel@vger.kernel.org Subject: Re: What happened to the sb writeback list (aka sync efficiency) fix? Message-ID: <20160622114159.GA5423@bfoster.bfoster> References: <20160620164334.GB9474@laptop.bfoster> <20160621125446.GA9266@quack2.suse.cz> <20160621140302.GB62352@bfoster.bfoster> <57694B41.1080106@applied-asynchrony.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <57694B41.1080106@applied-asynchrony.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, Jun 21, 2016 at 04:12:17PM +0200, Holger Hoffst�tte wrote: > On 06/21/16 16:03, Brian Foster wrote: > > On Tue, Jun 21, 2016 at 02:54:46PM +0200, Jan Kara wrote: > >> On Mon 20-06-16 12:43:34, Brian Foster wrote: > >>> On Mon, Jun 20, 2016 at 01:46:38PM +0000, Holger Hoffst�tte wrote: > >>>> > >>>> Once upon a time there was this fine patch set called > >>>> "improve sync efficiency with sb inode wb list" [1] > >>>> by Brian Foster, who fixed up the original version by Josef Bacik. > >>>> > >>>> I've been running with this since then and it seems to work flawlessly, > >>>> yet it doesn't seem that this ever got merged..does anybody know why? > >>>> > >>>> Waiman Long has been working on something similar with his per-CPU > >>>> lists, but those patches naturally collide a bit, so I'm wondering > >>>> what's what. > >>>> > >>>> Fwiw the effect of the wb list on systems with many cached inodes is > >>>> phenomal; it would be a shame if this went unmerged. > >>>> > >>> > >>> FWIW, the latest version posted was v7: > >>> > >>> http://thread.gmane.org/gmane.linux.file-systems/104078 > >>> > >>> Thanks for the testing feedback. Unfortunately, I've not really heard > >>> any feedback on getting this merged. I'm not sure the previous > >>> version(s) by Josef and Dave got much traction either. :( > >> > >> Actually it would be a pitty to have this dropped. Please rebase & resend > >> the patches and send them to AKPM asking him to merge them because Al isn't > >> replying... That should make things moving. > >> > > > > Sure, it rebases clean so I can post a v8 after I get a chance to run > > some regression tests. > > > > Holger, > > > > Care to offer a Tested-by? > > My pleasure: > > Tested-by: Holger Hoffst�tte > Should this be "holger.hoffstatte@..." (note the dropped 'e')? I got a return to sender for the above on my recent post. Probably not worth resending, but you could reply to the post on-list with an accurate email and I'll fix it up locally as well... Brian > I did this on my custom 4.4.x++ frankenkernel, but I don't see why it > wouldn't work on mainline. > > cheers, > Holger > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html