From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Snitzer Subject: Re: [PATCH 4/7] dm: introduce dm_kvmalloc Date: Tue, 7 Jul 2015 09:25:42 -0400 Message-ID: <20150707132542.GA15860@redhat.com> References: <20150706134750.GA31600@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Mikulas Patocka Cc: "Alasdair G. Kergon" , Edward Thornber , dm-devel@redhat.com, David Rientjes , Andrew Morton , linux-kernel@vger.kernel.org List-Id: dm-devel.ids On Tue, Jul 07 2015 at 8:50am -0400, Mikulas Patocka wrote: > > > On Mon, 6 Jul 2015, Mike Snitzer wrote: > > > In general I like what you've done with this patchset _except_ I'm not > > seeing why dm_kvmalloc() should be in DM at all. It should probably be > > elevated to an kvmalloc() export from mm/util.c and include/linux/mm.h > > along-side kvfree(). > > If memory management maintainers agree to put it into mm core, it would be > nice. If not, put it into device mapper. Please rework your patches so that the mm changes are split out into clean patch(es) at the start of the series. Please cc Andrew and David on posting v2. Also, please chain the reply so that the patchset is properly threaded.