Messages in this thread |  | | | Date | Wed, 3 Jun 2009 09:50:19 +0200 | | From | Jens Axboe <> | | Subject | Re: [PATCH 0/11] Per-bdi writeback flusher threads v9 |
| |
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. > > Ah, I see. Rather non-typical convention though. I expected > bdi_register() to clean-up stuff in case of failure. Isn't > it a better interface?
You already did a bdi_init() at that point. bdi_destroy() must be used to clean up after both bdi_init() and/or bdi_register().
-- Jens Axboe
|  |