summaryrefslogtreecommitdiffstats
path: root/drivers/scsi/fcoe
diff options
context:
space:
mode:
authorJiri Slaby2015-10-06 14:27:45 +0200
committerJames Bottomley2015-11-10 02:06:37 +0100
commit481ebb3daac39c6c1a8f78f98b823a9c9591616f (patch)
tree7c99b707e6c352536d455c1702b7f0abbcc79fd0 /drivers/scsi/fcoe
parentscsi: Fix a bdi reregistration race (diff)
downloadkernel-qcow2-linux-481ebb3daac39c6c1a8f78f98b823a9c9591616f.tar.gz
kernel-qcow2-linux-481ebb3daac39c6c1a8f78f98b823a9c9591616f.tar.xz
kernel-qcow2-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.c3
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);