Re: [ofa-general] Re: [PATCH 0/3] ib/ipoib: Enable IPoIB-UD 4K MTU support

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Shirley Ma
Date: Saturday, February 2, 2008 - 9:16 am

--0__=08BBF970DFCB57778f9e8a93df938690918c08BBF970DFCB5777
Content-type: text/plain; charset=US-ASCII
Content-transfer-encoding: quoted-printable






Hello Roland,

l

My patch has been passed the stress test for both PPC and Intel
architechture against OFED-1.3-RC2 bit for a couple days. And I didn't =
see
performance imapct for 2K mtu. But I rethink about your suggestion here=

yesterday night. I can modify my patch to meet your thoughts here by
keeping current implementation of 2K mtu and using if condition check f=
or
the new code. I will submit a new version of patchset today for review.=

Since I only have two days for my patch to be integred into OFED-1.3-RC=
3
for Distros to pick up. I would like to see your ack here for this appr=
oach
as soon as possible. I will compare two different implementation's
performance.

Thanks for your inputs. Appreciate your prompt response.

Thanks
Shirley=

--0__=08BBF970DFCB57778f9e8a93df938690918c08BBF970DFCB5777
Content-type: text/html; charset=US-ASCII
Content-Disposition: inline
Content-transfer-encoding: quoted-printable

<html><body>
<p><tt>Hello Roland,</tt><br>
<br>
<tt>&gt; So how about the following? &nbsp;When using a UD MTU of 4096 =
with a page<br>
&gt; size of 4096, allocate an skb of size 44 for the GRH and ethertype=
,<br>
&gt; and then allocate a single page for the fragment list. &nbsp;This =
means<br>
&gt; that the IP packet will start nicely 16-byte aligned for free, and=
 all<br>
&gt; the bookkeeping is very simple.<br>
</tt><br>
<tt>My patch has been passed the stress test for both PPC and Intel arc=
hitechture against OFED-1.3-RC2 bit for a couple days. And I didn't see=
 performance imapct for 2K mtu. But I rethink about your suggestion her=
e yesterday night. I can modify my patch to meet your thoughts here by =
keeping current implementation of 2K mtu and using if condition check f=
or the new code. I will submit a new version of patchset today for revi=
ew. Since I only have two days for my patch to be integred into OFED-1.=
3-RC3 for Distros to pick up. I would like to see your ack here for thi=
s approach as soon as possible. I will compare two different implementa=
tion's performance.</tt><br>
<br>
<tt>Thanks for your inputs. Appreciate your prompt response.</tt><br>
<br>
<tt>Thanks</tt><br>
<tt>Shirley</tt></body></html>=

--0__=08BBF970DFCB57778f9e8a93df938690918c08BBF970DFCB5777--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [ofa-general] Re: [PATCH 0/3] ib/ipoib: Enable IPoIB-U ..., Shirley Ma, (Sat Feb 2, 9:16 am)