lkml.org 
[lkml]   [2005]   [Dec]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Subject[PATCH 00/13] [RFC] IB: PathScale InfiniPath driver
Date
From
Here is an initial submission from PathScale of a driver for
InfiniPath InfiniBand HCAs. The driver is fairly big -- some single
files are more than the 100 KB limit for lkml posts -- so I've split
it up into a patch series so it can be reviewed inline. The split-up
doesn't make sense functionally but I want to make review as easy as
possible; any final import will merge the driver as a single git
patch.

I've also put the current splitup patchset into my git tree at

git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git

in the ipath branch.

There are some things I noticed that could maybe be cleaned up, like
having sysctls that set values also settable through module parameters
under /sys/module, code inside #ifndef __KERNEL__ so include files can
be shared with other PathScale code, code in ipath_i2c.c that might be
simplified by using drivers/i2c, etc. I'd like to try to get a sense
of whether I'm being too picky or whether PathScale really does need
to fix these up before the driver is merged.

Basically I'm trying to feel my way as a maintainer so I can find the
right balance between wanting kernel code to be absolutely perfect and
not wanting to put arbitrary hurdles in front of a vendor who has done
a lot of work on contributing an open driver for their hardware.

I am especially interested in feedback about the mergability of this
driver from the broader kernel community, although of course feedback
from the InfiniBand/RDMA community on IB-specific aspects of the
driver is very much appreciated as well.

Thanks,
Roland
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-12-17 00:52    [W:0.131 / U:0.520 seconds]
©2003-2014 Jasper Spaans. Advertise on this site