[openib-general] roland-uverbs: CQ consumer index

[Tziporet Koren]

Hi,

We thought about it a lot and decided to have two different drivers but with
as much as possible code sharing.
Since realy most code is the same and we did not wanted to duplicate logic
and bug fixes.

The way we did the code sharing is not complaint with Linux but I think one
can come up with another method that will enable to have different code only
to the objects that are required to be different from HW interface.

Tziporet 

-----Original Message-----
From: Roland Dreier [mailto:roland at topspin.com]
Sent: Monday, January 31, 2005 9:33 PM
To: Libor Michalek
Cc: openib-general at openib.org
Subject: Re: [openib-general] roland-uverbs: CQ consumer index


    Libor>   Without knowing how much code difference there will be
    Libor> between Tavor and Arbel native mode, I'm wondering if it
    Libor> makes sense at some point to have seperate hardware drivers
    Libor> for the two parts? Just curious what the your thinking is
    Libor> on the value of maintaining the same code base for the two.

I've gone back and forth between whether it's better to split mthca
into two drivers or keep it as one driver that supports both Tavor and
Arbel modes.  A lot of the code is common but there are also a lot of
differences.  Right now I think keeping one driver is a little better
because it's fairly easy to keep the Arbel-specific code in mthca, and
I keep coming up with places where it would be very awkward to turn
the common code into a library.  However I've gone back and forth in
my mind a bunch of times so we may end up with two drivers before it's
all over.

 - R.
_______________________________________________
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/general/attachments/20050201/04d782a2/attachment.html>