[Open-FCoE] [PATCH 3/6] libfcoe: fip: allow FIP receive to be called from IRQ.

[Joe Eykholt]

FIP's fcoe_ctlr_recv() function was previously only called from
the soft IRQ in FCoE.  It's not performance critical and is more
convenient for some drivers to call it from the IRQ level.  Just
Change to use skb_queue()/dequeue() which uses spinlock_irqsave
instead of separate locking with _bh locks.

Signed-off-by: Joe Eykholt <jeykh...@cisco.com>
---
 drivers/scsi/fcoe/libfcoe.c |   17 ++++-------------
 1 files changed, 4 insertions(+), 13 deletions(-)


diff --git a/drivers/scsi/fcoe/libfcoe.c b/drivers/scsi/fcoe/libfcoe.c
index e24bba9..216b2f5 100644
--- a/drivers/scsi/fcoe/libfcoe.c
+++ b/drivers/scsi/fcoe/libfcoe.c
@@ -151,9 +151,7 @@ static void fcoe_ctlr_reset_fcfs(struct fcoe_ctlr *fip)
 void fcoe_ctlr_destroy(struct fcoe_ctlr *fip)
 {
        cancel_work_sync(&fip->recv_work);
-       spin_lock_bh(&fip->fip_recv_list.lock);
-       __skb_queue_purge(&fip->fip_recv_list);
-       spin_unlock_bh(&fip->fip_recv_list.lock);
+       skb_queue_purge(&fip->fip_recv_list);
 
        spin_lock_bh(&fip->lock);
        fip->state = FIP_ST_DISABLED;
@@ -1000,13 +998,11 @@ static void fcoe_ctlr_recv_clr_vlink(struct fcoe_ctlr 
*fip,
  * @fip:       FCoE controller.
  * @skb:       Received FIP packet.
  *
- * This is called from NET_RX_SOFTIRQ.
+ * This may be called from either NET_RX_SOFTIRQ or IRQ.
  */
 void fcoe_ctlr_recv(struct fcoe_ctlr *fip, struct sk_buff *skb)
 {
-       spin_lock_bh(&fip->fip_recv_list.lock);
-       __skb_queue_tail(&fip->fip_recv_list, skb);
-       spin_unlock_bh(&fip->fip_recv_list.lock);
+       skb_queue_tail(&fip->fip_recv_list, skb);
        schedule_work(&fip->recv_work);
 }
 EXPORT_SYMBOL(fcoe_ctlr_recv);
@@ -1246,13 +1242,8 @@ static void fcoe_ctlr_recv_work(struct work_struct 
*recv_work)
        struct sk_buff *skb;
 
        fip = container_of(recv_work, struct fcoe_ctlr, recv_work);
-       spin_lock_bh(&fip->fip_recv_list.lock);
-       while ((skb = __skb_dequeue(&fip->fip_recv_list))) {
-               spin_unlock_bh(&fip->fip_recv_list.lock);
+       while ((skb = skb_dequeue(&fip->fip_recv_list)))
                fcoe_ctlr_recv_handler(fip, skb);
-               spin_lock_bh(&fip->fip_recv_list.lock);
-       }
-       spin_unlock_bh(&fip->fip_recv_list.lock);
 }
 
 /**


_______________________________________________
devel mailing list
devel@open-fcoe.org
http://www.open-fcoe.org/mailman/listinfo/devel