Re: [PATCH 17/18] io-controller: IO group refcounting support

From: Vivek Goyal
Date: Fri May 08 2009 - 08:50:36 EST


On Fri, May 08, 2009 at 10:59:31AM +0800, Gui Jianfeng wrote:
> Vivek Goyal wrote:
> ...
> >
> > - BUG_ON(!RB_EMPTY_ROOT(&st->active));
> > - BUG_ON(!RB_EMPTY_ROOT(&st->idle));
> > }
> >
> > - BUG_ON(iog->sched_data.next_active != NULL);
> > - BUG_ON(iog->sched_data.active_entity != NULL);
> > - BUG_ON(entity->tree != NULL);
> > + /* Drop iocg reference on io group */
>
> We'd better to reset iog->iocg_id here since the cgroup is going to destroy.
>

Hm.., that does not harm. Will do in next version.

Thanks
Vivek
--
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/