Re: [PATCH 5/8] fcoe: reduces lock cost when adding a new skb to fcoe_pending_queue

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

 



On Wed, 2009-05-06 at 10:52 -0700, Robert Love wrote:
> From: Vasu Dev <vasu.dev@xxxxxxxxx>
> 
> Currently fcoe_pending_queue.lock held twice for every new skb
> adding to this queue when already least one pkt is pending in this
> queue and that is not uncommon once skb pkts starts getting queued
> here upon fcoe_start_io => dev_queue_xmit failure.
> 
> This patch moves most fcoe_pending_queue logic to fcoe_check_wait_queue
> function, this new logic grabs fcoe_pending_queue.lock only once to
> add a new skb instead twice as used to be.
> 
> I think after this patch call flow around fcoe_check_wait_queue
> calling in fcoe_xmit is bit simplified with modified
> fcoe_check_wait_queue function taking care of adding and
> removing pending skb in one function.

This isn't really a -rc4 bug fix, is it?  It reads a lot more like a
feature enhancement that was given a bug like description.

James


--
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