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

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

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

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

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

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]

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

[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 ---