Re: Firefox/libthread_xu issue with pthread_attr_setschedparam

[Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>]

:...
:> THR_MAX_PRIORITY(31), both defined in 
:> lib/libthread_xu/thread/thr_private.h. When a value in the range [-20; 
:> -1] is used, pthread_attr_setschedparam returns an error code of 
:> ENOTSUPP, which NSPR doesn't like. More precisely, when Firefox is 
:> compiled with the 'debug' pkgsrc option, an assert fails, abort() is 
:> called and a coredump written.
:
:I thought we had fixed that some time ago already?

     Nope, I'm looking at the code and it is definitely a problem.  There
     are a bunch of THR_ defines that are essentially not used by
     the library at all, except for that (broken) range check.

     The only thing that uses THR_MAX_PRIORITY is the m_ceiling
     attribute in the mutex attribute structure, which we don't use
     at all (and I think it operates in a private scheduling domain
     anyway).

     I'll have a patch for people w/ the firefox setup to try in a moment.

					-Matt
					Matthew Dillon 
					<dillon@backplane.com>