From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 55BF57F73 for ; Fri, 10 Apr 2015 23:12:04 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id 48DAB8F808E for ; Fri, 10 Apr 2015 21:12:01 -0700 (PDT) Received: from mail-ig0-f172.google.com (mail-ig0-f172.google.com [209.85.213.172]) by cuda.sgi.com with ESMTP id KKubCpHFkbUYR0Fw (version=TLSv1 cipher=RC4-SHA bits=128 verify=NO) for ; Fri, 10 Apr 2015 21:11:59 -0700 (PDT) Received: by igblo3 with SMTP id lo3so12588365igb.1 for ; Fri, 10 Apr 2015 21:11:59 -0700 (PDT) Message-ID: <55289F0A.1040309@gmail.com> Date: Sat, 11 Apr 2015 00:11:54 -0400 From: nick MIME-Version: 1.0 Subject: Re: [f2fs-dev] [PATCH 2/2][v2] blk-plug: don't flush nested plug lists References: <1428347694-17704-1-git-send-email-jmoyer@redhat.com> <1428347694-17704-2-git-send-email-jmoyer@redhat.com> <20150408230203.GG15810@dastard> In-Reply-To: List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Jeff Moyer , Dave Chinner Cc: Vladimir Davydov , linux-aio@kvack.org, Miklos Szeredi , Mike Snitzer , Ming Lei , Ming Lei , Trond Myklebust , Jianyu Zhan , "Nicholas A. Bellinger" , David Sterba , Sagi Grimberg , Chris Mason , dm-devel@redhat.com, target-devel@vger.kernel.org, Andreas Dilger , Mikulas Patocka , Mark Rustad , Christoph Hellwig , Alasdair Kergon , Roger Pau Monn?? , linux-scsi@vger.kernel.org, Namjae Jeon , Michal Hocko , cluster-devel@redhat.com, Mel Gorman , Suleiman Souhlal , linux-ext4@vger.kernel.org, Rik van Riel , Konrad Rzeszutek Wilk , xfs@oss.sgi.com, linux-raid@vger.kernel.org, Tejun Heo , Matthew Wilcox , xen-devel@lists.xenproject.org, Jaegeuk Kim , Steven Whitehouse , Vlastimil Babka , Jens Axboe , Fabian Frederick , linux-nfs@vger.kernel.org, Weston Andros Adamson , Theodore Ts'o , "Martin K. Petersen" , linux-mm@kvack.org, Josef Bacik , Wang Sheng-Hui , linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, "Kirill A. Shutemov" , Johannes Weiner , Joe Perches , linux-fsdevel@vger.kernel.org, Andrew Morton , Fengguang Wu , Anna Schumaker , linux-btrfs@vger.kernel.org, Alexander Viro On 2015-04-10 05:50 PM, Jeff Moyer wrote: > Dave Chinner writes: > >> On Tue, Apr 07, 2015 at 02:55:13PM -0400, Jeff Moyer wrote: >>> The way the on-stack plugging currently works, each nesting level >>> flushes its own list of I/Os. This can be less than optimal (read >>> awful) for certain workloads. For example, consider an application >>> that issues asynchronous O_DIRECT I/Os. It can send down a bunch of >>> I/Os together in a single io_submit call, only to have each of them >>> dispatched individually down in the bowels of the dirct I/O code. >>> The reason is that there are blk_plug-s instantiated both at the upper >>> call site in do_io_submit and down in do_direct_IO. The latter will >>> submit as little as 1 I/O at a time (if you have a small enough I/O >>> size) instead of performing the batching that the plugging >>> infrastructure is supposed to provide. >> >> I'm wondering what impact this will have on filesystem metadata IO >> that needs to be issued immediately. e.g. we are doing writeback, so >> there is a high level plug in place and we need to page in btree >> blocks to do extent allocation. We do readahead at this point, >> but it looks like this change will prevent the readahead from being >> issued by the unplug in xfs_buf_iosubmit(). > > I'm not ignoring you, Dave, I'm just doing some more investigation and > testing. It's taking longer than I had hoped. > > -Jeff > Jeff, Would you mind sending your test reports to the list so we can see what workloads and tests your running your patch under. This is due to me and the others perhaps being able to give input into the other major benchmarks or workloads we need to test too in order to see if there are any regressions with your patch. Thanks, Nick > ------------------------------------------------------------------------------ > BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT > Develop your own process in accordance with the BPMN 2 standard > Learn Process modeling best practices with Bonita BPM through live exercises > http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_ > source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF > _______________________________________________ > Linux-f2fs-devel mailing list > Linux-f2fs-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel > _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs