Re: [PATCH 4/7] dm: introduce dm_kvmalloc

From: Mike Snitzer
Date: Tue Jul 07 2015 - 09:25:55 EST


On Tue, Jul 07 2015 at 8:50am -0400,
Mikulas Patocka <mpatocka@xxxxxxxxxx> 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/