<br><font size=2><tt>Roland Dreier <rdreier@cisco.com> wrote on 04/19/2006
04:36:50 PM:<br>
<br>
>     Shirley> Since I haven't found any kernel use 128
bit address, I<br>
>     Shirley> use wr_id to save skb address, DMA mapping
and other<br>
>     Shirley> stuffs are saved in skb->cb, which is
the private data<br>
>     Shirley> for each protocal layer in skb. Same for
rx_ring, so<br>
>     Shirley> rx_buff and tx_buff is not necessary to
be used.<br>
> <br>
> That sounds like a good plan.  Although I guess we want to keep
around<br>
> some sort of rx ring if only to free receive buffers when bringing
an<br>
> interface down.  But that could become a linked list with the<br>
> list_head in skb->cb I guess.<br>
</tt></font>
<br><font size=2><tt>Yes, a list is needed for gracefully shutdown.</tt></font>
<br>
<br><font size=2><tt>> Does all this matter for performance?<br>
> <br>
>  - R.</tt></font>
<br>
<br><font size=2><tt>It helps the performance about 10% for the touch netperf/netserver
test,</tt></font>
<br><font size=2><tt>then hit driver errors. I notice that the send is
faster than before. Let me </tt></font>
<br><font size=2><tt>send you the patch tomorrow, maybe you have the hints
to identify the </tt></font>
<br><font size=2><tt>problem.</tt></font>
<br>
<br><font size=2><tt>Thanks<br>
</tt></font><font size=2 face="sans-serif">Shirley Ma<br>
IBM Linux Technology Center<br>
15300 SW Koll Parkway<br>
Beaverton, OR 97006-6063<br>
Phone(Fax): (503) 578-7638</font>