On 07/12/2015 22:43, Dennis Dalessandro wrote:
> +struct rvt_dev_info {
> + struct ib_device ibdev;
> + int (*port_callback)(struct ib_device *, u8, struct kobject *);
> +
> + /*
> + * TODO:
> + * need to reflect module parameters that may vary by dev
> + */
> +};
On Thu, Dec 10, 2015 at 01:49:31PM +0200, Haggai Eran wrote:
On 07/12/2015 22:43, Dennis Dalessandro wrote:
+struct rvt_dev_info {
+ struct ib_device ibdev;
+ int (*port_callback)(struct ib_device *, u8, struct kobject *);
+
+ /*
+* TODO:
+* need to
On Tue, Dec 08, 2015 at 07:55:45AM +0200, Leon Romanovsky wrote:
On Mon, Dec 07, 2015 at 03:43:02PM -0500, Dennis Dalessandro wrote:
+#include
+#include
+#include "vt.h"
+
+#define RDMAVT_DRIVER_VERSION "0.1"
Do we really need driver version?
Based on these patches thus far. Probably not.
> >> +#define RDMAVT_DRIVER_VERSION "0.1"
> >Do we really need driver version?
>
> Based on these patches thus far. Probably not. We do still have a bit more
> development to go here, I added it more as a just in case but am not
> steadfast in keeping it.
>
> What's the general rule here? Should
On Mon, Dec 07, 2015 at 03:43:02PM -0500, Dennis Dalessandro wrote:
> +#include
> +#include
> +#include "vt.h"
> +
> +#define RDMAVT_DRIVER_VERSION "0.1"
Do we really need driver version?
> +
> +MODULE_LICENSE("Dual BSD/GPL");
> +MODULE_DESCRIPTION("RDMA Verbs Transport Library");
>
This patch introduces the basics for a new module called rdma_vt. This new
driver is a software implementation of the InfiniBand verbs and aims to
replace the multiple implementations that exist and duplicate each others'
code.
While the call to actually register the device with the IB core
> @@ -0,0 +1,89 @@
> +/*
> + *
> + * This file is provided under a dual BSD/GPLv2 license. When using or
> + * redistributing this file, you may do so under either license.
> + *
> + * GPL LICENSE SUMMARY
> + *
> + * Copyright(c) 2015 Intel Corporation.
I'm guessing that the GPL license text