On Thursday 16 November 2006 01:11, divy wrote:
> This patch adds support for the latest Chelsio adapter, T3.
>
> Since some files are bigger than the 40kB advertized in the submit
> guidelines, a monolithic patch against 2.6.19-rc5 is posted at the
> following URL: http://service.chelsio.com/kernel.org/cxgb3.patch.bz2
>
> Please advise on any other form you would like to see the code.
You should still attempt to post it as inline email for better review.
The hard limit on mails is 100kb right now, IIRC, so it should be
possible to split it up into smaller chunks for review.
> We wish this patch to be considered for inclusion in 2.6.20. This driver
> will be required by the Chelsio T3 RDMA driver which will be posted for
> review asap.
Two things I noticed on a very brief review:
- The driver is _huge_. Even if your hardware is terribly complicated,
the fact that you need 600kb to drive it is a hint that you do
something wrong. In particular, the header files contain hundreds
of macros that are completely unused. You should get rid of them.
- You define a driver specific multiplexing ioctl method. This is
something you really shouldn't do in a network driver, since the
ioctl number will conflict with other drivers. In order to get
the rest of your code merged, I'd suggest you split out the
private ioctl handling into a separate patch that you don't submit
for now, and then we can think of how to replace the interfaces
with something better, e.g. new driver-independent calls like ioctl
or ethtool or sysfs, or get rid of them if they turn out to be
not required.
Arnd <><
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
