> 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html