Regression timing fix (#1361)

* Removed testbed initialization on standalone tests
* .jenkins renabled all tests

[ROCm/rccl commit: 7fb9189760]
This commit is contained in:
akolliasAMD
2024-10-03 10:41:26 -06:00
committed by GitHub
parent 974c13cd62
commit 9c4ac4cae5
2 changed files with 3 additions and 6 deletions
-3
View File
@@ -17,9 +17,6 @@ def runCompileCommand(platform, project, jobName)
def runTestCommand (platform, project, gfilter, envars)
{
// Temporarily disable Standalone.RegressionTiming for RCCL CI
gfilter = "*:-Standalone.Regression*"
String sudo = auxiliary.sudo(platform.jenkinsLabel)
def command = """#!/usr/bin/env bash
+3 -3
View File
@@ -157,7 +157,6 @@ namespace RcclUnitTesting
* ******************************************************************************************/
TEST(Standalone, RegressionTiming)
{
TestBed testBed;
// timing
using namespace std::chrono;
using Clock = std::chrono::high_resolution_clock;
@@ -169,7 +168,8 @@ namespace RcclUnitTesting
if (numGpus < 2) {
GTEST_SKIP() << "This test requires at least 2 devices.";
}
hipDeviceProp_t devProp;
HIPCALL(hipGetDeviceProperties(&devProp, 0));
// Initialize RCCL
int numRanks = 2;
std::vector<ncclComm_t> comms(numRanks);
@@ -180,7 +180,7 @@ namespace RcclUnitTesting
for (auto p : protocolList)
{
usElapsed = 0;
if(testBed.ev.isGfx12) {
if(strncmp("gfx12",devProp.gcnArchName,5) == 0) {
setenv("NCCL_PROTO", "Simple", 1);
} else {
setenv("NCCL_PROTO", p, 1);