From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH for-next 09/10] IB/mlx4: Add timestamp_mask and hca_core_clock to query_device Date: Tue, 26 May 2015 12:53:15 -0600 Message-ID: <20150526185315.GF11800@obsidianresearch.com> References: <20150519191553.GP18675@obsidianresearch.com> <20150520002915.GD16941@obsidianresearch.com> <555C9D00.2090609@mellanox.com> <1432134677.5304.23.camel@opteya.com> <20150526160624.GA11800@obsidianresearch.com> 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: Or Gerlitz Cc: Doug Ledford , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Amir Vadai , Tal Alon , Matan Barak , Yann Droneaud List-Id: linux-rdma@vger.kernel.org On Tue, May 26, 2015 at 09:33:18PM +0300, Or Gerlitz wrote: > On Tue, May 26, 2015 at 7:06 PM, Jason Gunthorpe > wrote: > > On Tue, May 26, 2015 at 11:10:45AM +0300, Or Gerlitz wrote: > > > >> Jason, ping, it's fair to require that if you made a review argument against > >> the design done here and we've responded about a week ago, saying why > >> this design is valid (e.g goes along the 10y old IB stack udata mechanism and > >> such) -- you would comment on the response and not leave it in the air. > > > > Was it not clear? Yann and I asked to see the user space side before > > reviewing this series further. > > Jason, you (U2 BTW) play really, really hard - refusing to say **one** > word on your approach towards the built-in udata mechanism for uverbs > which I asked you to comment on. And I asked to see the user space side and you have angrily refused every time. So I guess we are both playing hard. FWIW, your comments on udata seemed compelling, but I want to see the whole solution before saying I'm OK with it. 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