On Mon, May 2, 2016 at 6:11 PM, Christoph Lameter <cl@xxxxxxxxx> wrote: > On Mon, 2 May 2016, Erez Shitrit wrote: > >> + if (test_bit(IPOIB_MCAST_FLAG_SENDONLY, &mcast->flags) && >> + priv->sm_fullmember_sendonly_support) >> + /* SM supports sendonly-fullmember, otherwise fallback to full-member */ >> + rec.join_state = 8; > > Why 8? and could we maybe use less cryptic values, e.g based on defined/enums or what ever needed to describe that clearly. -- 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