From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alasdair G Kergon Subject: Re: [dm-devel] [PATCH 12/13] Make generic_make_request handle arbitrarily large bios Date: Sat, 19 May 2012 01:59:32 +0100 Message-ID: <20120519005932.GM29330@agk-dp.fab.redhat.com> References: <1114e7019b0055fc09a54b59b36398d5c54f5e32.1337308722.git.koverstreet@google.com> <20120518175216.GL19388@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: koverstreet@google.com, axboe@kernel.dk, dm-devel@redhat.com, linux-kernel@vger.kernel.org, linux-bcache@vger.kernel.org, linux-fsdevel@vger.kernel.org, agk@redhat.com To: Tejun Heo Return-path: Received: from mx1.redhat.com ([209.132.183.28]:13470 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759853Ab2ESA7n (ORCPT ); Fri, 18 May 2012 20:59:43 -0400 Content-Disposition: inline In-Reply-To: <20120518175216.GL19388@google.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri, May 18, 2012 at 10:52:16AM -0700, Tejun Heo wrote: > Does this by any chance allow killing ->merge_bvec_fn()? merge_bvec also has an impact on efficiency. Why create a too-large bio in the first place if it's only going to have to get split up again? If a change like this goes in, the various trade-offs should be re-examined. Alasdair