Commit 34743bfd authored by Mike Snitzer's avatar Mike Snitzer

dm rq: cleanup leftover code from recently removed q->mq_ops branching

When commit 6a23e05c ("dm: remove legacy request-based IO path")
removed some q->mq_ops branching from map_request() it left in place a
goto that was only needed if that branching (and conditional 'r'
assignment) existed.  Now that the branching is gone map_request()'s
goto can be removed too.
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent bbf6a566
...@@ -372,7 +372,6 @@ static int map_request(struct dm_rq_target_io *tio) ...@@ -372,7 +372,6 @@ static int map_request(struct dm_rq_target_io *tio)
blk_status_t ret; blk_status_t ret;
r = ti->type->clone_and_map_rq(ti, rq, &tio->info, &clone); r = ti->type->clone_and_map_rq(ti, rq, &tio->info, &clone);
check_again:
switch (r) { switch (r) {
case DM_MAPIO_SUBMITTED: case DM_MAPIO_SUBMITTED:
/* The target has taken the I/O to submit by itself later */ /* The target has taken the I/O to submit by itself later */
...@@ -392,8 +391,7 @@ static int map_request(struct dm_rq_target_io *tio) ...@@ -392,8 +391,7 @@ static int map_request(struct dm_rq_target_io *tio)
blk_rq_unprep_clone(clone); blk_rq_unprep_clone(clone);
tio->ti->type->release_clone_rq(clone); tio->ti->type->release_clone_rq(clone);
tio->clone = NULL; tio->clone = NULL;
r = DM_MAPIO_REQUEUE; return DM_MAPIO_REQUEUE;
goto check_again;
} }
break; break;
case DM_MAPIO_REQUEUE: case DM_MAPIO_REQUEUE:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment