[openib-general] Re: [PATCH] CMA and iWARP

David Ford dford at netapp.com
Thu Jan 19 11:07:03 PST 2006


At 02:02 PM 1/19/2006, Steve Wise wrote:
>On Thu, 2006-01-19 at 10:20 -0800, Sean Hefty wrote:
> > Tom Tucker wrote:
> > > Enclosed is a combined include file and core patch for iWARP support 
> in CMA. This
> > > patch includes changes per your last review.
> >
> > It was brought up that there's still not an iWarp driver in the OpenIB 
> trunk.
> > Is one ready to be merged?  Thinking about this more, I'm not sure that 
> we want
> > to merge these changes into the trunk without one.  (I can see 
> arguments both
> > ways.)  Comments?
>
>The Ammasso rnic code could be merged in at this point.  But the company
>Ammasso is no more, so I question whether we want it in the main trunk?

I'd like to see it. A running implementation of OpenIB on iWARP is a very 
useful thing to have right now, assuming a reasonable supply of Ammasso cards.

-- Dave


> > Looking at the patch, it seems that the biggest issue that you would 
> have with
> > merging changes is how the cm_id is stored.  I can at least fold that 
> change
> > into the CMA to help with maintenance if an iWarp driver isn't ready yet.
> >
>
>Getting the core changes in now will help avoid having to keep merging
>trunk code back into the iwarp branch.  It will also expose the iwarp
>changes to a larger audience for review and improvement.
>
>My 2 cents.
>
>
>Steve.
>
>
>
>_______________________________________________
>openib-general mailing list
>openib-general at openib.org
>http://openib.org/mailman/listinfo/openib-general
>
>To unsubscribe, please visit 
>http://openib.org/mailman/listinfo/openib-general



More information about the general mailing list