Merge ~igorluppi/ubuntu/+source/ggml:igorluppi/ggml-0.9.5-3ubuntu1 into ubuntu/+source/ggml:ubuntu/devel

Proposed by Igor Luppi
Status: Needs review
Proposed branch: ~igorluppi/ubuntu/+source/ggml:igorluppi/ggml-0.9.5-3ubuntu1
Merge into: ubuntu/+source/ggml:ubuntu/devel
Diff against target: 84 lines (+25/-9)
5 files modified
debian/changelog (+16/-0)
debian/control (+3/-3)
debian/copyright (+3/-3)
debian/not-installed.cuda (+2/-2)
debian/rules (+1/-1)
Reviewer Review Type Date Requested Status
Andreas Hasenack Needs Information
Ubuntu Sponsors Pending
git-ubuntu import Pending
Review via email: mp+499208@code.launchpad.net

Description of the change

Fix ppc64el build and check ggml against new hipcc (7.1.0) from proposed

To post a comment you must log in.
Revision history for this message
Igor Luppi (igorluppi) wrote :
Revision history for this message
Igor Luppi (igorluppi) wrote :

Bug #2138906: This is not affecting ggml, it is not using hipcc for ppc64el.
-> tested in ppa with -proposed hipcc from src:rocm-llvm v7.1.0

Revision history for this message
Igor Luppi (igorluppi) wrote :
Revision history for this message
Igor Luppi (igorluppi) wrote :

It appears that ubuntu/devel is 0.9.5-2 while my branch was based from 0.9.5-3.

Revision history for this message
Igor Luppi (igorluppi) wrote :

Now 0.9.5-3 (patches unapplied) is in Ubuntu/devel after a new importer trigger.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

I think you still need to rebase, there are things in the diff that come from 0.9.5-3.

Revision history for this message
Igor Luppi (igorluppi) wrote :

Thanks for the review, Andreas!

Actually, it is already rebased. That diff is only showing on the Launchpad webpage because, by the time the merge proposal was opened, the ubuntu/devel branch hadn't imported 0.9.5-3 yet, but now it has.

We can verify that locally with git diff. It's probably a refresh bug in the web UI.

$ git diff ubuntu/devel -- debian/changelog
diff --git a/debian/changelog b/debian/changelog
index 24ef2f3..19b9939 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+ggml (0.9.5-3ubuntu1) resolute; urgency=medium
+
+ * d/control: update maintainer field
+ * d/rules: Disable GGML_CPU_ALL_VARIANTS on ppc64el to avoid illegal instructions
+
+ -- Igor Luppi <email address hidden> Wed, 14 Jan 2026 17:08:13 -0300
+
 ggml (0.9.5-3) unstable; urgency=medium

   * CUDA: Adapt not-installed list to new library locations

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

So the HIP backend was dropped for ppc64el in 0.9.4-4:
ggml (0.9.4-4) unstable; urgency=medium

  [ Christian Kastner ]
  * Temporarily disable the HIP backend in unstable, until the ROCm stack has
    stabilized again
  * Introduce a build profile for single AMD GPU target, to enable reducing
    Salsa load

  [ Cordell Bloor ]
  * Drop HIP backend for ppc64el
  * Use default CMAKE_HIP_COMPILER as detected by CMake

 -- Christian Kastner <email address hidden> Wed, 03 Dec 2025 22:49:02 +0100

Ubuntu still has it in resolute-proposed, though, for some reason:
ubuntu:
 libggml0-backend-hip | 0.9.4-3 | resolute-proposed/universe | ppc64el
 libggml0-backend-hip | 0.9.5-3 | resolute-proposed/universe | amd64, amd64v3, arm64

debian:
 libggml0-backend-hip | 0.9.5-3 | unstable | amd64, arm64
 libggml0-backend-hip | 0.9.5-4+hip~exp1 | experimental | amd64, arm64

Excuses says:
ggml (0.0~git20250712.d62df60-1 to 0.9.5-3)
Migration status for ggml (0.0~git20250712.d62df60-1 to 0.9.5-3): BLOCKED: Maybe temporary, maybe blocked but Britney is missing information (check below)
Issues preventing migration:
missing build on ppc64el: libggml0-backend-hip (from 0.9.4-3); libggml-dev, libggml0, libggml0-backend-blas, libggml0-backend-vulkan (from 0.9.4-6)
arch:ppc64el not built yet, autopkgtest delayed there
Additional info:
14 days old
Also see bug 2115706 last updated on Tue Jul 1 05:39:03 2025

Acking that libggml0-backend-hip is missing. But that's because in 0.9.5-3 it failed to build. Actually, a test failed:
test 1
    Start 1: test-backend-ops

1: Test command: /<<PKGBUILDDIR>>/obj-powerpc64le-linux-gnu/bin/test-backend-ops
1: Working Directory: /<<PKGBUILDDIR>>/obj-powerpc64le-linux-gnu/tests
1: Environment variables:
1: LLVM_PROFILE_FILE=test-backend-ops.profraw
1: Test timeout computed to be: 10000000
1: ggml_backend_load_best: search path /usr/lib/powerpc64le-linux-gnu/ggml/backends0 does not exist
1: load_backend: loaded BLAS backend from /<<PKGBUILDDIR>>/obj-powerpc64le-linux-gnu/bin/libggml-blas.so
1: ggml_backend_load_best: search path /usr/lib/powerpc64le-linux-gnu/ggml/backends0 does not exist

Because something backends0 doesn't exist anymore. Did 0.9.5-3 pass in debian?

review: Needs Information
Revision history for this message
Igor Luppi (igorluppi) wrote :

Unmerged commits

5784801... by Igor Luppi

d/changelog: add entry for resolute build

Add the changelog entry to the package on Ubuntu Resolute

385e72e... by Igor Luppi

d/rules: Disable GGML_CPU_ALL_VARIANTS on ppc64el to avoid illegal instructions

8c6a4d8... by Igor Luppi

d/control: update maintainer field

Update maintainer for ubuntu archive.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 04d1c4a..19b9939 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,19 @@
6+ggml (0.9.5-3ubuntu1) resolute; urgency=medium
7+
8+ * d/control: update maintainer field
9+ * d/rules: Disable GGML_CPU_ALL_VARIANTS on ppc64el to avoid illegal instructions
10+
11+ -- Igor Luppi <igor.luppi@canonical.com> Wed, 14 Jan 2026 17:08:13 -0300
12+
13+ggml (0.9.5-3) unstable; urgency=medium
14+
15+ * CUDA: Adapt not-installed list to new library locations
16+ * Bump Standards-Version to 4.7.3
17+ - Drop Priority: optional, it's the dpkg default since trixie
18+ * Bump copyrights
19+
20+ -- Christian Kastner <ckk@debian.org> Sun, 11 Jan 2026 16:46:07 +0100
21+
22 ggml (0.9.5-2) unstable; urgency=medium
23
24 * Temporarily revert riscv64 to minimal feature flags (Closes: #1124644)
25diff --git a/debian/control b/debian/control
26index 2afd4b9..2d01a39 100644
27--- a/debian/control
28+++ b/debian/control
29@@ -1,9 +1,9 @@
30 Source: ggml
31 Section: libs
32-Priority: optional
33-Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>
34+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
35+XSBC-Original-Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>
36 Uploaders: Christian Kastner <ckk@debian.org>
37-Standards-Version: 4.7.2
38+Standards-Version: 4.7.3
39 Vcs-Browser: https://salsa.debian.org/deeplearning-team/ggml
40 Vcs-Git: https://salsa.debian.org/deeplearning-team/ggml.git
41 Homepage: https://github.com/ggml-org/ggml
42diff --git a/debian/copyright b/debian/copyright
43index 2d57fa4..e2003bb 100644
44--- a/debian/copyright
45+++ b/debian/copyright
46@@ -13,9 +13,9 @@ Copyright: 2023-2024, The ggml authors
47 License: Expat
48
49 Files: debian/*
50-Copyright: 2025, Christian Kastner <ckk@debian.org>
51- 2025, Mathieu Baudier <mbaudier@argeo.org>
52- 2025, Mo Zhou <lumin@debian.org>
53+Copyright: 2025-2026, Christian Kastner <ckk@debian.org>
54+ 2025-2026, Mathieu Baudier <mbaudier@argeo.org>
55+ 2025, Mo Zhou <lumin@debian.org>
56 License: Expat
57
58 Files: examples/stb_image.h
59diff --git a/debian/not-installed.cuda b/debian/not-installed.cuda
60index c8d2d4f..ab36f6d 100644
61--- a/debian/not-installed.cuda
62+++ b/debian/not-installed.cuda
63@@ -1,6 +1,6 @@
64 usr/bin/*
65 usr/include/*
66 usr/lib/*/cmake
67-usr/lib/*/ggml/libggml-base.*
68-usr/lib/*/ggml/libggml.*
69+usr/lib/*/libggml-base.*
70+usr/lib/*/libggml.*
71 /usr/share/*
72diff --git a/debian/rules b/debian/rules
73index c9b656d..5c6ad25 100755
74--- a/debian/rules
75+++ b/debian/rules
76@@ -26,7 +26,7 @@ CMAKE_OPTS += -DGGML_BLAS=ON \
77
78 ## CPU ##
79 # Build all CPU variants where supported
80-ifneq ($(filter $(DEB_HOST_ARCH),amd64 arm64 ppc64el),)
81+ifneq ($(filter $(DEB_HOST_ARCH),amd64 arm64),)
82 CMAKE_OPTS += -DGGML_CPU_ALL_VARIANTS=ON
83 # Disable all features where all CPU variants are not supported
84 else ifeq ($(DEB_HOST_ARCH),riscv64)

Subscribers

People subscribed via source and target branches

to all changes: