Messages in this thread | | | Date | Fri, 24 Feb 2012 09:20:33 -0500 | From | Vivek Goyal <> | Subject | Re: [PATCHSET] mempool, percpu, blkcg: fix percpu stat allocation and remove stats_lock |
| |
On Thu, Feb 23, 2012 at 03:12:04PM -0800, Tejun Heo wrote: > On Thu, Feb 23, 2012 at 03:01:23PM -0800, Tejun Heo wrote: > > Hmmm... going through the thread again, ah, okay, I forgot about that > > completely. Yeah, that is an actual problem. Both __GFP_WAIT which > > isn't GFP_KERNEL and GFP_KERNEL are valid use cases. I guess we'll be > > building async percpu pool in blkcg then. Great. :( > > Vivek, you win. :) Can you please refresh the async alloc patch on top > of blkcg-stacking branch? I'll rool that into this series and drop > the mempool stuff. >
Ok. I will write a patch. Things have changed a lot since last time. I think there is only one tricky part and that is waiting for any scheduled work to finish during blkg destruction. Because group destruction happens under both queue and blkcg spin locks, I think I will have to take the group off list, drop locks, wait for worker thread to finish and then take locks again and walk through list again to kill remaining groups.
/me goes to try it.
Thanks Vivek
| |