Most active commenters
  • asveikau(3)
  • orbisvicis(3)
  • winrid(3)

←back to thread

345 points splitbrain | 16 comments | | HN request time: 0.001s | source | bottom
Show context
OsrsNeedsf2P ◴[] No.41837682[source]
I love how simple this is- Barely 100 lines or C++ (ignoring comments). That's one thing that makes me prefer X11 over Wayland.
replies(8): >>41837906 #>>41838181 #>>41838339 #>>41838393 #>>41838489 #>>41838500 #>>41838693 #>>41844644 #
asveikau ◴[] No.41838339[source]
The code is a little weird. There is no XLib event loop. It calls sleep(100) in a loop until it hits SIGINT. That will have high cpu usage for no reason.
replies(3): >>41838378 #>>41844664 #>>41848003 #
diath ◴[] No.41838378[source]
It will not, even adding just a 1ms sleep in a loop will drop CPU usage to barely noticeable levels, 10 wakes a second is barely anything for any CPU from the past 3 decades.
replies(5): >>41838399 #>>41839735 #>>41840345 #>>41845898 #>>41848081 #
1. asveikau ◴[] No.41838399{3}[source]
Not my experience at all. Granted I haven't tried writing a loop like this in 20ish years, because once you spot that mistake you don't tend to make it again, and CPUs are better now.

Another thing to note is when you call sleep with a low value it may decide not to sleep at all, so this loop just might be constantly doing syscalls in a tight loop.

replies(1): >>41838510 #
2. diath ◴[] No.41838510[source]
> Not my experience at all. Granted I haven't tried writing a loop like this in 20ish years, because once you spot that mistake you don't tend to make it again, and CPUs are better now.

You can trivially verify it by running the following, I have personally been using "sleep for 1ms in a loop to prevent CPU burn" for years and never noticed it having any impact, it's not until I go into microseconds when I can start noticing my CPU doing more busy work.

    // g++ -std=c++20 -osleep sleep.cpp
    #include <thread>
    #include <chrono>

    int main(int, char **)
    {
     while (true) {
     std::this_thread::sleep_for(std::chrono::milliseconds {1});
     }
     return 0;
    }
> Another thing to note is when you call sleep with a low value it may decide not to sleep at all, so this loop just might be constantly doing syscalls in a tight loop.

On what system? AFAIK, if your sleep time is low enough, it will round up to whatever is the OS clock resolution multiple, not skip the sleep call completely. On Linux, it will use nanosleep(2) and I cannot see any mention of the sleep not suspending the thread at all with low values.

replies(2): >>41839461 #>>41843456 #
3. asveikau ◴[] No.41839461[source]
If memory serves, Windows treats a sleep under the scheduler quantum length as a yield. It may take you off the cpu if there's something else to run but it may not. Meanwhile burning up cycles may prevent low power states.

At any rate, back to the code at hand, there are many ways to block on SIGINT without polling. But it's also hugely odd that this code does not read events from the X11 socket while it does so. This is code smell, and a poorly behaved X client.

replies(1): >>41841429 #
4. orbisvicis ◴[] No.41841429{3}[source]
I thought that Linux behaved the same, but I'm not finding any proof in `man 2 nanosleep`...
replies(2): >>41842283 #>>41847422 #
5. eqvinox ◴[] No.41842283{4}[source]
You can't find that proof because Linux does the opposite. Unless your task is SCHED_REALTIME, all timers have a little bit of slack at the end that allows the kernel to group wakeup events. You can configure this (for non-RT tasks) with prctl(PR_SET_TIMERSLACK).

https://lxr.linux.no/#linux+v6.7.1/kernel/time/hrtimer.c#L20...

https://www.man7.org/linux/man-pages/man2/PR_SET_TIMERSLACK....

replies(1): >>41845644 #
6. 01HNNWZ0MV43FF ◴[] No.41843456[source]
> never noticed

I'd love to see numbers with a Kill-A-watt between the PC and the wall

replies(1): >>41844219 #
7. winrid ◴[] No.41844219{3}[source]
Why? Running an empty loop a thousand times a second is literally almost nothing to any cpu released in the past 20yrs at least
replies(2): >>41844714 #>>41850243 #
8. EasyMark ◴[] No.41844714{4}[source]
Running an empty loop with no sleep or other yield type operation will peg one of your cores if you pin it to that core.

  Int main() {
    while(1) {}
  }
compile that with -O0 and see what happens.
replies(2): >>41845249 #>>41846966 #
9. winrid ◴[] No.41845249{5}[source]
When did we say anything about no sleep or yield? That's completely different. Read the thread.
10. orbisvicis ◴[] No.41845644{5}[source]
Sorry! I was looking for documentation that on Linux sleep(0) yields.
replies(1): >>41846639 #
11. eqvinox ◴[] No.41846639{6}[source]
There is no code in nanosleep that converts it into a yield, and in fact a nanosleep(0) is a nanosleep(50µs) with the default timer slack value. If you want to yield, call sched_yield() …
replies(1): >>41847428 #
12. lupusreal ◴[] No.41846966{5}[source]
"A thousand times a second" obviously implies sleep or yield unless your computer is old enough to be your grandfather.
13. gpderetta ◴[] No.41847422{4}[source]
It used to be the case that glibc implemented nanosleep for small values below the scheduling quantum with a spin loop. It was explicitly documented to do so.

This was changed sometimes in the last 20 years, probably with battery powered devices becoming more prevalent and CPUs implementing more advanced sleep states.

14. orbisvicis ◴[] No.41847428{7}[source]
I looked into this a bit further and it seems to be a range from [0,50]. [1] explains that if there is a pre-existing timer interrupt at 0, then the queue will be resumed at 0. But yes - given no other timers it will resume at 50.

1. https://people.kernel.org/joelfernandes/on-workings-of-hrtim...

15. 01HNNWZ0MV43FF ◴[] No.41850243{4}[source]
For science, literal proving hypotheses science
replies(1): >>41871409 #
16. winrid ◴[] No.41871409{5}[source]
Well, I already run my setup through a kilowatt. I don't see a difference with Python, which you can argue should be 10-100x less efficient:

import time

while True:

    time.sleep(0.001)

also the script itself it bounces between 0% and 1% cpu usage