Messages in this thread | | | Date | Fri, 17 Feb 2012 17:56:15 -0500 | From | Vivek Goyal <> | Subject | Re: [PATCH 7/9] block: implement bio_associate_current() |
| |
On Fri, Feb 17, 2012 at 02:41:03PM -0800, Tejun Heo wrote: > On Fri, Feb 17, 2012 at 05:34:20PM -0500, Vivek Goyal wrote: > > Nope. We make note of task migration and drop cic->cfqq association > > and establish a new association where new cfqq is part of new cgroup. > > (ioc_cgroup_changed()). > > Yeah, that's the CHANGED bit thing. I probably got confused with > blk-throttle losing blkcg while testing.
BTW, blk-throttle mangling the context, is a real problem you are facing or it is just one of things which is nice to fix. If a cgroup is being throttled do people really care about the iopriority of original task that much. (Given the fact that iopriority helps only so much and kills performance on fast storage).
Thanks Vivek
| |