Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753338AbbFAGPS (ORCPT ); Mon, 1 Jun 2015 02:15:18 -0400 Received: from mail.kernel.org ([198.145.29.136]:37975 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751505AbbFAGPN convert rfc822-to-8bit (ORCPT ); Mon, 1 Jun 2015 02:15:13 -0400 MIME-Version: 1.0 In-Reply-To: <20150523141522.GA19470@lst.de> References: <1432318723-18829-1-git-send-email-mlin@kernel.org> <20150523141522.GA19470@lst.de> Date: Sun, 31 May 2015 23:15:09 -0700 Message-ID: Subject: Re: [PATCH v4 00/11] simplify block layer based on immutable biovecs From: Ming Lin To: Christoph Hellwig Cc: lkml , Kent Overstreet , Jens Axboe , Dongsu Park Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1520 Lines: 30 On Sat, May 23, 2015 at 7:15 AM, Christoph Hellwig wrote: > On Fri, May 22, 2015 at 11:18:32AM -0700, Ming Lin wrote: >> This will bring not only performance improvements, but also a great amount >> of reduction in code complexity all over the block layer. Performance gain >> is possible due to the fact that bio_add_page() does not have to check >> unnecesary conditions such as queue limits or if biovecs are mergeable. >> Those will be delegated to the driver level. Kent already said that he >> actually benchmarked the impact of this with fio on a micron p320h, which >> showed definitely a positive impact. > > We'll need some actual numbers. I actually like these changes a lot > and don't even need a performance justification for this fundamentally > better model, but I'd really prefer to avoid any large scale regressions. > I don't really expect them, but for code this fundamental we'll just > need some benchmarks. > > Except for that these changes looks good, and the previous version > passed my tests fine, so with some benchmarks you'ĺl have my ACK. Can I have your ACK with these numbers? https://lkml.org/lkml/2015/6/1/38 > > I'd love to see this go into 4.2, but for that we'll need Jens > approval and a merge into for-next very soon. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/