feat: upgrade llama.cpp from b10903 to b10905 - #433
Merged
Conversation
Two commits, five files, 8 KiB, none of it on this project's review surface.
Upstream #28102 tunes CUDA/HIP Flash Attention for gfx1201 (three ggml-cuda
files plus a test-backend-ops case) and #28708 keys upstream's own sanitizer CI
cache per matrix entry.
Zero files under common/, include/, tools/server/, tools/mtmd/ or src/, so
every row of the API-compatibility table is vacuously satisfied and the three
mechanical tools/server contract greps have no input. The ggml-cuda files reach
only the CUDA and HIP classifier jobs (gfx1201 is RDNA4, so the rocm-* ones are
where it matters), all build-only on GPU-less runners; test-backend-ops.cpp is
never compiled here (LLAMA_BUILD_TESTS=OFF for a FetchContent subproject) and
server-sanitize.yml is upstream's CI. The default JAR and every CPU classifier
are unaffected.
All ten patches apply untouched, and every patch-target file is byte-unchanged
in the range -- verified by diffing those paths explicitly rather than inferring
it from the aggregate.
First bump with four standing drop-checks rather than three, since 0013 joined
the set. All four run against the pristine tag, because the fail-loud applier
detects "does not apply" but never "upstream already fixed this":
* 0001 -- common_params_parse_main appears 0 times in b10905:common/arg.h and
the WIN32 override is still at common/arg.cpp:1282. Still required.
* 0010 -- b10905:tools/server/server-context.cpp:4554 still emits vocab_type
uncast. Still required.
* 0012 -- b10905:src/llama-model.cpp:1491 still carries a bare
splits[i] /= split_sum with no zero-sum guard. Still required.
* 0013 -- b10905 still leaves vxe_dot_acc / vxe_splat_granule / vxe_fold at
file scope (73/77/83) between the guarded blocks. Still required. Upstream
master at this point (a2878d30d) carries that file byte-identical, so the
defect is live there too.
Verified from a fresh configure: stamp head 16378d93f with all ten SHA-256
lines, build clean, ctest 537/537, extraction 138/57/15, nm -D 40 Java_* exports
and 0 mangled, NativeLibraryLoadSmokeTest 4/4 after a clean, mvn test 1755/0,
SpotBugs 0, spotless and javadoc clean.
0013 has no runnable guard beyond the s390x CI job, so the bump routine now
re-verifies it with the real cross toolchain: s390x-linux-gnu-g++ compiles the
applier's repack.cpp clean both with the job's own scalar flags and with
-mvx -mzvector -march=z15.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AnNYn8W1xuVxVJtyL34GyH
bernardladenthin
had a problem deploying
to
maven-central
September 11, 2026 11:06 — with
GitHub Actions
Failure
bernardladenthin
had a problem deploying
to
maven-central
September 11, 2026 11:06 — with
GitHub Actions
Failure
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Summary
Bumps the pinned llama.cpp version from
b10903tob10905across all four pin sites, and appends the upgrade range to the breaking-changes history.Two commits, five files, 8 KiB — none of it on this project's review surface.
gfx1201—ggml-cuda/fattn-common.cuh,fattn-mma-f16.cuh,fattn.cu, plus atests/test-backend-ops.cppcasegfx1201is RDNA4, so therocm-*ones are where it matters), all build-only on GPU-less runners. The test file is never compiled here — a FetchContent subproject setsLLAMA_BUILD_TESTS=OFF..github/workflows/server-sanitize.yml)Zero files under
common/,include/,tools/server/,tools/mtmd/orsrc/, so every row of the API-compatibility table is vacuously satisfied and the three mechanicaltools/server/contract greps have no input. Third bump running with that property. Far under the 100 KiB threshold, so no chunking question arises. The default JAR and every CPU classifier are unaffected.Patches: all ten apply untouched
Every patch-target file is byte-unchanged in the range — verified by diffing those paths explicitly (
common/arg.{cpp,h},common/peg-parser.cpp, all oftools/server/,src/llama-model.{cpp,h},tests/CMakeLists.txt,ggml/src/ggml-cpu/arch/s390/) rather than inferring it from the aggregate.This is the first bump with four standing drop-checks rather than three, since
0013joined the set. All four were run against the pristine tag, because the fail-loud applier detects "does not apply" but never "upstream already fixed this":0001common_params_parse_mainincommon/arg.hcommon/arg.cpp:1282→ still required0010{"vocab_type", meta.model_vocab_type}server-context.cpp:4554→ still required0012split_sumzero guard insrc/llama-model.cppsplits[i] /= split_sum;at line 1491 → still required0013vxe_dot_acc/vxe_splat_granule/vxe_foldstill at file scope (73/77/83) between the guards at 28–70 and 100–155 → still requiredOn
0013specifically: upstreammasterat the time of this bump (a2878d30d) carries that file byte-identical to b10905, so the defect is live there too. It was reported on #28667, the PR that introduced it.Test plan
Verified locally on Linux x86_64 from a fresh configure (build directory removed first, so the applier took its clean-tree path and wrote a fresh stamp):
16378d93f94012d4228c8c7683adce3f286aee5d(=b10905) with all ten SHA-256 lines; no "does not apply cleanly" abort.cmake --build --config Release, zero errors.ctest: 537/537.OAI_LAYERreader sweep re-ran against b10905's sources).0013re-verified with the real cross toolchain. It has no runnable guard beyond the s390x CI job — unlike0012(test_model_split.cpp) or0011(ContentOnlyParseUtf8) — so the bump routine now includes it:s390x-linux-gnu-g++compiles the applier'srepack.cppclean both with the job's own (scalar) flags and with-mvx -mzvector -march=z15. This is recorded in the history row so it does not stay tribal knowledge.nm -D: 40Java_*exports, 0 C++-mangled.NativeLibraryLoadSmokeTest— 4/4, 0 skipped, run with acleanfirst sonativeBuildInfoMatchesPinnedVersionConstantcompared the rebuilt binary against the new constant rather than a stale class.mvn test: 1755 run, 0 failures, 0 errors (269 skipped — the model-gated classes; no GGUF in this sandbox).spotless:checkclean (243 files);javadoc:jarBUILD SUCCESS.Related issues
None.
Checklist
mvn spotless:applyrun for Java changes) — no.javaformatting changes; the one Java file touched is a constant plus javadoc.README.mdbadge/link,CLAUDE.mdpinned-version line (plus three incidental version mentions: the WebUI local-build recipe, the sccache section, the FetchContent source-tree section), and a newb10903–b10905pair of rows indocs/history/llama-cpp-breaking-changes.md.Files changed (5)
llama/CMakeLists.txtGIT_TAG b10903→b10905llama/src/main/java/net/ladenthin/llama/value/LlamaCppVersion.javaLLAMA_CPP_VERSIONconstant + 3 javadoc mentionsREADME.mdCLAUDE.mddocs/history/llama-cpp-breaking-changes.mdCHANGELOG.mdis deliberately untouched: it records consumer-visible behaviour, and llama.cpp pin bumps are not recorded there.🤖 Generated with Claude Code
https://claude.ai/code/session_01AnNYn8W1xuVxVJtyL34GyH
Generated by Claude Code