Friday, April 13, 2018 7:16 AM > I have two major issues with this series: > 1. You collect statistics but never report it.
This set of patch was just for collecting and storage of the data. My intention was, in a later set of patches, add possibility to collect this data. As it is now, there are no TLV mentioned in the draft for this purpose. I mentioned that in my review comments of the draft. But the idea was to add the collection and storage of the data. If feature is not enabled, no recordlists will be filled or code executed. Just the counters will be collected. But that's just to clear when feature is enabled. Other comments was TLV to enable/disable in runtime. > 2. The implementation is overly complex. I'll explain more in a reply > to patch #4. Thanks, I will look at your comments on it /Anders ------------------------------------------------------------------------------ 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