Merge pull request #49282 from AdrianoDee/nHitsToCells_clamp_fix_160X

Proper Clamping for CA Doublets Histogram

Author: cmsbuild

RecoTracker/PixelSeeding/plugins/alpaka/CAHitNtupletGeneratorKernels.dev.cc

@@ -57,8 +57,8 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {
 #ifdef GPU_DEBUG
     std::cout << "Allocation for tuple building with: " << std::endl;
     std::cout << "- nHits          = " << nHits << std::endl;
-    std::cout << "- maxDoublets    = " << maxTuples << std::endl;
-    std::cout << "- maxTracks      = " << maxDoublets << std::endl;
+    std::cout << "- maxDoublets    = " << maxDoublets << std::endl;
+    std::cout << "- maxTracks      = " << maxTuples << std::endl;
 
     std::cout << "- nCellsToCells  = " << nCellsToCells << std::endl;
     std::cout << "- nHitsToCells   = " << nHitsToCells << std::endl;

RecoTracker/PixelSeeding/plugins/alpaka/CAPixelDoubletsAlgos.h

@@ -420,9 +420,11 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE::caPixelDoublets {
           }
 
           auto ind = alpaka::atomicAdd(acc, nCells, 1u, alpaka::hierarchy::Blocks{});
-          if (ind >= maxNumOfDoublets) {
+          if (ind >= maxNumOfDoublets or ind >= uint32_t(outerHitHisto->capacity())) {
 #ifdef CA_WARNINGS
-            printf("Warning!!!! Too many cells (limit = %d)!\n", maxNumOfDoublets);
+            printf("Warning!!!! Too many cells (maxNumOfDoublets = %d - nHitsToCell = %d)!\n",
+                   maxNumOfDoublets,
+                   outerHitHisto->capacity());
 #endif
             alpaka::atomicSub(acc, nCells, 1u, alpaka::hierarchy::Blocks{});
             break;