Skip to content

ci: update device matrix for Snapdragon jobs by removing SM8750 - #30620

Merged
max-krasnyansky merged 2 commits into
ggml-org:masterfrom
qualcomm:zack/ci-update
Oct 16, 2026
Merged

max-krasnyansky merged 2 commits into
ggml-org:masterfrom
qualcomm:zack/ci-update

Conversation

@zhiyuan8

@zhiyuan8 zhiyuan8 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Update device matrix for Snapdragon jobs by removing SM8750, the current SM8850 test is enough to cover CI test for snapdragon mobile.

Additional information

SM8750 repeatedly failed with an Appium/adb session-setup timeout.

CI passes: https://github.com/qualcomm/llama.cpp/actions/runs/38819844027

Requirements

@zhiyuan8
zhiyuan8 requested a review from a team as a code owner October 15, 2026 18:31
@github-actions github-actions Bot added the devops improvements to build systems and github actions label Oct 15, 2026
@ggml-gh-bot

ggml-gh-bot Bot commented Oct 15, 2026

Copy link
Copy Markdown

Hi @zhiyuan8, thanks for your contribution!

Per our contribution guidelines, the automated PR checker found the following issue(s) that need your attention:

  • Maintainers cannot push to this PR: Please enable Allow edits by maintainers. If this PR comes from an organization-owned fork, that option is not available on GitHub; please re-open the PR from a fork owned by your personal account.

Please note that maintainers reserve the right to make final decisions on PRs. If you believe there is a mistake, please comment below.

@ggml-gh-bot ggml-gh-bot Bot added the draft PR will be changed to draft by github-actions bot label Oct 15, 2026
@github-actions
github-actions Bot marked this pull request as draft October 15, 2026 18:38
@github-actions github-actions Bot removed the draft PR will be changed to draft by github-actions bot label Oct 15, 2026
@CISC

CISC commented Oct 15, 2026

Copy link
Copy Markdown
Member

Appreciated, but I don't think any of them succeed at the moment?
https://github.com/ggml-org/llama.cpp/actions/runs/38416476431

@zhiyuan8

Copy link
Copy Markdown
Contributor Author

Appreciated, but I don't think any of them succeed at the moment? https://github.com/ggml-org/llama.cpp/actions/runs/38416476431

The above run from @max-krasnyansky was 3 days ago, this is the most recent CI run with my code change, and it is working, with two changes:

  1. OpenCL GPU kernel fix from wanghqc
  2. remove SM8750 from CI test, due to some transient error issues with QDC.
image

@CISC

CISC commented Oct 16, 2026

Copy link
Copy Markdown
Member

So is this ready for review?

@max-krasnyansky
max-krasnyansky marked this pull request as ready for review October 16, 2026 22:21
@max-krasnyansky

Copy link
Copy Markdown
Member

So is this ready for review?

Yep. I think Zack @zhiyuan8 just ran into his PR limit. I marked it as ready and aproved the pipeline

@max-krasnyansky
max-krasnyansky merged commit 6420cf5 into ggml-org:master Oct 16, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants