Re: [PATCH for-next 09/10] IB/mlx4: Add timestamp_mask and hca_core_clock to query_device

Or Gerlitz
2015-05-19T19:11:43Z

On Tue, May 19, 2015 at 10:00 PM, Jason Gunthorpe
<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> wrote:
> On Tue, May 19, 2015 at 12:58:01PM -0600, Jason Gunthorpe wrote:
>> On Sun, May 17, 2015 at 04:36:25PM +0300, Or Gerlitz wrote:
>> > From: Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>> >
>> > mlx4 needs to report the number of supported timestamp
>> > bits (mask) and the hca_core_clock frequency.
>>
>> This is critical information to parse the timestamp, why is it hidden
>> in vendor specific land?
>
> Sorry, direct this toward the page offset in the next patch.

We see this as a piece of detail related to the specific vendor
driver. I don't see the point of overloading the user space verbs
library or app with an API to get the offset of the clock within the
mmap page and such, do you?

Or.