Skip to content

THistSvc Threading Improvement, master branch (2021.05.13.)

Attila Krasznahorkay requested to merge akraszna/GaudiGaudi:THistSvcUpdate-master-20210513 into master

This is a twin of atlas/Gaudi!228, collecting the fix for #179 (closed) into the main repository as well.

As explained in #179 (closed), THistSvc shows up as a bottleneck in ATLAS reconstruction jobs with >32 threads. This is meant to fix that bottleneck. (Surprisingly enough this is the only "threading issue" in the service.)

The update, as you can see, is very trivial. I just added a const TClass* pointer to the service, which is set up in THistSvc::initialize(). This way we avoid ROOT's internal mutex when calling THistSvc::updateFiles().

Closes #179 (closed)

Edited by Marco Clemencic

Merge request reports