Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Proper system clocks #120

Closed
2 tasks done
brabo opened this issue Mar 8, 2017 · 5 comments
Closed
2 tasks done

Proper system clocks #120

brabo opened this issue Mar 8, 2017 · 5 comments

Comments

@brabo
Copy link
Member

brabo commented Mar 8, 2017 •
edited
Loading

  • use jiffies and read that as CLOCK_MONOTONIC
  • use rt offset and read/write that as CLOCK_REALTIME and for [s|g]ettimeofday()
brabo added a commit to insane-adding-machines/frosted-headers that referenced this issue Mar 13, 2017
@brabo
switched gettimeofday syscall to clock_gettime, insane-adding-machine…
e1c4b62 
…s/frosted#120
brabo added a commit to insane-adding-machines/newlib that referenced this issue Mar 13, 2017
@brabo
(get/set)timeofday now using sys_clock_(get/set)time, insane-adding-m…
5e40781 
…achines/frosted#120
brabo added a commit to insane-adding-machines/frosted-headers that referenced this issue Mar 13, 2017
@brabo
switched gettimeofday syscall to clock_gettime and added clock_settime,
7351690 
insane-adding-machines/frosted#120
brabo added a commit that referenced this issue Mar 13, 2017
@brabo
clock rework, #120
91e4748
@brabo brabo closed this as completed Mar 13, 2017
@danielinux
Copy link
Contributor

Where was CLOCK_MONOTONIC supposed to be defined in kernel space? I had to add an explicit define in sys.c because I cannot find any reference to that symbol in kernel code. Missing common header?

@danielinux danielinux reopened this Mar 14, 2017
@danielinux danielinux mentioned this issue Mar 14, 2017
Closed
@brabo
Copy link
Member Author

brabo commented Mar 14, 2017 •
edited
Loading

<time.h> as:
https://github.com/insane-adding-machines/frosted/blob/master/kernel/sys.c#L23

@danielinux
Copy link
Contributor

Time.h is nowhere to be found in kernel code. Is it a libc include? Libc include are ignored in kernel, as we compile with nostdinc.

@brabo
Copy link
Member Author

brabo commented Mar 14, 2017

it is in newlib. that flag seems gone, compiles and works fine this way. i agree we should split it off to frosted-headers though and include that both ways.

@danielinux
Copy link
Contributor

The stdinc problem is not only related to this. This issue can be closed, but the problem persists until we fix #121

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brabo @danielinux