[PATCH 0/3] Address review comments for fnic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The following series addresses Mike C.'s review comments for fnic:
    1) Use static workQ item per fnic for link transitions
    2) Use static workQ item and skb queue per fnic for FCS frames
    3) Use shost_printk to print host_no instead of printing fnic_no

(1) and (2) address the comment that allocating memory in the critical path
for link transitions and FCS frames (fabric login etc) is not good since
failure in memory allocation can prove fatal. The patches get rid of the
event structure and its allocation, and instead use static workQ items.

---

Abhijeet Joglekar (3):
      fnic: Use shost_printk and print host no in print messages
      fnic: Change FCS frame handling to use skb queues and no events
      fnic: Change link transition handling


 drivers/scsi/fnic/fnic.h      |   45 +++-------
 drivers/scsi/fnic/fnic_fcs.c  |  155 +++++++++++++++++++--------------
 drivers/scsi/fnic/fnic_isr.c  |   15 ++-
 drivers/scsi/fnic/fnic_main.c |  186 +++++++++++++++-------------------------
 drivers/scsi/fnic/fnic_res.c  |   80 ++++++++++-------
 drivers/scsi/fnic/fnic_scsi.c |  192 ++++++++++++++++++++---------------------
 6 files changed, 322 insertions(+), 351 deletions(-)

-- 
Signature
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux