From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Snitzer Subject: Re: [PATCH v5 01/11] block: make generic_make_request handle arbitrarily sized bios Date: Tue, 18 Aug 2015 10:45:05 -0400 Message-ID: <20150818144505.GA15832@redhat.com> References: <20150801163356.GA21478@redhat.com> <1439099990.7880.0.camel@hasee> <20150809064128.GB2436@infradead.org> <1439103347.7880.3.camel@hasee> <20150809070128.GA25739@infradead.org> <1439104707.7880.8.camel@hasee> <20150810150216.GA22719@redhat.com> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Ming Lin Cc: Joe Thornber , device-mapper development , Christoph Hellwig , "Martin K. Petersen" , Kent Overstreet List-Id: dm-devel.ids On Tue, Aug 18 2015 at 3:04am -0400, Ming Lin wrote: > On Mon, Aug 17, 2015 at 10:09 PM, Ming Lin wrote: > > On Mon, Aug 10, 2015 at 8:02 AM, Mike Snitzer wrote: > >> p.s. I'll be working with Joe Thornber on optimizing DM (particularly > >> dm-thinp and dm-cache) once this patchset is included upstream. You'll > >> see I've already added a couple WIP dm-thinp patches ontop. > > > > Hi Mike, > > > > Just to avoid duplicated work. > > Are you going to work on the dm-thinp/dm-cache discard rewritten? > > Seems dm-stripe discard also needs rewrite. Can you elaborate on what you feel needs re-writing in these targets? This is the basic initial cleanup I had in mind for dm-thinp: http://git.kernel.org/cgit/linux/kernel/git/snitzer/linux.git/commit/?h=dm-4.4&id=cb0aca0a6bfad6b7f7146dde776f374082a73db6 A much more involved refactoring of the dm-cache and dm-thinp targets to eliminate the need for splitting will involve bio-prison range locking and a new metadata format for both targets to express ranges as opposed to blocks. This line of work is on Joe's radar but it is much further out given the associated on-disk metadata format change. That aside, I do need to look at DM core to see how we can do things differently so that block core's bio_split() et al is doing the splitting rather than DM core having a role. I'd prefer to be the one working these DM changes. But if you have ideas of how things should be cleaned up I'd be happy to consider them. Thanks, Mike