Re: [PATCH v4 00/50] Add OPA gen1 driver

[Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 2454 bytes --]

On 07/31/2015 03:34 AM, Christoph Hellwig wrote:
> On Fri, Jul 31, 2015 at 02:05:06AM +0300, Or Gerlitz wrote:
>> So... enough is enough, please put it in a kernel module residing in
>> the IB core and use it in this driver, to begin with. The fact that
>> ipath is going to go, makes the cope duplication "only" 2X vs the 3X,
>> but it's still 2X
> 
> Agreed.  Any if anyone tries to submit software RoCE again it'll be
> another duplication.

Or, I haven't looked at the soft-roce driver (ever).  Is it going to
need this library as well?  If it is, then as you have rung the bell for
getting this library written, I will expect Mellanox to work with Intel
to make sure that this library is suitable for not just their hardware
drivers but also the soft-roce driver you guys are working on.  I might
even suggest that it's time to work up the soft-roce submission sooner
rather than later in light of what I've done with the hfi1 driver.

On that note, Mike, I've taken the hfi1 driver into my tree, but I moved
it to the staging tree.  There, I created a TODO file with these items
from the review:

July, 2015

- Remove unneeded file entries in sysfs
- Remove software processing of IB protocol and place in library for use
  by qib, ipath (if still present), hfi1, and eventually soft-roce
- Clean up comments in code around CNP opcode

You will note that I have not placed "rework the /dev/ipath file
write/writev semantics" on that list.  I have yet to hear a convincing
technical reason that this must be done so I won't require it (you are
free to do so, however, if you just want to quite down the noise).  That
might change on the basis of any further emails in the thread.

So my plan here is that we will move the driver out of staging once
we've gotten these items taken care of.  However, I will note this
specific item: if the soft-roce driver needs the library too, and it is
not yet in a state to be submitted and even not in a state where its
needs can be articulated clearly so that the library can be written with
soft-roce's needs in mind as well as the Intel driver's needs, then I'll
let the hfi1 driver out of staging before that task is complete and
we'll just get to the library when we have all of the stake holders in
place and can actually work on all of them at once.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]