Messages in this thread | | | Date | Thu, 6 Sep 2012 16:42:42 -0700 | From | Tejun Heo <> | Subject | Re: [PATCH v10 6/8] block: Kill bi_destructor |
| |
On Thu, Sep 06, 2012 at 03:35:00PM -0700, Kent Overstreet wrote: > Now that we've got generic code for freeing bios allocated from bio > pools, this isn't needed anymore. > > This patch also makes bio_free() static, since without bi_destructor > there should be no need for it to be called anywhere else. > > bio_free() is now only called from bio_put, so we can refactor those a > bit - move some code from bio_put() to bio_free() and kill the redundant > bio->bi_next = NULL. > > v5: Switch to BIO_KMALLOC_POOL ((void *)~0), per Boaz > v6: BIO_KMALLOC_POOL now NULL, drop bio_free's EXPORT_SYMBOL > v7: No #define BIO_KMALLOC_POOL anymore > > Signed-off-by: Kent Overstreet <koverstreet@google.com> > CC: Jens Axboe <axboe@kernel.dk>
Acked-by: Tejun Heo <tj@kernel.org>
Thanks.
-- tejun
| |