Date: Sun, 12 Jul 2020 18:17:15 +0200 From: Alexander Leidinger <Alexander@leidinger.net> To: Dimitry Andric <dim@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head@freebsd.org Subject: Re: svn commit: r363125 - head/sys/compat/linux Message-ID: <20200712181715.Horde.YfOdq0fEkeFHjcx0fXxQcX1@webmail.leidinger.net> In-Reply-To: <010ACFFA-35B7-4984-A5E7-1504C72D56F9@FreeBSD.org> References: <202007120951.06C9pA4F024281@repo.freebsd.org> <010ACFFA-35B7-4984-A5E7-1504C72D56F9@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format and has been PGP signed. --=_wOku1sUdsJw5iUPKYvDG9Fi Content-Type: text/plain; charset=utf-8; format=flowed; DelSp=Yes Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Quoting Dimitry Andric <dim@freebsd.org> (from Sun, 12 Jul 2020=20=20 15:36:25=20+0200): > On 12 Jul 2020, at 11:51, Alexander Leidinger <netchild@freebsd.org> wrot= e: >> >> Author: netchild >> Date: Sun Jul 12 09:51:09 2020 >> New Revision: 363125 >> URL: https://svnweb.freebsd.org/changeset/base/363125 >> >> Log: >> Implement CLOCK_MONOTONIC_RAW (linux >=3D 2.6.28). >> >> It is documented as a raw hardware-based clock not subject to NTP or >> incremental adjustments. With this "not as precise as CLOCK_MONOTONIC" >> description in mind, map it to our CLOCK_MONOTNIC_FAST (the same >> mapping as for the linux CLOCK_MONOTONIC_COARSE). > > Okay, but: > >> @@ -212,6 +212,7 @@ linux_to_native_clockid(clockid_t *n, clockid_t l) >> *n =3D CLOCK_THREAD_CPUTIME_ID; >> break; >> case LINUX_CLOCK_REALTIME_COARSE: >> + case LINUX_CLOCK_MONOTONIC_RAW: >> *n =3D CLOCK_REALTIME_FAST; >> break; >> case LINUX_CLOCK_MONOTONIC_COARSE: > > this shows it is actually mapped to CLOCK_REALTIME_FAST, not > CLOCK_MONOTONIC_FAST. Is the code right, or the commit message? :) The commit message and the code I have running are right, the code I=20=20 committed=20from a "no other change than that"-svn-tree was not. :( I think I should switch to "diff | patch" even for one-liners like that. I commited the fix based upon another message I've seen before this=20=20 one,=20but thanks for looking at the code. Bye, Alexander. --=20 http://www.Leidinger.net=20Alexander@Leidinger.net: PGP 0x8F31830F9F2772BF http://www.FreeBSD.org netchild@FreeBSD.org : PGP 0x8F31830F9F2772BF --=_wOku1sUdsJw5iUPKYvDG9Fi Content-Type: application/pgp-signature Content-Description: Digitale PGP-Signatur Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAABAgAGBQJfCzeLAAoJEBINsJsD+NiGiH0P/jUVckgElq1GHVof81Ns1uAl s8LUcIHt32XgRTNNJKH672y0X0QipVqrpz+iCghdsfF+zgpIzPI6giz+/ZETaqAs en4ypIW1XZhKjVoSzAhvSUv7B6B0HpQoZxi9O3nSpkDeaRdBFsVuoBcoT/DN3AcY Jb2H87Pl1nBVM4inkqyXfA19ALE+ZSuoYrfoQbJBLuV4iSWJLYecV1YE1Fr4SPfi 2t2lmSYBFpQXhVaXSa351thzpSNrRO1LS5axDkmIjORKnDxb9sn+FPkJrXD5RKfT +8MjjQ5kUrYWfqFPhhPclUXN5Dc2pMrELAJi0lfAH7TBBhL7YVP/IbBBvKBbthtn nUEUYGF471VMnD81EAI016d9SYyoqyIjQM0ab53a8EUDsz+r/tt76UC+nyHs2Rmr yqr5MGTB+dvugrAV22sgCrzxoQ5MbwrS+noqZqBrhNlY3s3ZumMdNcHRu50h9MKB WfBSciH/9061RQXxFG7z7qnH9tr8F376ttydytG4Zu5RDVY7tVx5rBdx4/ImFaLK B8IjoJbLSwnLTtCFPexrKT2BGdnGcj7g4kc3/x3LsjWBHpZUeyFEjF7o5kZJPJEG jKsDJMCvxvGvmpnxZeUYV2Nh1V9/gAVTxLI4UVNrF+q/msvxRG/H7h4p/dwwoHLr 2VMzx8GeUgW59sbOapgK =WHPn -----END PGP SIGNATURE----- --=_wOku1sUdsJw5iUPKYvDG9Fi--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20200712181715.Horde.YfOdq0fEkeFHjcx0fXxQcX1>