RE: Fix nanosleep returning negative rem

2021-07-21 Thread Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
> I can get it easily get this on my desktop (AMD Ryzen Threadripper 3990X) but > not at all on my laptop (Intel Core i7-8650U) Not sure if that's really related but: Have you checked what is your default Windows timer resolution? Some applications change it from the default 100HZ to 1000HZ (b

RE: Fix nanosleep returning negative rem

2021-07-21 Thread David Allsopp
Corinna Vinschen wrote: > Sent: 21 July 2021 10:33 > To: cygwin-patches@cygwin.com > Subject: Re: Fix nanosleep returning negative rem > > On Jul 21 11:30, Corinna Vinschen wrote: > > I wrote a quick STC using the NT API calls and I can't reproduce the > > problem with this code either. The outpu

Re: [PATCH 0/3] Add more winsymlinks values

2021-07-21 Thread Christian Franke
Corinna Vinschen wrote: On Jul 19 17:31, Jon Turney wrote: I'm not sure this is the best idea, since it adds more configurations that aren't going to get tested often, but the idea is that this would enable proper and consistent control of the symlink type used from setup, as discussed in [1].

Re: Fix nanosleep returning negative rem

2021-07-21 Thread Corinna Vinschen
On Jul 21 11:30, Corinna Vinschen wrote: > I wrote a quick STC using the NT API calls and I can't reproduce the > problem with this code either. The output is either > > SignalState: 1 TimeRemaining: -5354077459183 > > or > > SignalState: 0 TimeRemaining: 653 > > I never get a small negati

Re: Fix nanosleep returning negative rem

2021-07-21 Thread Corinna Vinschen
On Jul 21 09:07, David Allsopp wrote: > > On Jul 20 16:16, David Allsopp wrote: > > > I've pushed a repro case for this to > > > https://github.com/dra27/cygwin-nanosleep-bug.git > > > > > > Originally noticed as the main CI system for OCaml has been failing > > > sporadically for the signal.ml tes

RE: Fix nanosleep returning negative rem

2021-07-21 Thread David Allsopp
> On Jul 20 16:16, David Allsopp wrote: > > I've pushed a repro case for this to > > https://github.com/dra27/cygwin-nanosleep-bug.git > > > > Originally noticed as the main CI system for OCaml has been failing > > sporadically for the signal.ml test mentioned in that repo. This > > morning I tried

Re: Fix nanosleep returning negative rem

2021-07-21 Thread Corinna Vinschen
Hi David, On Jul 20 16:16, David Allsopp wrote: > I've pushed a repro case for this to > https://github.com/dra27/cygwin-nanosleep-bug.git > > Originally noticed as the main CI system for OCaml has been failing > sporadically for the signal.ml test mentioned in that repo. This morning I > tried h

Re: [PATCH 0/3] Add more winsymlinks values

2021-07-21 Thread Corinna Vinschen
On Jul 19 17:31, Jon Turney wrote: > I'm not sure this is the best idea, since it adds more configurations that > aren't going to get tested often, but the idea is that this would enable > proper and consistent control of the symlink type used from setup, as > discussed in [1]. > > [1] https://cyg

Re: [PATCH] Cygwin: fix format warnings in profiler.cc

2021-07-21 Thread Corinna Vinschen
On Jul 21 01:00, Mark Geisert wrote: > Use new typedef to normalize pids for printing on both 32- and 64-bit Cygwin. I pushed my patch before I saw yours. If you want to use ulong, please send a followup patch which introduces the typedef. Thanks, Corinna

Re: [PATCH 1/3] Cygwin: New tool: profiler

2021-07-21 Thread Corinna Vinschen
On Jul 19 16:43, Jon Turney wrote: > On 19/07/2021 15:23, Jon Turney wrote: > > On 19/07/2021 11:04, Corinna Vinschen wrote: > > > Hi Matt, > > > > > > On Jul 15 21:49, Mark Geisert wrote: > > > > The new tool formerly known as cygmon is renamed to 'profiler'.  For the > > > > name I considered 'i

[PATCH] Cygwin: fix format warnings in profiler.cc

2021-07-21 Thread Mark Geisert
Use new typedef to normalize pids for printing on both 32- and 64-bit Cygwin. --- winsup/utils/profiler.cc | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/winsup/utils/profiler.cc b/winsup/utils/profiler.cc index d1a01c3a2..152bf1cca 100644 --- a/winsup/utils/profiler