On Wed, Jun 03 2009, Artem Bityutskiy wrote: > Jens Axboe wrote: >>> Just to make sure I understood you correctly. I assume my original >>> patch is fine (because there is bdi_destroy()) and merge it to >>> ubifs tree. >> >> It needs to be: >> >> err = bdi_register(&c->bdi, NULL, "ubifs"); >> if (err) >> goto out_bdi; >> >> so you hit the bdi_destroy() for that failure, not goto out_close; >> Otherwise it was fine. > > Did this, also added a > Reviewed-by: Jens Axboe <jens.axboe@xxxxxxxxxx> > > http://git.infradead.org/ubifs-2.6.git?a=commit;h=813fdc16ad591e79d0c1b88d31970dcd1c2aa3f1 Looks good! -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
