This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: [PATCH] pthread_getattr_np, pthread_setschedprio
- From: "Yaakov (Cygwin/X)" <yselkowitz at users dot sourceforge dot net>
- To: cygwin-patches <cygwin-patches at cygwin dot com>
- Date: Thu, 14 Apr 2011 22:00:32 -0500
- Subject: Re: [PATCH] pthread_getattr_np, pthread_setschedprio
- References: <1302489035.4944.20.camel@YAAKOV04>
On Sun, 2011-04-10 at 21:30 -0500, Yaakov (Cygwin/X) wrote:
> This patch adds two pthread functions which appear to be "low-hanging
> fruit".
>
> pthread_setschedprio(3) is a POSIX function[1][2] which changes the
> scheduling priority for the given thread. It is similar to
> pthread_setschedparam(3) but does not change the scheduling policy and
> doesn't require the priority to be wrapped in a struct.
>
> pthread_getattr_np(3) is a GNU extension[3] which initializes the given
> pthread_attr_t with the actual attributes of the given thread. While
> the example code does not have the pthread_attr_t pre-initialized by
> pthread_attr_init(3), I have seen real world code where it is, so either
> possibility is handled.
After further analysis, I should work further on pthread_getattr_np() in
conjunction with adding pthread_attr_getstack(), both of which I need
for webkitgtk-1.3.13. But it will be a couple of weeks before I'll have
the time to get to that.
So for now, lets just implement pthread_setschedprio(), which looks
pretty straight-forward and is unrelated to the others. Revised patch
attached.
Yaakov
2011-04-10 Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
* thread.cc (pthread_setschedprio): New function.
* include/pthread.h (pthread_setschedprio): Declare.
* include/cygwin/version.h (CYGWIN_VERSION_API_MINOR): Bump.
* cygwin.din (pthread_setschedprio): Export.
* posix.sgml (std-notimpl) Move pthread_setschedprio from here...
(std-susv4) ...to here.
Index: cygwin.din
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/cygwin.din,v
retrieving revision 1.234
diff -u -r1.234 cygwin.din
--- cygwin.din 29 Mar 2011 10:32:40 -0000 1.234
+++ cygwin.din 10 Apr 2011 08:49:54 -0000
@@ -1241,6 +1241,7 @@
pthread_setcanceltype SIGFE
pthread_setconcurrency SIGFE
pthread_setschedparam SIGFE
+pthread_setschedprio SIGFE
pthread_setspecific SIGFE
pthread_sigmask SIGFE
pthread_suspend SIGFE
Index: posix.sgml
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/posix.sgml,v
retrieving revision 1.55
diff -u -r1.55 posix.sgml
--- posix.sgml 29 Mar 2011 10:32:40 -0000 1.55
+++ posix.sgml 10 Apr 2011 08:49:54 -0000
@@ -598,6 +598,7 @@
pthread_setcanceltype
pthread_setconcurrency
pthread_setschedparam
+ pthread_setschedprio
pthread_setspecific
pthread_sigmask
pthread_spin_destroy
@@ -1388,7 +1389,6 @@
pthread_mutex_timedlock
pthread_rwlock_timedrdlock
pthread_rwlock_timedwrlock
- pthread_setschedprio
putmsg
reminderl
remquol
Index: thread.cc
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/thread.cc,v
retrieving revision 1.227
diff -u -r1.227 thread.cc
--- thread.cc 29 Mar 2011 10:32:40 -0000 1.227
+++ thread.cc 10 Apr 2011 08:49:55 -0000
@@ -2306,6 +2329,17 @@
return rv;
}
+extern "C" int
+pthread_setschedprio (pthread_t thread, int priority)
+{
+ if (!pthread::is_good_object (&thread))
+ return ESRCH;
+ int rv =
+ sched_set_thread_priority (thread->win32_obj_id, priority);
+ if (!rv)
+ thread->attr.schedparam.sched_priority = priority;
+ return rv;
+}
extern "C" int
pthread_setspecific (pthread_key_t key, const void *value)
Index: include/pthread.h
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/include/pthread.h,v
retrieving revision 1.29
diff -u -r1.29 pthread.h
--- include/pthread.h 29 Mar 2011 10:32:40 -0000 1.29
+++ include/pthread.h 10 Apr 2011 08:49:55 -0000
@@ -194,6 +194,7 @@
int pthread_setcancelstate (int, int *);
int pthread_setcanceltype (int, int *);
int pthread_setschedparam (pthread_t, int, const struct sched_param *);
+int pthread_setschedprio (pthread_t, int);
int pthread_setspecific (pthread_key_t, const void *);
void pthread_testcancel (void);
Index: include/cygwin/version.h
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/include/cygwin/version.h,v
retrieving revision 1.339
diff -u -r1.339 version.h
--- include/cygwin/version.h 29 Mar 2011 10:32:40 -0000 1.339
+++ include/cygwin/version.h 10 Apr 2011 08:49:55 -0000
@@ -403,12 +403,13 @@
237: Export strchrnul.
238: Export pthread_spin_destroy, pthread_spin_init, pthread_spin_lock,
pthread_spin_trylock, pthread_spin_unlock.
+ 239: Export pthread_setschedprio.
*/
/* Note that we forgot to bump the api for ualarm, strtoll, strtoull */
#define CYGWIN_VERSION_API_MAJOR 0
-#define CYGWIN_VERSION_API_MINOR 238
+#define CYGWIN_VERSION_API_MINOR 239
/* There is also a compatibity version number associated with the
shared memory regions. It is incremented when incompatible