Milan Zamazal has posted comments on this change. Change subject: doc: Typo fixes in utils.monotonic_time docstring ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/59295/1/lib/vdsm/utils.py File lib/vdsm/utils.py: Line 817: This function is useful if the client just Line 818: needs to use the difference between two given time points. Line 819: Line 820: With respect to time.time(): Line 821: * The resolution of this function is lower. On Linux, > This is not correct, the resolution of time.time() is nanoseconds, while th Oops, you're right. I got confused, thanks for correcting me. Reverted. Line 822: the resolution is 1/_SC_CLK_TCK, which in turn depends on Line 823: the value of HZ configured in the kernel. A commonly Line 824: found resolution is 10 (ten) ms. Line 825: * This function is resilient with respect to system clock -- To view, visit https://gerrit.ovirt.org/59295 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0dc1560e81783a7c169896c235b043f50484cc97 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org