From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sasha Khapyorsky Subject: Re: [PATCH] replace (long*)(long) casting with transportable data type (uintptr_t) Date: Tue, 30 Nov 2010 20:57:54 +0200 Message-ID: <20101130185754.GY11898@me> References: <7C00F472050C412BABCB4F2A59CFBD63@amr.corp.intel.com> <20101130175555.GX11898@me> 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: "Hefty, Sean" Cc: "Smith, Stan" , Linux RDMA List-Id: linux-rdma@vger.kernel.org On 10:22 Tue 30 Nov , Hefty, Sean wrote: > > Wouldn't it be better to remove those additional castings at all? > > Like below? > > > .. > > p_query_req_copy = (osmv_query_req_t *) p_req_madw->context.ni_context.node_guid; > > I think the Windows compiler will complain about data loss on a 32-bit system. p_req_madw->context.ni_context.node_guid has 64 bits type. Sasha -- 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