diff options
| author | Jiri Slaby <jslaby@suse.cz> | 2015-10-06 14:27:45 +0200 |
|---|---|---|
| committer | James Bottomley <JBottomley@Odin.com> | 2015-11-10 02:06:37 +0100 |
| commit | 481ebb3daac39c6c1a8f78f98b823a9c9591616f (patch) | |
| tree | 7c99b707e6c352536d455c1702b7f0abbcc79fd0 /drivers/scsi/fcoe | |
| parent | scsi: Fix a bdi reregistration race (diff) | |
| download | linux-481ebb3daac39c6c1a8f78f98b823a9c9591616f.tar.xz linux-481ebb3daac39c6c1a8f78f98b823a9c9591616f.zip | |
fcoe: use continue instead of goto+label
There is a label pointing to the start of a while loop and a goto
nested only in the loop. The goto jumps to the label in some cases.
Replace the goto and the label by simple continue.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Acked-by: Vasu Dev <vasu.dev@intel.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
Diffstat (limited to 'drivers/scsi/fcoe')
| -rw-r--r-- | drivers/scsi/fcoe/fcoe.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c index d3eb80c46bbe..d187ba9b361d 100644 --- a/drivers/scsi/fcoe/fcoe.c +++ b/drivers/scsi/fcoe/fcoe.c @@ -1873,7 +1873,6 @@ static int fcoe_percpu_receive_thread(void *arg) set_user_nice(current, MIN_NICE); -retry: while (!kthread_should_stop()) { spin_lock_bh(&p->fcoe_rx_list.lock); @@ -1883,7 +1882,7 @@ retry: set_current_state(TASK_INTERRUPTIBLE); spin_unlock_bh(&p->fcoe_rx_list.lock); schedule(); - goto retry; + continue; } spin_unlock_bh(&p->fcoe_rx_list.lock); |