
<html><body>

<p><tt>general-bounces@lists.openfabrics.org wrote on 02/03/2008 09:07:29 AM:<br>

<br>

> Hi Shirley,<br>

> <br>

> I have reviewed the patches against Roland's tree and have the following<br>

> comments:<br>

</tt><br>

<tt>Appreciate you quick review.</tt><br>

<tt><br>

> 1. I see that there are a few if statements added on the fast pass and I<br>

> am concerned they might hurt performance of slow UDP messages.<br>

> Unfortunately I have not been able to test with an SM defining the<br>

> broadcast group to 4K MTU (currently opensm uses 2K).<br>

What kind of parameters you prefer here for me to test this patch? I can test it right away when you send me your recommendations.</tt><br>

<tt> <br>

> 2. The usage of ipoib_ud_skb_put_frags() seems to be redundant and will<br>

> only hurt performance since you would never reuse anything from the old<br>

> SKB. This is because the headlen is 40 bytes for GRH and the rest of the<br>

> data is in the first (and only) fragment.<br>

The header is 44 bytes, the IP payload data is in the first fragment.</tt><br>

<br>

<tt>> 3. I think it would be better to allocate room for real data in the head<br>

> of the SKB since the tcp/ip stack seems to have less overhead if the<br>

> headers are on the linear data.<br>

> 4. I would consider using a pre-allocated buffer for the GRH of all<br>

> received data (not as part of the SKB).</tt><br>

<br>

<tt>Comments 2,3,4 can be combined of one if we use a pre-allocated buffer for GRH+IPoIB-head for all IP payload data, right? This is a performance enhancement if any. I think this could be done after this patch being checked in. And I will fix it before RC4 out. Do you agree?</tt><br>

<br>

<tt>Thanks</tt><br>

<tt>Shirley</tt></body></html>