Skip to content

Commit edea888

Browse files
Merge pull request #1397 from mrniranjan/skip_llc_vm
OCPBUGS-62153: E2E: skip SMT disabled test when L3 cache spans entire NUMA node
2 parents ccb071f + c008682 commit edea888

File tree

1 file changed

+12
-0
lines changed
  • test/e2e/performanceprofile/functests/13_llc

1 file changed

+12
-0
lines changed

test/e2e/performanceprofile/functests/13_llc/llc.go

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -782,6 +782,18 @@ var _ = Describe("[rfe_id:77446] LLC-aware cpu pinning", Label(string(label.Open
782782
Expect(err).ToNot(HaveOccurred(), "Unable to fetch numa nodes")
783783
coresiblings, err := nodes.GetCoreSiblings(ctx, &cnfnode)
784784
Expect(err).ToNot(HaveOccurred(), "Unable to get numa information from the node")
785+
getCCX = nodes.GetL3SharedCPUs(&cnfnode)
786+
ccx, err := getCCX(0)
787+
Expect(err).ToNot(HaveOccurred())
788+
L3CacheGroupSize = ccx.Size()
789+
// Get actual CPU ID's from Numa Node 0
790+
numaNode0Cpus := cpuset.New(numaInfo[0]...)
791+
792+
// Compare if L3 cache group CPUs match NUMA Node 0 CPUs
793+
if ccx.Equals(numaNode0Cpus) {
794+
Skip("This test requires systems where L3 cache is shared amount subset of cpus")
795+
}
796+
785797
nosmt = transformToNoSMT(coresiblings)
786798
if len(numaInfo) < 2 {
787799
Skip(fmt.Sprintf("This test need 2 Numa nodes. The number of numa nodes on node %s < 2", cnfnode.Name))

0 commit comments

Comments
 (0)