diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-04-20 12:16:54 -0700 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-05-10 08:59:26 +0100 |
commit | b3b8abd85780e1bb92703354f3c16c921edfa4f6 (patch) | |
tree | ba0694c5b5924addf4ab95b585bd8a4dd9610e5d /drivers/scsi/fcoe | |
parent | d227f029c2e1e7c226372a6c11969d33ed146f6f (diff) | |
download | talos-obmc-linux-b3b8abd85780e1bb92703354f3c16c921edfa4f6.tar.gz talos-obmc-linux-b3b8abd85780e1bb92703354f3c16c921edfa4f6.zip |
[SCSI] fcoe: remove a stray unlock
We moved the locking in dd060e74fb "[SCSI] fcoe: remove frame dropping
code from fcoe_percpu_clean" but this unlock was missed.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/fcoe')
-rw-r--r-- | drivers/scsi/fcoe/fcoe.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c index ac1df93de3d8..76e3d0b5bfa6 100644 --- a/drivers/scsi/fcoe/fcoe.c +++ b/drivers/scsi/fcoe/fcoe.c @@ -2294,10 +2294,9 @@ static void fcoe_percpu_clean(struct fc_lport *lport) continue; skb = dev_alloc_skb(0); - if (!skb) { - spin_unlock_bh(&pp->fcoe_rx_list.lock); + if (!skb) continue; - } + skb->destructor = fcoe_percpu_flush_done; spin_lock_bh(&pp->fcoe_rx_list.lock); |