Messages in this thread | | | Date | Wed, 15 Feb 2012 10:43:41 -0500 | From | Vivek Goyal <> | Subject | Re: [PATCHSET] block, mempool, percpu: implement percpu mempool and fix blkcg percpu alloc deadlock |
| |
On Mon, Feb 13, 2012 at 02:31:56PM -0800, Tejun Heo wrote: > Hello, Vivek. > > On Fri, Feb 10, 2012 at 11:26:58AM -0500, Vivek Goyal wrote: > > The only difference is that by putting this logic in per cpu counters, > > we make it somewhat generic so that other users who can't do GFP_KERNEL > > allocation of per cpu data, can use it. I can live with that. > > Also, it has fallback mechanism while percpu data isn't there, so the > counts are guaranteed to be right. Probably doesn't matter all that > much for blkcg stats.
Ok, I had missed that part. Nice.
I have no objections to making this lazy allocation in per cpu counter idea work and migrate blkcg per cpu stats to use per cpu counters. If this patchset is not too intrusive, it might even be backportable to stable kernels.
Andrew does not seem to like mempool idea, so instead of not making any progress, it is better to go with this idea. We need to fix this issue. More poeple might run into this.
Thanks Vivek
| |