DUNE-DAQ / nanorc

2 stars 2 forks source link

Thread pinning #243

Closed plasorak closed 4 months ago

plasorak commented 4 months ago

Better thread pinning:

plasorak commented 4 months ago

This needs to be used with https://github.com/DUNE-DAQ/fddaqconf/pull/20 (but not necessarily, as this is backwards compatible)

wesketchum commented 4 months ago

Just tested at EHN1. Works / is backwards compatible with existing configs for workflow at NP04. Reduced messages are very nice, though could additionally suppress the details on the pin command. Fixes the spurious error on pin_threads for WIBs/hermes/etc.

I would be happy to see this in fddaq-v4.3.0, even without https://github.com/DUNE-DAQ/fddaqconf/pull/20.

wesketchum commented 4 months ago

@plasorak is it possible to split out the changes on the verbosity of print messages and the errors on WIB/hermes/etc. blocks versus the other updates that are tied to the configuration? If so, that may be enough to get in now, and allow for further discussion if needed on the other elements, assuming that is needed.

plasorak commented 4 months ago

I can certainly cherry-pick the commits which reduce the printout. For the WIB/Hermes errors, if you are referring to https://github.com/DUNE-DAQ/ehn1-operations-issues/issues/19, yes that can be done too.

plasorak commented 4 months ago

done here: https://github.com/DUNE-DAQ/nanorc/pull/244