The ibv_context structure will be getting additional ops,
to be added at the end of the structure (and not as part of
the existing ibv_context_ops structure).

Reason: ibv_context_ops is declared directly as a member of ibv_context,
and not as a pointer.  Binaries compiled with previous libibverbs versions
will not be backwards compatible if we add new operations to ibv_context_ops,
since fields following the ops structure will move.

To enable adding new operations at the end of the existing ibv_context struct,
all driver libraries MUST zero their context structure at allocation time, so
that new ops will be NULL by default.

Signed-off-by: Jack Morgenstein <[EMAIL PROTECTED]>

diff --git a/src/mlx4.c b/src/mlx4.c
index c845fc1..93c1ce8 100644
--- a/src/mlx4.c
+++ b/src/mlx4.c
@@ -115,6 +115,7 @@ static struct ibv_context *mlx4_alloc_context(struct 
ibv_device *ibdev, int cmd_
        if (!context)
                return NULL;
 
+       memset(context, 0, sizeof *context);
        context->ibv_ctx.cmd_fd = cmd_fd;
 
        if (ibv_cmd_get_context(&context->ibv_ctx, &cmd, sizeof cmd,
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to