Skip to content

Upgrade llama.cpp from b10197 to b10217 - #373

Merged
bernardladenthin merged 6 commits into
mainfrom
claude/update-b10217-5glzlx
Aug 1, 2026
Merged

Upgrade llama.cpp from b10197 to b10217#373
bernardladenthin merged 6 commits into
mainfrom
claude/update-b10217-5glzlx

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Bumps the pinned llama.cpp version from b10197 to b10217 across all configuration files and documentation
  • Updates all version references in CMakeLists.txt, Java constants, README, and CLAUDE.md
  • Re-verifies all 7 patches against the new upstream version in four incremental chunks (b10197→b10204→b10207→b10216→b10217)
  • Updates patch documentation to reflect that PR #22393 (slot_prompt_similarity getter/setter) was rejected upstream and is now carried indefinitely

Changes

Version pin updates:

  • llama/CMakeLists.txt: GIT_TAG and LLAMA_TAG updated to b10217
  • llama/src/main/java/net/ladenthin/llama/value/LlamaCppVersion.java: LLAMA_CPP_VERSION constant updated
  • README.md: Badge link updated to b10217
  • CLAUDE.md: Documentation updated with new version and detailed upstream verification notes

Patch verification:
All 7 patches re-verified and applied cleanly against b10217:

  • 0001-win32-arg-parse-embed-guard.patch — No changes to patch-target files in this range
  • 0002-server-preserve-caller-load-progress-callback.patch — No changes to patch-target files in this range
  • 0003-pr22393-server-add-slot-prompt-similarity-getter-setter.patch — Updated header comment to document that upstream PR #22393 was closed without merging; patch is now permanent
  • 0006-server-embed-native-server-jni.patch — No changes to patch-target files in this range
  • 0007-server-attach-http-frontend.patch — No changes to patch-target files in this range
  • 0008-server-models-worker-cmd-override.patch — No changes to patch-target files in this range

Upstream changes in b10197→b10217 range:

  • b10197→b10204: Server embedded-batch support (additive server_batch::add() overload and server_slot::inp_embd member for TTS-style models)
  • b10204→b10207: K/V cache type validation and quantized-V flash-attention auto-enabling
  • b10207→b10216: Speculative-replay accepted-token accounting and MTP tensor loading support
  • b10216→b10217: DeepSeek-V4 tool-call-in-thinking parsing improvements

No project-source changes required; all upstream modifications are internal or additive with no impact on public APIs or JNI signatures.

Test plan

  • All 7 patches re-verified against clean b10217 checkout with fail-loud PATCH_COMMAND
  • Full local verification: fresh configure + cmake --build (libjllama.so + jllama_test compile and link, -O3, no undefined references) + ctest 485/485 passing
  • NativeLibraryLoadSmokeTest (JNI FindClass/build-info cross-check) green
  • OuteTTS generator anchors verified (tts.cpp @ b10217 unchanged)

Per-platform build + ctest confirmation deferred to the CI pipeline.

Related issues / PRs

Upstream: ggml-org/llama.cpp#22393 (rejected; now carried permanently)

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes

https://claude.ai/code/session_01CV2A6f4ezHubpZgsqTDkj4

claude added 6 commits August 1, 2026 08:55
7 commits: server support for inp embd to generate next token
(server-context.cpp, additive), SYCL q2 mul_mat + RMS_NORM/MUL fusion +
dev2dev memcpy, ggml-webgpu flash-attn quantized-KV tuning, Vulkan
quantized-concat support, and an mtmd Lanczos image-resize method. All
7 patches apply cleanly (no overlap with the server-context.cpp hunks
they touch). Full local build + ctest (485/485) + NativeLibraryLoadSmokeTest
verified.
3 commits: llama_init_from_model gains a K/V cache type-matching check
for DeepSeek-V4/MLA models and folds the quantized-V-cache validation
into a combined block that auto-enables flash_attn (internal validation
logic, no signature change), plus SYCL cpy type coverage and a
ggml-zendnn mul_mat_id refactor. All 7 patches apply cleanly. Full local
build + ctest (485/485) + NativeLibraryLoadSmokeTest verified.
9 commits: include/llama.h gains a purely-additive load_mtp field;
server-context.cpp adds speculative-replay accepted-token accounting
(server_slot::spec_is_replay, corrects draft-acceptance counters on
partial-acceptance replay) -- clear of the 0002/0003 patch hunks.
Plus CUDA MMQ tile-loading refactors, SYCL oneMKL XMX flash attention,
Vulkan POOL_1D op, and mtmd n_embd_head. All 7 patches apply cleanly.
Full local build + ctest (485/485) + NativeLibraryLoadSmokeTest verified.
1 commit: DeepSeek-V4 tool-call-in-thinking chat-format parsing rework
(common/chat.cpp internals, no common/chat.h signature change). No
patches or project source needed changes. This is the final chunk of
the b10197 -> b10217 bump (via b10204, b10207, b10216). Full local
build + ctest (485/485) + NativeLibraryLoadSmokeTest + javadoc verified.
Upstream (ngxson) closed ggml-org/llama.cpp#22393 without merging,
reasoning that slot_prompt_similarity is internal single-use-case state
that shouldn't be public API. Update the patch 0003 header and its
CLAUDE.md table row so both stop framing the patch as "pending merge" --
it is now carried permanently and will never be droppable via a version
bump. No functional change; verified the patch still applies cleanly
against the current b10217 pin.
Keep each patch's header self-contained instead of naming sibling
patch numbers, which drift as patches are added/dropped over time.
@bernardladenthin
bernardladenthin merged commit 1b8e526 into main Aug 1, 2026
10 of 14 checks passed
@bernardladenthin
bernardladenthin deleted the claude/update-b10217-5glzlx branch August 1, 2026 09:49
@claude

claude Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Summary: llama.cpp b10197 to b10217 Upgrade

This merged PR demonstrates excellent quality. All version strings consistently updated across CMakeLists.txt, LlamaCppVersion.java, README.md, and CLAUDE.md. All 7 patches re-verified and applied cleanly through chunked verification steps (b10197 to b10204 to b10207 to b10216 to b10217). Priority-8 headers unchanged. Patch 0003 correctly documented as permanently carried after upstream rejection. Test coverage complete: 485/485 C++ tests passing, NativeLibraryLoadSmokeTest green. No breaking API changes to JNI surface. OuteTTS generator anchors verified. No issues found - all changes correct and complete.

@sonarqubecloud

sonarqubecloud Bot commented Aug 1, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants