From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com [209.85.212.176]) by kanga.kvack.org (Postfix) with ESMTP id 391076B0032 for ; Wed, 8 Apr 2015 12:13:58 -0400 (EDT) Received: by wizk4 with SMTP id k4so61149543wiz.1 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-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Jeff Moyer Cc: Jens Axboe , Ming Lei , Konrad Rzeszutek Wilk , Roger Pau Monn?? , Alasdair Kergon , Mike Snitzer , Neil Brown , "Nicholas A. Bellinger" , Alexander Viro , Chris Mason , Josef Bacik , David Sterba , Theodore Ts'o , Andreas Dilger , Jaegeuk Kim , Changman Lee , Steven Whitehouse , Mikulas Patocka , Andrew Morton , Rik van Riel , Johannes Weiner , Mel Gorman , Trond Myklebust , Anna Schumaker , Dave Chinner , xfs@oss.sgi.com, Christoph Hellwig , Weston Andros Adamson , "Martin K. Petersen" , Sagi Grimberg , Tejun Heo , Fabian Frederick , Matthew Wilcox , Ming Lei , "Kirill A. Shutemov" , Wang Sheng-Hui , Michal Hocko , Joe Perches , Miklos Szeredi , Namjae Jeon , Mark Rustad , Jianyu Zhan , Fengguang Wu , Vladimir Davydov , Vlastimil Babka , Suleiman Souhlal , linux-kernel@vger.kernel.org, dm-devel@redhat.com, xen-devel@lists.xenproject.org, linux-raid@vger.kernel.org, linux-scsi@vger.kernel.org, target-devel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-aio@kvack.org, linux-btrfs@vger.kernel.org, linux-ext4@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com, linux-nfs@vger.kernel.org, linux-mm@kvack.org 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? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org