Here's a series of patches that add an InfiniBand adapter driver for IBM eHCA hardware. Please look it over with an eye towards issues that need to be addressed before merging this upstream. This patch series is somewhat unusual in that I am not the original author of this driver -- I am just sending it for review for the authors, who are apparently not able to post patches themselves due to internal issues at IBM. However they are cc'ed and will respond to comments in this thread. In fact I have some issues with the code myself that need to be addressed before this driver is mergeable. I've included most of them in the individual patches, although I have some general comments too. However I would like to get some early feedback for the ehca authors from the wider community. In particular I think its important to run this past the ppc64 experts, since I'm not sure what the standards for this sort of pSeries driver are. Anyway, my general comments: - The #ifs that test EHCA_USERDRIVER and __KERNEL__ should be killed. We know that this is kernel code, so there's no reason to include userspace compatibility junk. - Many of the comments look like they are for some automatic documentation system that is not quite kerneldoc. They should be fixed to be real kerneldoc comments. - In general there is a huge amount of code in large inline functions in .h files. Things should be reorganized to cut this down to a sane amount. Thanks, Roland - 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/
- Follow-Ups:
- Re: [PATCH 00/22] [RFC] IBM eHCA InfiniBand adapter driver
- From: Christoph Raisch <[email protected]>
- [PATCH 12/22] ehca low-level verbs
- From: Roland Dreier <[email protected]>
- [PATCH 05/22] HW register abstractions
- From: Roland Dreier <[email protected]>
- [PATCH 07/22] Hypercall definitions
- From: Roland Dreier <[email protected]>
- [PATCH 21/22] ehca main file
- From: Roland Dreier <[email protected]>
- [PATCH 03/22] pHype specific stuff
- From: Roland Dreier <[email protected]>
- [PATCH 14/22] ehca completion queue handling
- From: Roland Dreier <[email protected]>
- [PATCH 18/22] ehca address vectors, multicast groups, protection domains
- From: Roland Dreier <[email protected]>
- [PATCH 22/22] ehca Makefile/Kconfig changes
- From: Roland Dreier <[email protected]>
- [PATCH 16/22] ehca post send/receive and poll CQ
- From: Roland Dreier <[email protected]>
- [PATCH 04/22] OF adapter probing
- From: Roland Dreier <[email protected]>
- [PATCH 19/22] ehca memory regions
- From: Roland Dreier <[email protected]>
- [PATCH 17/22] Special QP functions
- From: Roland Dreier <[email protected]>
- [PATCH 15/22] ehca queue pair handling
- From: Roland Dreier <[email protected]>
- [PATCH 20/22] ehca userspace verbs
- From: Roland Dreier <[email protected]>
- [PATCH 11/22] ehca event queues
- From: Roland Dreier <[email protected]>
- [PATCH 13/22] HCA query functions
- From: Roland Dreier <[email protected]>
- [PATCH 06/22] Queue handling
- From: Roland Dreier <[email protected]>
- [PATCH 08/22] Generic ehca headers
- From: Roland Dreier <[email protected]>
- [PATCH 02/22] Firmware interface code for IB device.
- From: Roland Dreier <[email protected]>
- [PATCH 09/22] ehca classes
- From: Roland Dreier <[email protected]>
- [PATCH 01/22] Add powerpc-specific clear_cacheline(), which just compiles to "dcbz".
- From: Roland Dreier <[email protected]>
- Re: [PATCH 00/22] [RFC] IBM eHCA InfiniBand adapter driver
- Prev by Date: [PATCH 02/22] Firmware interface code for IB device.
- Next by Date: [PATCH 08/22] Generic ehca headers
- Previous by thread: [PATCH] mm: Implement swap prefetching v27
- Next by thread: [PATCH 01/22] Add powerpc-specific clear_cacheline(), which just compiles to "dcbz".
- Index(es):