This is the mail archive of the
mailing list for the Cygwin project.
re: sched *
- To: <cygwin-patches at cygwin dot com>
- Subject: re: sched *
- From: "Robert Collins" <robert dot collins at itdomain dot com dot au>
- Date: Tue, 20 Mar 2001 08:05:06 +1100
Sorry this isn't a threaded replie... wasn't on the list.
I'm interested in this patch but the ChangeLog needs work. Please read
the Contributing link on the Cygwin web page and make suitable
The problems you had with this ChangeLog are the same as your previous
crossed email - this went in before way you needed the changelog was
clear in my mind. (I had read the guidlines.. but somehow :?)
- You can't add fields to external_pinfo. That's pretty much set in
stone. I probably should have added a version number to this
but... I didn't.
- Please just add appropriate copyrights using other files as
Copyright 2001 Red Hat, Inc.
- Please use GNU coding standards. I forgot to check this with your
patch. This one has varying styles for spaces before parentheses and
arithmatic operators. If you run your code through 'indent' it should
an ok job. You'll probably have to hand inspect it afterwards,
- Why are you setting the thread priority as well as the process
Cygwin's nice implementation sticks to just setting the process
I try not to add to the pinfo structure if I can help it since it
shared memory, so if the addition of a thread id can be eliminated
I think we should do so.
- What's MEOR2000only? It seems to be used around w2k-only stuff and,
- I don't like the sched.cc/schedule.cc file names. The names are too
Can they be put into one file or renamed?
- Do these fall under the category of pthreads routines? If so, the
lines should be prefixed with @PTH_ALLOW@.
- Earnie, if you're reading this, please note the w32api change. It
w2k/winme OpenThread() function to winbase.h
external_pinfo doesn't have to change now. (Thanks to pinfo).
copyright: thanks will do.
indent - sure. What version, and what parameters? (I've been bitten
before :] )
thread priority - because down the track I hope to implement the posix
pthread_attr_*sched* functions. What they do is set the priority of
threads at creation time. .... and I've just taken the time to reread
all the detail.. We don't _need_ to set the thread priority, I was just
trying to get a better map function (32 priorities to 15 rather than 4 )
MEOR2000only - it was a prompt for you to go "what is this". It was to
compile the cygwin1.dll on non win2k or Me machines as I realised late
in the piece that Openthread was 2k only. If we can bind the functions
in cygwin1.dll only when we need them that could be a test for what os
we are running on. but as that code now goes away, it's not an issue.
Again we've crossed paths. I'll merge sched.cc and schedule.cc as I was
just following existing code layout (ironically you then asked me why
it's done that way!).
Pthreads routines? Uhmm no the are realtime routines but they operate
independently of threaded code. If I start to use threaded code features
in them then I'll code an appropriate check ?
The w32api change was written of the MS Documentation at msdn... It
won't be needed now, but I figure completeness is a good thing.