SYCL Updates, master branch (2020.08.14.)

I wrote a long and nice description about this. And then GitLab completely died under me, and forgot the whole text that I wrote.

So screw it. One of the updates is here to get rid of these sort of warnings while compiling Athena with DPC++:

In file included from /data/projects/athena-gpu/athena/Control/StoreGate/src/ActiveStoreSvc.cxx:6:
In file included from /data/projects/athena-gpu/athena/Control/StoreGate/StoreGate/StoreGateSvc.h:22:
In file included from /data/projects/athena-gpu/athena/Control/StoreGate/StoreGate/tools/SGImplSvc.h:54:
In file included from /data/projects/athena-gpu/athena/Control/StoreGate/StoreGate/SGIterator.h:18:
In file included from /data/projects/athena-gpu/athena/Control/SGTools/SGTools/DataProxy.h:431:
In file included from /data/projects/athena-gpu/athena/Control/SGTools/SGTools/DataProxy.icc:16:
In file included from /data/projects/athena-gpu/athena/Control/AthenaKernel/AthenaKernel/StorableConversions.h:15:
In file included from /data/projects/athena-gpu/athena/Control/AthenaKernel/AthenaKernel/DataBucket.h:129:
/data/projects/athena-gpu/athena/Control/AthenaKernel/AthenaKernel/DataBucket.icc:186:18: warning: ISO C++ requires the name after '::~' to be found in the same scope as the name before '::~' [-Wdtor-name]
SG::DataBucket<T>::~DataBucket()
~~~~~~~~~~~~~~~~~^~
                 ::DataBucket
1 warning generated.

The other update is necessary to make DPC++ not freak out about multi-versioned functions. (See: https://github.com/intel/llvm/issues/2128)

Pinging @ssnyder.

This merge request affects 2 packages:

• Control/AthenaKernel
• Control/CxxUtils

Adding @ssnyder ,@rbianchi as watchers

added Core master review-pending-level-1 labels

mentioned in merge request atlasexternals!714 (merged)

ok, thanks.

CI Result FAILURE (hash 580d365c)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 18743]

"Trigger_athena_MC-test" failed with:

...
EMVertexBuilder                                   FATAL No TrackParticleInputContainer found in TDS
EMVertexBuilder                                   ERROR Maximum number of errors ( 'ErrorMax':1) reached.
AthAlgSeq                                          INFO execute of [EMVertexBuilder] did NOT succeed
AthAlgSeq                                         ERROR Maximum number of errors ( 'ErrorMax':1) reached.
AthAllAlgSeq                                       INFO execute of [AthAlgSeq] did NOT succeed
AthAllAlgSeq                                      ERROR Maximum number of errors ( 'ErrorMax':1) reached.
AthAlgEvtSeq                                       INFO execute of [AthAllAlgSeq] did NOT succeed
AthAlgEvtSeq                                      ERROR Maximum number of errors ( 'ErrorMax':1) reached.
AthMasterSeq                                       INFO execute of [AthAlgEvtSeq] did NOT succeed
AthMasterSeq                                      ERROR Maximum number of errors ( 'ErrorMax':1) reached.
AthenaEventLoopMgr                                 INFO Execution of algorithm AthMasterSeq failed with StatusCode::FAILURE
AthenaEventLoopMgr                                 INFO   ===>>>  done processing event #76505, run #300000 1 events processed so far  <<<===
...

Which I can just not imagine would be related to these changes. Are such issues seen in other CIs as well?

The "DataQuality_required-test" seems to have run into a Wall time limit. At least at the top of the log file it says:

[CI_TEST] Starting test script /var/lib/jenkins/workspace/CI-test-driver/ci_test_driver/DataQuality_required.citest
[CI_TEST] on machine aibuild20-050.cern.ch
[CI_TEST] Wall clock time limit is 30 minutes

And then the log finishes with:

...
PyJobTransforms.main 2020-08-15 02:21:50,326 INFO /build2/ci-builds/master/Athena/build/Athena/x86_64-centos7-gcc8-opt/share/Reco_tf.py stopped at Sat Aug 15 02:21:50 2020, trf exit code 0
Run3DQTestingDriver.py 'Input.Files=["myAOD.pool.root"]' DQ.Environment=AOD Sat Aug 15 02:21:50 CEST 2020
ci_test_kill_fam: handling process 25482 ... 
ci_test_kill_fam: killing 25482 with signal 9 
ci_test_kill_fam: killing 28440 with signal 9 
/var/lib/jenkins/workspace/CI-test-driver/ci_test_driver/ci_test_wrapper.sh: line 45: 25482 Killed                  $@

real	31m15.228s
user	28m56.325s
sys	1m3.316s
[CI_TEST_WRAPPER, test start time] Sat Aug 15 01:50:46 CEST 2020
[CI_TEST_WRAPPER, test end time] Sat Aug 15 02:22:01 CEST 2020 exit code 137
CI_TEST/CI_TEST_RUNNER: Info: test completed, name, stat and date: DataQuality_required 137 Sat Aug 15 02:22:02 CEST 2020
CI_TEST/CI_TEST_RUNNER: TEST FAILURE: DataQuality_required returned code 137

EOS/DB issues perhaps? Normally I would just restart the CI, but the issue in "Trigger_athena_MC-test" would probably persist. So there doesn't seem to be much point...

mentioned in merge request atlasexternals!715 (closed)

"Trigger_athena_MC-test"

EMVertexBuilder                                   FATAL No TrackParticleInputContainer found in TDS

should have been fixed by this

!35612 (merged)

CI after that seem to succeed on this test

Jenkins please retry a build

This merge request affects 2 packages:

• Control/AthenaKernel
• Control/CxxUtils

Adding @ssnyder ,@rbianchi as watchers

CI Result SUCCESS (hash 580d365c)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 18798]

Thanks Christos!

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit cce0c611

added sweep:ignore label