From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH for-next V2 0/9] Add completion timestamping support Date: Mon, 1 Jun 2015 10:43:22 -0600 Message-ID: <20150601164322.GA14391@obsidianresearch.com> References: <1433074457-26437-1-git-send-email-ogerlitz@mellanox.com> <1433098827.114391.179.camel@redhat.com> <1433157904.114391.188.camel@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1433157904.114391.188.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Ledford Cc: Matan Barak , Or Gerlitz , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Amir Vadai , Tal Alon List-Id: linux-rdma@vger.kernel.org On Mon, Jun 01, 2015 at 07:25:04AM -0400, Doug Ledford wrote: > attempted abstraction of ibverbs. Passing in the wc struct allows the > driver to internally allocate a wc struct with extra private elements > and pass that back to the user, when the user passes it back to > ibv_get_timestamp the elements are there in the private portion of the > struct. wc structures are allocated by the caller, there is no option for the driver to create private elements. AFAIK, Christoph's use case is essentially the only meaningful use case for this feature, generalizing too much may destroy the performance that is valuable here. 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