From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: How far back does rdma-core X supports the kernel Y? Date: Thu, 2 Nov 2017 15:35:40 -0600 Message-ID: <20171102213540.GX18874@ziepe.ca> References: <20171101163915.GB1030@ziepe.ca> <20171101173324.GF1030@ziepe.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Amrani, Ram" Cc: "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "Elior, Ariel" , "Kalderon, Michal" , Matan Barak , Bodong Wang List-Id: linux-rdma@vger.kernel.org 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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html