From mboxrd@z Thu Jan 1 00:00:00 1970 From: Evgeniy Polyakov Subject: [ofa-general] Re: [PATCH 04/21] RDS: Transport code Date: Wed, 28 Jan 2009 00:56:58 +0300 Message-ID: <20090127215657.GD12431@ioremap.net> References: <1233022678-9259-1-git-send-email-andy.grover@oracle.com> <1233022678-9259-5-git-send-email-andy.grover@oracle.com> <20090127131808.GF2646@ioremap.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, rdreier@cisco.com, rds-devel@oss.oracle.com, general@lists.openfabrics.org To: Andrew Grover Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: general-bounces@lists.openfabrics.org Errors-To: general-bounces@lists.openfabrics.org List-Id: netdev.vger.kernel.org On Tue, Jan 27, 2009 at 11:36:37AM -0800, Andrew Grover (andy.grover@gmail.com) wrote: > On Tue, Jan 27, 2009 at 5:18 AM, Evgeniy Polyakov wrote: > > On Mon, Jan 26, 2009 at 06:17:41PM -0800, Andy Grover (andy.grover@oracle.com) wrote: > >> +static LIST_HEAD(transports); > >> +static DECLARE_RWSEM(trans_sem); > >> + > > > > RDS_ prefix? > > Even needed for statics? It confuses tags and the like otherwise, and looks more consistent with the rest of the code. Likely it is not a must, but just better look. -- Evgeniy Polyakov