On Wed 01-02-17 14:45:20, Jens Axboe wrote: > 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. Yes, will send it later today. I was just waiting whether someone else does not have more comments to the series. Honza -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-block" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
