[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130116010229.GB3942@liubo>
Date: Wed, 16 Jan 2013 09:02:30 +0800
From: Liu Bo <bo.li.liu@...cle.com>
To: Zach Brown <zab@...bo.net>
Cc: Christoph Lameter <cl@...ux.com>, linux-mm@...ck.org,
linux-btrfs@...r.kernel.org, linux-kernel@...r.kernel.org,
Pekka Enberg <penberg@...nel.org>
Subject: Re: [PATCH] mm/slab: add a leak decoder callback
On Tue, Jan 15, 2013 at 09:01:05AM -0800, Zach Brown wrote:
> > The merge processing occurs during kmem_cache_create and you are setting
> > up the decoder field afterwards! Wont work.
>
> In the thread I suggested providing the callback at destruction:
>
> http://www.mail-archive.com/linux-btrfs@vger.kernel.org/msg21130.html
>
> I liked that it limits accesibility of the callback to the only path
> that uses it.
Well, I was trying to avoid API change, but seems we have to, I'll
update the patch as your suggestion in the next version.
thanks,
liubo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists