On Thu, Nov 02, 2017 at 07:01:58AM +0000, Amrani, Ram wrote: > > We have tended to use reserved to mean dont use. Someone should > > inspect the git history before trying to use a reserved value.. > > > > When reserved are used in structures, usually for alignment, they are > freely taken when a new field needs adding. Even then you should check the git history to figure out why the field is reserved. Padding ones are OK to steal, but we also have deprecated struct members. :\ > Still, I think it is a better practice to give a more meaningful name > and/or comment to differentiate. E.g.: > > IB_QP_XXX_DONT_USE /* Kept for backward compatibility only */ It would be good to do this, yes. 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
