Re: [PATCH] blk-mq-sched: fix possible crash if changing scheduler fails

2017-01-24 Thread Jens Axboe
On 01/23/2017 04:47 PM, Omar Sandoval wrote: > From: Omar Sandoval > > In elevator_switch(), we free the old scheduler's tags and then > initialize the new scheduler. If initializing the new scheduler fails, > we fall back to the old scheduler, but our tags have already been

[PATCH] blk-mq-sched: fix possible crash if changing scheduler fails

2017-01-23 Thread Omar Sandoval
From: Omar Sandoval In elevator_switch(), we free the old scheduler's tags and then initialize the new scheduler. If initializing the new scheduler fails, we fall back to the old scheduler, but our tags have already been freed. There's no reason to free the sched_tags only to