On 02/01/2017 04:22 AM, Jan Kara wrote: > On Wed 01-02-17 01:50:07, Christoph Hellwig wrote: >> On Tue, Jan 31, 2017 at 01:54:28PM +0100, Jan Kara wrote: >>> Instead of storing backing_dev_info inside struct request_queue, >>> allocate it dynamically, reference count it, and free it when the last >>> reference is dropped. Currently only request_queue holds the reference >>> but in the following patch we add other users referencing >>> backing_dev_info. >> >> Do we really need the separate slab cache? Otherwise this looks >> fine to me. > > Yeah, probably it is not worth it. I'll remove it.
I agree on that, it should not be a hot path. Will you respin the series after making this change? Would be great to get this queued up. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-block" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
