Skip to content

Commit 0c0cee5

Browse files
authored
Revert "Arm backend: Enable pytest VKML op unit testing in ciflow/trunk" (#15841)
Reverts #14109 This caused timeout on arm ops unit tests cc @freddan80 @per @oscarandersson8218 @digantdesai
1 parent b8ed35d commit 0c0cee5

File tree

5 files changed

+3
-30
lines changed

5 files changed

+3
-30
lines changed

.ci/scripts/test_backend.sh

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -57,25 +57,15 @@ if [[ "$FLOW" == *vulkan* ]]; then
5757
fi
5858

5959
if [[ "$FLOW" == *arm* ]]; then
60-
6160
# Setup ARM deps.
62-
if [[ "$FLOW" == *vgf* ]]; then
63-
.ci/scripts/setup-arm-baremetal-tools.sh --enable-mlsdk-deps --install-mlsdk-deps-with-pip
64-
else
65-
.ci/scripts/setup-arm-baremetal-tools.sh
66-
fi
61+
.ci/scripts/setup-arm-baremetal-tools.sh
6762
source examples/arm/ethos-u-scratch/setup_path.sh
6863

6964
if [[ "$FLOW" == *ethos_u* ]]; then
7065
# Prepare a test runner binary that can run on the Corstone-3x0 FVPs
7166
backends/arm/scripts/build_executorch.sh
7267
backends/arm/test/setup_testing.sh
7368
fi
74-
75-
if [[ "$FLOW" == *vgf* ]]; then
76-
# Prepare a test runner binary for VKML runtime
77-
backends/arm/test/setup_testing_vkml.sh
78-
fi
7969
fi
8070

8171
if [[ $IS_MACOS -eq 1 ]]; then

.github/workflows/test-backend-arm.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
uses: ./.github/workflows/_test_backend.yml
2727
with:
2828
backend: arm
29-
flows: '["arm_tosa_fp", "arm_tosa_int", "arm_ethos_u55", "arm_ethos_u85", "arm_vgf_fp", "arm_vgf_int"]'
29+
flows: '["arm_tosa_fp", "arm_tosa_int", "arm_ethos_u55", "arm_ethos_u85"]'
3030
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
3131
timeout: 120
3232
run-linux: true

.github/workflows/trunk.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,6 @@ jobs:
284284
include:
285285
- test_arm_baremetal: test_pytest_ops_ethosu_fvp
286286
- test_arm_baremetal: test_pytest_models_ethosu_fvp
287-
- test_arm_baremetal: test_pytest_ops_vkml
288287
- test_arm_baremetal: test_run_ethosu_fvp
289288
- test_arm_baremetal: test_models_tosa
290289
- test_arm_baremetal: test_models_ethos-u55
@@ -307,7 +306,7 @@ jobs:
307306
source .ci/scripts/utils.sh
308307
install_executorch "--use-pt-pinned-commit"
309308
310-
.ci/scripts/setup-arm-baremetal-tools.sh --enable-mlsdk-deps --install-mlsdk-deps-with-pip
309+
.ci/scripts/setup-arm-baremetal-tools.sh
311310
312311
# Increase number of files user can monitor to bypass buck failures.
313312
# Hopefully this is high enough for this setup.

backends/test/suite/flow.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -132,17 +132,13 @@ def all_flows() -> dict[str, TestFlow]:
132132
ARM_ETHOS_U85_FLOW,
133133
ARM_TOSA_FP_FLOW,
134134
ARM_TOSA_INT_FLOW,
135-
ARM_VGF_FP_FLOW,
136-
ARM_VGF_INT_FLOW,
137135
)
138136

139137
flows += [
140138
ARM_TOSA_FP_FLOW,
141139
ARM_TOSA_INT_FLOW,
142140
ARM_ETHOS_U55_FLOW,
143141
ARM_ETHOS_U85_FLOW,
144-
ARM_VGF_FP_FLOW,
145-
ARM_VGF_INT_FLOW,
146142
]
147143
except Exception as e:
148144
logger.info(f"Skipping ARM flow registration: {e}")

backends/test/suite/flows/arm.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -78,15 +78,3 @@ def create_quantize_stage() -> Quantize:
7878
lambda: common.get_u85_compile_spec(),
7979
quantize=True,
8080
)
81-
ARM_VGF_FP_FLOW = _create_arm_flow(
82-
"arm_vgf_fp",
83-
lambda: common.get_vgf_compile_spec(tosa_spec="TOSA-1.0+FP"),
84-
quantize=False,
85-
use_portable_ops=False,
86-
)
87-
ARM_VGF_INT_FLOW = _create_arm_flow(
88-
"arm_vgf_int",
89-
lambda: common.get_vgf_compile_spec(tosa_spec="TOSA-1.0+INT"),
90-
quantize=True,
91-
use_portable_ops=False,
92-
)

0 commit comments

Comments
 (0)