Re: [PATCH 2/2] usbip: use monotonic timestamps
On 11/07/2017 04:22 AM, Krzysztof Opasiak wrote: > > > On 11/07/2017 11:39 AM, Arnd Bergmann wrote: >> This gets rid of the deprecated do_gettimeofday() function in usbip. >> The comment above vgadget_get_frame() mentions that it suffers >> from issues with the time jumps due to suspend and settimeofday, >> so I'm changing it to use ktime_get_ts64() to use monotonic times >> that don't have this problem. >> >> I couldn't tell whether we should use CLOCK_MONOTONIC or >> CLOCK_MONOTONIC_RAW here, the difference being the exact rate >> when correcting for NTP. I picked monotonic time since it doesn't >> change the speed to the existing code and should be better >> synchronized with other machines we talk to. >> >> Signed-off-by: Arnd Bergmann> > Looks good to me: > Reviewed-by: Krzysztof Opasiak > Acked-by: Shuah Khan thanks, -- Shuah -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/2] usbip: use monotonic timestamps
On 11/07/2017 11:39 AM, Arnd Bergmann wrote: This gets rid of the deprecated do_gettimeofday() function in usbip. The comment above vgadget_get_frame() mentions that it suffers from issues with the time jumps due to suspend and settimeofday, so I'm changing it to use ktime_get_ts64() to use monotonic times that don't have this problem. I couldn't tell whether we should use CLOCK_MONOTONIC or CLOCK_MONOTONIC_RAW here, the difference being the exact rate when correcting for NTP. I picked monotonic time since it doesn't change the speed to the existing code and should be better synchronized with other machines we talk to. Signed-off-by: Arnd BergmannLooks good to me: Reviewed-by: Krzysztof Opasiak -- Krzysztof Opasiak Samsung R Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html