Page MenuHomeFreeBSD

riscv: Implement non-stub __vdso_gettc and __vdso_gettimekeep
ClosedPublic

Authored by jrtc27 on Jun 30 2021, 3:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 25, 7:35 PM
Unknown Object (File)
Sat, Jan 25, 9:01 AM
Unknown Object (File)
Fri, Jan 24, 10:57 PM
Unknown Object (File)
Wed, Jan 22, 10:53 PM
Unknown Object (File)
Sat, Jan 18, 10:12 PM
Unknown Object (File)
Sat, Jan 18, 5:43 PM
Unknown Object (File)
Thu, Jan 9, 11:50 AM
Unknown Object (File)
Nov 28 2024, 7:54 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 40188
Build 37077: arc lint + arc unit

Event Timeline

lib/libc/riscv/sys/__vdso_gettc.c
47

Hm, plain csr_read might be more appropriate given the result is put in a u_int, though it's a bit academic given we don't support RV32. The two are the same on RV64 but on RV32 it'll loop until it reads a consistent time and timeh to get a 64-bit value from two 32-bit CSRs and then throw away the upper half.

Use csr_read in place of csr_read64

Thanks! I can confirm that truss no longer shows the clock_gettime() syscalls with this version of the patch.

This revision is now accepted and ready to land.Jun 30 2021, 4:35 PM
mhorne added a subscriber: mhorne.

Thanks for this one.

lib/libc/riscv/sys/__vdso_gettc.c
28

Nit: not required for new files

lib/libc/riscv/sys/__vdso_gettc.c
28

for new files that won't be MFC'd to stable/12..... I don't think this will be, given the riscv support in 12 isn't so good (IIRC, I might be wrong), but thought I'd mention in case I am wrong.

lib/libc/riscv/sys/__vdso_gettc.c
28

That's right, there's no point bringing this past stable/13, but the precision is helpful.