On 01/03/18 06:40, Richard Cochran wrote:
At first glance this series looks good. I expect to merge the first
half or so, because we really do want the tmv_t operations to be used
consistently.
Regarding changing the implementation, I think we'll want to hold off
on that until kernel support for high resolution time stamps becomes
available. Merging this bit right away makes the computations slower
and benefits very few users!
Understood. It may also make sense to merge some of the later patches:
for example, "clock: Pass floating point values to stats_add_value()"
simply changes the point at which an already-existing cast to double
occurs, and I could easily create an inline int64_t version of "tmv: Add
converters for hardware timestamps".
Would you like me to respin a v2 series which contains only the patches
that will not impose a speed penalty?
Many thanks,
Michael
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
