Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030520Ab2HXU6W (ORCPT ); Fri, 24 Aug 2012 16:58:22 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:41657 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030240Ab2HXU6R (ORCPT ); Fri, 24 Aug 2012 16:58:17 -0400 Date: Fri, 24 Aug 2012 13:58:12 -0700 From: Tejun Heo To: "Martin K. Petersen" Cc: Kent Overstreet , linux-bcache@vger.kernel.org, linux-kernel@vger.kernel.org, dm-devel@redhat.com, vgoyal@redhat.com, mpatocka@redhat.com, bharrosh@panasas.com, Jens Axboe , NeilBrown , Lars Ellenberg , Peter Osterlund , Sage Weil Subject: Re: [PATCH v6 11/13] block: Rework bio_pair_split() Message-ID: <20120824205812.GJ21325@google.com> References: <1345655050-28199-1-git-send-email-koverstreet@google.com> <1345655050-28199-12-git-send-email-koverstreet@google.com> <20120822210410.GL19212@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1329 Lines: 29 Hello, Martin. On Thu, Aug 23, 2012 at 10:25:47PM -0400, Martin K. Petersen wrote: > However, I'm not sure I like the overall approach of the new splitting. > Instead of all this cloning, slicing and dicing of bio_vecs I'd rather > we bit the bullet and had an offset + length for the vector inside each > bio. That way we could keep the bio_vec immutable and make clones more > lightweight since their vecs would always point to the parent. This also > makes it trivial to split I/Os in the stacking drivers and removes evils > in the partial completion code path. It would also allow to sever the > ties between "size of block range operated on" vs. bi_size which we need > for copy offload, discard, etc. Yeah, I'm fairly sure we all want that but that's gonna have to be a separate not-so-small project. Also, how we split underlying bvec doesn't affect the split interface update being done here. This patchset is updating split so that it can handle arbirarily sized bios, which is useful whether its implementation is using immutable bvec or COWing it. Thanks. -- tejun -- 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/