On Mon, Nov 20, 2017 at 07:11:09PM +0000, Dhanalakota, Venkata S wrote: > Hi Hal, > > The patch intends to only check if smlid is assigned or not. It does > not intend to check if smlid is a valid lid or not. Validity of lid > is taken care by the individual drivers, during create_ah(). I think that is Hal's point - there are multiple ways to say 'not assigned' for SMLID, including == 0 and == 0xFFFF It is best to use !valid as a test for an unassigned SM lid. Jason -- 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