From: Subrata M. <su...@li...> - 2009-08-25 09:59:04
|
On Mon, 2009-08-24 at 15:16 -0700, Darren Hart wrote: > prio-wake: enable using more than 95 threads > > prio-wake will currently spew pthread errors on systems with more > than 95 CPUS, or if an -n value > 95 is specified on the command line. > To avoid this, spread threads out equally over the priorities by > calculated the number of threads per priority. > > Tested with all pathological numbers (i.e. mod=0 mod=1, etc) of threads. > Ran 10000 times in parallel. Confirmed failure continues to exist without > requeue PI (as it should) and success is seen with requeue PI > (as it should be). > > Signed-off-by: Darren Hart <dv...@us...> All Patches are in. Regards-- Subrata > Acked-By: Dinakar Guniguntala <di...@in...> > Acked-by: Vernon Mauery <ve...@us...> > Acked-by: Gowrishankar <gow...@in...> > > --- > func/prio-wake/prio-wake.c | 24 +++++++++++++++++++----- > 1 file changed, 19 insertions(+), 5 deletions(-) > > Index: realtime/func/prio-wake/prio-wake.c > =================================================================== > --- realtime.orig/func/prio-wake/prio-wake.c > +++ realtime/func/prio-wake/prio-wake.c > @@ -54,6 +54,8 @@ > #include <errno.h> > #include <sys/syscall.h> > #include <librttest.h> > +#include <libstats.h> > + > volatile int running_threads = 0; > static int rt_threads = 0; > static int locked_broadcast = 1; > @@ -164,8 +166,10 @@ void *worker_thread(void* arg) > > int main(int argc, char* argv[]) > { > - int pri_boost; > + int threads_per_prio; > int numcpus; > + int numprios; > + int prio; > int i; > setup(); > > @@ -184,19 +188,29 @@ int main(int argc, char* argv[]) > printf("Calling pthread_cond_broadcast() with mutex: %s\n\n", > locked_broadcast ? "LOCKED" : "UNLOCKED"); > > - pri_boost = 3; > - > beginrun = rt_gettime(); > > init_pi_mutex(&mutex); > > + /* calculate the number of threads per priority */ > + /* we get num numprios -1 for the workers, leaving one for the master */ > + numprios = sched_get_priority_max(SCHED_FIFO) - > + sched_get_priority_min(SCHED_FIFO); > + > + threads_per_prio = rt_threads / numprios; > + if (rt_threads % numprios) > + threads_per_prio++; > + > /* start the worker threads */ > + prio = sched_get_priority_min(SCHED_FIFO); > for (i = rt_threads; i > 0; i--) { > - create_fifo_thread(worker_thread, (void*)(intptr_t)i, sched_get_priority_min(SCHED_FIFO) + pri_boost++); > + if ((i != rt_threads && (i % threads_per_prio) == 0)) > + prio++; > + create_fifo_thread(worker_thread, (void*)(intptr_t)i, prio); > } > > /* start the master thread */ > - create_fifo_thread(master_thread, (void*)(intptr_t)i, sched_get_priority_min(SCHED_FIFO) + pri_boost); > + create_fifo_thread(master_thread, (void*)(intptr_t)i, ++prio); > > /* wait for threads to complete */ > join_threads(); |