File tree Expand file tree Collapse file tree
Expand file tree Collapse file tree Original file line number Diff line number Diff line change @@ -719,7 +719,8 @@ void elevator_set_default(struct request_queue *q)
719719 .name = "mq-deadline" ,
720720 .no_uevent = true,
721721 };
722- int err = 0 ;
722+ int err ;
723+ struct elevator_type * e ;
723724
724725 /* now we allow to switch elevator */
725726 blk_queue_flag_clear (QUEUE_FLAG_NO_ELV_SWITCH , q );
@@ -732,12 +733,18 @@ void elevator_set_default(struct request_queue *q)
732733 * have multiple queues or mq-deadline is not available, default
733734 * to "none".
734735 */
735- if (elevator_find_get (ctx .name ) && (q -> nr_hw_queues == 1 ||
736- blk_mq_is_shared_tags (q -> tag_set -> flags )))
736+ e = elevator_find_get (ctx .name );
737+ if (!e )
738+ return ;
739+
740+ if ((q -> nr_hw_queues == 1 ||
741+ blk_mq_is_shared_tags (q -> tag_set -> flags ))) {
737742 err = elevator_change (q , & ctx );
738- if (err < 0 )
739- pr_warn ("\"%s\" elevator initialization, failed %d, "
740- "falling back to \"none\"\n" , ctx .name , err );
743+ if (err < 0 )
744+ pr_warn ("\"%s\" elevator initialization, failed %d, falling back to \"none\"\n" ,
745+ ctx .name , err );
746+ }
747+ elevator_put (e );
741748}
742749
743750void elevator_set_none (struct request_queue * q )
You can’t perform that action at this time.
0 commit comments