<br><font size=2 face="sans-serif">Hello Michael,</font>
<br>
<br><font size=2><tt>"Michael S. Tsirkin" <mst@mellanox.co.il>
wrote on 04/04/2006 10:22:08 AM:<br>
<br>
> Quoting r. Shirley Ma <xma@us.ibm.com>:<br>
> > Subject: Re: [PATCH] repost: IPoIB queue size tune patch<br>
> > <br>
> > <br>
> > Hello Roland,<br>
> > <br>
> > Thanks.<br>
> > <br>
> > Roland Dreier <rdreier@cisco.com> wrote on 04/03/2006 10:50:37
PM:<br>
> > <br>
> > > Sorry I hadn't gotten a chance to read this over until now...<br>
> > ><br>
> > >  > -       IPOIB_RX_RING_SIZE  
     = 128,<br>
> > >  > -       IPOIB_TX_RING_SIZE  
     = 64,<br>
> > >  > +       IPOIB_SENDQ_SIZE  
       = 64,<br>
> > >  > +       IPOIB_RECVQ_SIZE  
       = 128,<br>
> > ><br>
> > > Can you explain again why it's a good idea to rename these?
 Is the<br>
> > > name "sendq_size" really clearer than "tx_ring_size,"
especially in<br>
> > > the context of a network driver?<br>
> > <br>
> > tx_ring and rx_ring are not good names for IPoIB. tx_ring in
IPoIBis a place<br>
> > to save pointers and rx_ring a place to allocate skb buffs.<br>
> <br>
> Actually names make sense to me - they are cyclic buffers, hence ring.<br>
> <br>
> > I am working on<br>
> > a patch to remove tx_ring and replace rx_ring with a list, which
would<br>
> > reduce some tx_ring overhead.<br>
> <br>
> What overhead?  Please don't, lists are bad for cache, require
locks ...<br>
> Circular buffers are much better, I don't have the time now but I
think we can<br>
> even make event handler completely lockless with them.<br>
</tt></font>
<br><font size=2><tt>IPoIB is not a real driver. Packets can be dropped
from dev xmit queue to </tt></font>
<br><font size=2><tt>QPs send queue directly. Why we use tx_ring to induce
overhead in </tt></font>
<br><font size=2><tt>send path?</tt></font>
<br>
<br><font size=2><tt>Also, using a list doesn't always mean to induce a
lock. I will test the patch</tt></font>
<br><font size=2><tt>performance before I submit for review.<br>
</tt></font>
<br><font size=2><tt>We really want this patch to be in RC2 release. I
will change the name back </tt></font>
<br><font size=2><tt>if this would prevent the patch from being into upper
stream.</tt></font>
<br>
<br><font size=2 face="sans-serif">Thanks<br>
Shirley Ma<br>
IBM Linux Technology Center<br>
15300 SW Koll Parkway<br>
Beaverton, OR 97006-6063<br>
Phone(Fax): (503) 578-7638</font>