[ofa-general] Re: [ewg] OFED Sep 10 meeting summary on OFED 1.3 development status

[Tziporet Koren]

Or Gerlitz wrote:
>
> Based on the fact that OFED 1.3 is now based on 2.6.23-rc5, it simply 
> does not make --any-- sense to have sixty "fix" patches from day one ( 
> full list below). Zooming to to IPoIB (twelve patches!) I see plenty 
> of non cooked materials:
>
Same was with bonding when we started integration in OFED 1.2 but 
eventually we succeeded to make it work
> The discussion/thread that followed the "IB/ipoib: S/G and HW checksum 
> support" patch posted by Michael is not over yet. There was a 
> direction offered by Michael which is pending to Roland's feedback.
We will enhance the patches once we get Roland's feedback
>
> As for the "ipoib - add LSO support" patch(es) posted by Eli, I did 
> not see any response of Roland nor of anyone else (except for me 
> asking some questions...). Eli/Michael - where does this stand? are 
> you planning to push this for 2.6.24?
Yes we are
>
> Also I understand that the patch set supports what you call "interrupt 
> moderation", does this goes well with NAPI? if yes, how and why put in 
> OFED something you never sent to review on the general list nor 
> attempted to push into the kernel?
>
>
We have sent this for review
>
>>     RDS - RDMA API - done     SDP: Keepalive - done; Asynch IO - 
>> done, Zero Copy - 80% 
>
> have these patches ever sent to review on the general or ewg lists?
RDS - patches were reviewed by the RDS developers in the rds list.
SDP - Jim sent patches for keepalive. Zcopy will be sent when ready
>
>
> I don't think we can start with sixty patches, sorry for not bringing 
> this input before yesterday. I would be happy to hear what others here 
> think.
There are101 patches in OFED 1.2, and I don't think it harmed anyone.

Note that most of the patches are code that will make it to kernel 2.6.24.
So the only way to avoid many patches is to change OFED 1.3 base kernel 
to 2.6.24
I suggest we will wait and see if we want to do it once 2.6.24 is out.
 

Tziporet