Move LogWarning into the EDProducer (#422)

In CMSSW 11.0.0 the MessageLogger is no longer usable in CUDA code due to the use of c++17 featutres.
cms-patatrack · Nov 6, 2020 · a46353c · a46353c
1 parent 3af2fac
commit a46353c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/RecoLocalTracker/SiPixelRecHits/plugins/PixelRecHits.cu b/RecoLocalTracker/SiPixelRecHits/plugins/PixelRecHits.cu
@@ -6,7 +6,6 @@
 #include <cuda_runtime.h>
 
 // CMSSW headers
-#include "FWCore/MessageLogger/interface/MessageLogger.h"
 #include "HeterogeneousCore/CUDAUtilities/interface/cudaCheck.h"
 #include "HeterogeneousCore/CUDAUtilities/interface/device_unique_ptr.h"
 #include "RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.h"
@@ -63,11 +62,6 @@ namespace pixelgpudetails {
       cudaCheck(cudaGetLastError());
     }
 
-    if (nHits >= TrackingRecHit2DSOAView::maxHits()) {
-      edm::LogWarning("PixelRecHitGPUKernel")
-          << "Hits Overflow " << nHits << " > " << TrackingRecHit2DSOAView::maxHits();
-    }
-
     if (nHits) {
       auto hws = cudautils::make_device_unique<uint8_t[]>(TrackingRecHit2DSOAView::Hist::wsSize(), stream);
       cudautils::fillManyFromVector(

diff --git a/RecoLocalTracker/SiPixelRecHits/plugins/SiPixelRecHitCUDA.cc b/RecoLocalTracker/SiPixelRecHits/plugins/SiPixelRecHitCUDA.cc
@@ -96,6 +96,12 @@ void SiPixelRecHitCUDA::produce(edm::StreamID streamID, edm::Event& iEvent, cons
   iEvent.getByToken(tBeamSpot, hbs);
   auto const& bs = ctx.get(*hbs);
 
+  auto nHits = clusters.nClusters();
+  if (nHits >= TrackingRecHit2DSOAView::maxHits()) {
+    edm::LogWarning("PixelRecHitGPUKernel")
+        << "Clusters/Hits Overflow " << nHits << " >= " << TrackingRecHit2DSOAView::maxHits();
+  }
+
   ctx.emplace(iEvent,
               tokenHit_,
               gpuAlgo_.makeHitsAsync(digis, clusters, bs, fcpe->getGPUProductAsync(ctx.stream()), ctx.stream()));