visit-dav / visit

VisIt - Visualization and Data Analysis for Mesh-based Scientific Data
https://visit.llnl.gov
BSD 3-Clause "New" or "Revised" License
443 stars 116 forks source link

Modify the internallauncher so that it doesn't create core files on non-AMD systems. (#19996) #19998

Closed brugger1 closed 3 weeks ago

brugger1 commented 3 weeks ago

Merge from the 3.4RC to develop.

Description

Resolves #19992

The internallauncher script uses offload-arch to determine the AMD GPU architecture. Unfortunately this utility can exist on non-AMD systems and cores when run on them. I added additional logic to prevent the script from running offload-arch on non-AMD systems.

I didn't update the release notes since this feature was just added and has not been in a release.

Type of change

How Has This Been Tested?

I built and installed VisIt on poodle, which is a non-AMD system. I ran VisIt from the installation and verified that it properly identified the system type and didn't produce any core files. The system type in this case was linux-x86_64.

I built and installed VisIt on rzvernal, which is an AMD system with MI250X GPUs. I ran VisIt from the installation and verified that it properly identified the system type and didn't produce any core files. The system type in this case was linux-x86_64-gfx90a.

Reminders:

Checklist: