Re: [PATCH 1/3] blk-mq: cleanup after blk_mq_init_rq_map failures

2014-07-18 Thread Christoph Hellwig
On Thu, Jul 17, 2014 at 02:39:09PM -0500, Robert Elliott wrote: > In blk-mq.c blk_mq_alloc_tag_set, if: > set->tags = kmalloc_node() > succeeds, but one of the blk_mq_init_rq_map() calls fails, > goto out_unwind; > needs to free set->tags so the caller is not obligated > to do so. None

[PATCH 1/3] blk-mq: cleanup after blk_mq_init_rq_map failures

2014-07-17 Thread Robert Elliott
In blk-mq.c blk_mq_alloc_tag_set, if: set->tags = kmalloc_node() succeeds, but one of the blk_mq_init_rq_map() calls fails, goto out_unwind; needs to free set->tags so the caller is not obligated to do so. None of the current callers (null_blk, virtio_blk, virtio_blk, or the forthc