On Tue, 19 Jul 2016, Matan Barak wrote: > diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h > index 432bed5..14bfe3b 100644 > --- a/include/rdma/ib_verbs.h > +++ b/include/rdma/ib_verbs.h > @@ -1706,6 +1706,10 @@ struct ib_device { > > struct iw_cm_verbs *iwcm; > > + struct idr idr; > + /* Global lock in use to safely release device IDR */ > + spinlock_t idr_lock; Global? In what sense? This is a lock for the idrs for the particular device right? The comment is a bit confusing. Maybe better remove it? -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html