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 B1CB77F7C for ; Wed, 8 Apr 2015 11:14:02 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id A10FC8F8068 for ; Wed, 8 Apr 2015 09:13:59 -0700 (PDT) Received: from newverein.lst.de (verein.lst.de [213.95.11.211]) by cuda.sgi.com with ESMTP id E8cJkRi91pvcA291 (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO) for ; Wed, 08 Apr 2015 09:13:57 -0700 (PDT) Date: Wed, 8 Apr 2015 18:13:55 +0200 From: Christoph Hellwig Subject: Re: [PATCH 2/2][v2] blk-plug: don't flush nested plug lists Message-ID: <20150408161355.GA16052@lst.de> References: <1428347694-17704-1-git-send-email-jmoyer@redhat.com> <1428347694-17704-2-git-send-email-jmoyer@redhat.com> MIME-Version: 1.0 Content-Disposition: inline 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 Cc: Vladimir Davydov , linux-aio@kvack.org, Miklos Szeredi , Mike Snitzer , Neil Brown , Ming Lei , Ming Lei , Trond Myklebust , Jianyu Zhan , "Nicholas A. Bellinger" , linux-kernel@vger.kernel.org, Sagi Grimberg , Chris Mason , dm-devel@redhat.com, target-devel@vger.kernel.org, Andreas Dilger , Mikulas Patocka , Mark Rustad , Christoph Hellwig , Alasdair Kergon , Matthew Wilcox , linux-scsi@vger.kernel.org, Namjae Jeon , linux-raid@vger.kernel.org, cluster-devel@redhat.com, Mel Gorman , Suleiman Souhlal , linux-ext4@vger.kernel.org, linux-mm@kvack.org, Changman Lee , Rik van Riel , Konrad Rzeszutek Wilk , xfs@oss.sgi.com, Fabian Frederick , Joe Perches , Alexander Viro , xen-devel@lists.xenproject.org, Jaegeuk Kim , Steven Whitehouse , Vlastimil Babka , Jens Axboe , Michal Hocko , linux-nfs@vger.kernel.org, Fengguang Wu , Theodore Ts'o , "Martin K. Petersen" , Wang Sheng-Hui , Josef Bacik , David Sterba , linux-f2fs-devel@lists.sourceforge.net, linux-btrfs@vger.kernel.org, Johannes Weiner , Tejun Heo , linux-fsdevel@vger.kernel.org, Andrew Morton , Weston Andros Adamson , Anna Schumaker , "Kirill A. Shutemov" , Roger Pau Monn?? This looks good, but without the blk_finish_plug argument we're bound to grow programming mistakes where people forget it. Any chance we could have annotations similar to say rcu_read_lock/rcu_read_unlock or the spinlocks so that sparse warns about it? _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs