From mboxrd@z Thu Jan 1 00:00:00 1970 From: Haggai Eran Subject: Re: [PATCH v2 09/13] IB/cma: Add net_dev and private data checks to RDMA CM Date: Tue, 28 Jul 2015 11:12:22 +0300 Message-ID: <55B73966.8030103@mellanox.com> References: <1437924832-18327-1-git-send-email-haggaie@mellanox.com> <1437924832-18327-10-git-send-email-haggaie@mellanox.com> <1828884A29C6694DAF28B7E6B8A82373A901F99B@ORSMSX109.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1828884A29C6694DAF28B7E6B8A82373A901F99B-P5GAC/sN6hkd3b2yrw5b5LfspsVTdybXVpNB7YpNyf8@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Hefty, Sean" , Doug Ledford Cc: Liran Liss , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Jason Gunthorpe List-Id: linux-rdma@vger.kernel.org On 28/07/2015 02:07, Hefty, Sean wrote: >> @@ -1040,6 +1040,181 @@ static int cma_save_net_info(struct sockaddr >> *src_addr, >> return cma_save_ip_info(src_addr, dst_addr, ib_event, service_id); >> } >> >> +struct cma_req_info { >> + struct ib_device *device; >> + int port; >> + const union ib_gid *local_gid; > > The use of a pointer to some gid inside some other structure looks questionable. There's no reference count held on another structure. We would be better off copying the GID or setting an index into the device GID table. Okay, I'll copy the GID to the struct. > >> + __be64 service_id; >> + u16 pkey; >> +}; > > Please relocate to the top of the source file to group it with other structure definitions. Sure. Thanks, Haggai -- 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