On Fri, May 03, 2013 at 05:50:44PM -0700, Tejun Heo wrote: .... > +static struct bio *throtl_pop_queued(struct list_head *queued, > + struct throtl_grp **tg_to_put) > +{ > + struct throtl_qnode *qn = list_first_entry(queued, struct throtl_qnode, > node); > + struct bio *bio; > + > + if (list_empty(queued)) > + return NULL; > + > + bio = bio_list_pop(&qn->bios); > + WARN_ON_ONCE(!bio); > + > + if (bio_list_empty(&qn->bios)) { > + list_del_init(&qn->node); > + if (tg_to_put) > + *tg_to_put = qn->tg; > + else > + blkg_put(tg_to_blkg(tg_to_put));
Oops, this should have been blkg_put(tg_to_blkg(qn->tg)); Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/