> Is this a bug? I think it is...
>
> diff --git a/drivers/infiniband/core/ucma.c b/drivers/infiniband/core/ucma.c
> index 2709ff5..fb24f05 100644
> --- a/drivers/infiniband/core/ucma.c
> +++ b/drivers/infiniband/core/ucma.c
> @@ -806,8 +806,13 @@ static ssize_t ucma_accept(struct ucma_file *file,
> const ch
> if (!ret)
> ctx->uid = cmd.uid;
> mutex_unlock(&file->mut);
> - } else
> + } else {
> + mutex_lock(&file->mut);
> ret = rdma_accept(ctx->cm_id, NULL);
> + if (!ret)
> + ctx->uid = cmd.uid;
> + mutex_unlock(&file->mut);
> + }
The ctx->uid has already been set in the else case when the user explicitly
created the cm_id. We only need to set it when the ctx has been created in the
kernel for the user in response to a connection request (the 'if' part to the
above else).
- Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
