Re: [PATCH 0 of 20] [RFC] ipath - PathScale InfiniPath driver

From: Bryan O'Sullivan
Date: Mon Jan 02 2006 - 17:39:10 EST


On Mon, 2006-01-02 at 13:35 -0700, Eric W. Biederman wrote:

> I haven't looked closely enough at the state of the openib tree but
> you should not need an additional interface to send/receive standard
> IB subnet management packets. That is something that should be provided
> the same way by all infiniband drivers.

We provide the standard OpenIB mechanisms for doing that, of course.
However, our driver is layered. The OpenIB layer uses facilities
provided by the main driver (via ipath_layer.c). The main driver can
stand alone, without the OpenIB code compiled into the kernel or
available as a module at all. In that case, a userland subnet
management agent must still be able to send and receive management
packets.

> Given Linus's comments and looking at where you are getting stuck I
> would recommend you split out support for the nonstandard ipath
> protocol from the rest of the driver.

While we can split the main driver source file up along those lines, we
are not planning to make the ipath protocol optional. We are planning
to submit another non-OpenIB network driver that depends on the ipath
protocol support.

<b

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