Skip to content

Add protections to muon decoding

Saverio Mariani requested to merge samarian_protect_muon_decoding into master

MR following up LHCb!3736 (merged) and adding protections to the Allen muon decoding to avoid the crashes that were reported in LHCb#242 because of an inadequate raw bank size. As resulting from some offline discussions, the full raw bank has to be skipped in case a Tell40 link gives corrupted info.

Performed tests:

  • Allen launched from Moore can decode the full data file indicated in LHCb#242 with no crashes
  • Moore hlt1_hlt2_muon_hits, comparing Rec and Allen decoded hits, is successful
  • throughput on data of the muon_decoding_sequence slows down by a relative 5.9%. Considering the muon decoding only represents a 0.8% of the overall sequence, this is expected to be acceptable
Edited by Saverio Mariani

Merge request reports