Benjamin Michael Wynne authored 4 years ago and Charles Leggett committed 4 years ago

patch generated by https://gitlab.cern.ch/bwynne/Gaudi_gaudi/-/jobs/8392259

 o avoid missing one worker thread in the TBB global thread pool when ThreadPoolSvc is configured to use all available hardware threads;
 o initialize boost::barrier counter accordingly to the above change to avoid leaving one ThreadInitTask in limbo at termination of ThreadPoolSvc;
 o make ThreadPoolSvc set the right number of worker threads in Gaudi::Concurrency::ConcurrencyFlags.

Also, change the AvalancheSchedulerSvc's help message about worker thread pool size to reflect what is actually going on in the ThreadPoolSvc.

remove move semantics from AvalancheScheduler::signoff, which was added for symmetry with other facilities but is actually an overkill

extend the category of tasks that can be scheduled preemptively to 'CPU-blocking' from just 'I/O-bound' (terminological change only)

merge AlgoExecutionTask and IOBoundAlgTask into AlgTask, introduce TaskSpec and rename IAlgTask to ITask

schedule blocking tasks directly from the AvalancheSchedulerSvc (instead of delegating this to IOBoundAlgSchedulerSvc)

merge scheduling of blocking tasks with scheduling of regular tasks into a single method (and move some log messages around)

Charles Leggett authored 5 years ago and Marco Clemencic committed 5 years ago

This reverts merge requests !869 and !932

Gitlab CI authored 6 years ago and Marco Clemencic committed 6 years ago

patch generated by standalone job

This reverts merge request !719

detect scheduling stalls event-wise, and don't force Avalanche Scheduler to self-deactivate upon a failure/stall (the latter is still handled by HiveSlimEventLoopMgr)