Skip to content

fix: Add missing replacement of VGFMAFLAG in configure#1171

Closed
rwolke wants to merge 1 commit intomadler:developfrom
rwolke:rwolke-missing-vgfmaflag
Closed

fix: Add missing replacement of VGFMAFLAG in configure#1171
rwolke wants to merge 1 commit intomadler:developfrom
rwolke:rwolke-missing-vgfmaflag

Conversation

@rwolke
Copy link
Copy Markdown
Contributor

@rwolke rwolke commented Feb 18, 2026

The variable VGFMAFLAG is set in the configure script, but will not be exported to the Makefile. Therefore, builds on s390x will fail.

cc -O2 -fomit-frame-pointer -pipe  -fPIC -D_LARGEFILE64_SOURCE=1 -DHAVE_S390X_VX  -c -o gzwrite.o gzwrite.c
cc -O2 -fomit-frame-pointer -pipe  -fPIC -D_LARGEFILE64_SOURCE=1 -DHAVE_S390X_VX   -c -o crc32_vx.o contrib/crc32vx/crc32_vx.c
In file included from contrib/crc32vx/crc32_vx.c:20:
contrib/crc32vx/crc32_vx.c: In function 'crc32_le_vgfm_16':
contrib/crc32vx/crc32_vx.c:79:17: error: '__builtin_s390_vec_insert' requires '-mvx'
   79 |     v0 = (uv2di)vec_insert(crc, (uv4si)v0, 3);
      |                 ^~~~~~~~~~
contrib/crc32vx/crc32_vx.c:82:16: error: '__builtin_s390_vec_perm' requires '-mvx'
...

This patch adds the missing replacements.

@madler
Copy link
Copy Markdown
Owner

madler commented Mar 24, 2026

Applied.

@madler madler closed this Mar 24, 2026
@alexfmpe
Copy link
Copy Markdown

Er, where?

@madler
Copy link
Copy Markdown
Owner

madler commented Mar 24, 2026

Locally. Not pushed yet.

giuseppe added a commit to giuseppe/crun that referenced this pull request Mar 24, 2026
zlib's configure script detects s390x vector extensions but fails
to export VGFMAFLAG, causing the build to fail with
'__builtin_s390_vec_*' requires '-mvx'.

Apply the upstream fix (madler/zlib#1171) as a patch, gated behind
an overlay that only activates for s390x builds.

Upstream: madler/zlib#1200
Fix: NixOS/nixpkgs#502917

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
giuseppe added a commit to giuseppe/crun that referenced this pull request Mar 24, 2026
zlib's configure script detects s390x vector extensions but fails
to export VGFMAFLAG, causing the build to fail with
'__builtin_s390_vec_*' requires '-mvx'.

Apply the upstream fix (madler/zlib#1171) as a patch, gated behind
an overlay that only activates for s390x builds.

Upstream: madler/zlib#1200
Fix: NixOS/nixpkgs#502917

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
giuseppe added a commit to giuseppe/crun that referenced this pull request Mar 24, 2026
zlib's configure script detects s390x vector extensions but fails
to export VGFMAFLAG, causing the build to fail with
'__builtin_s390_vec_*' requires '-mvx'.

Apply the upstream fix (madler/zlib#1171) as a patch, gated behind
an overlay that only activates for s390x builds.

Upstream: madler/zlib#1200
Fix: NixOS/nixpkgs#502917

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
giuseppe added a commit to giuseppe/crun that referenced this pull request Mar 25, 2026
zlib's configure script detects s390x vector extensions but fails
to export VGFMAFLAG, causing the build to fail with
'__builtin_s390_vec_*' requires '-mvx'.

Apply the upstream fix (madler/zlib#1171) as a patch, gated behind
an overlay that only activates for s390x builds.

Upstream: madler/zlib#1200
Fix: NixOS/nixpkgs#502917

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
giuseppe added a commit to giuseppe/crun that referenced this pull request Mar 25, 2026
zlib's configure script detects s390x vector extensions but fails
to export VGFMAFLAG, causing the build to fail with
'__builtin_s390_vec_*' requires '-mvx'.

Apply the upstream fix (madler/zlib#1171) as a patch, gated behind
an overlay that only activates for s390x builds.

Upstream: madler/zlib#1200
Fix: NixOS/nixpkgs#502917

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Mar 30, 2026
Bug: madler/zlib#1200
See-also: madler/zlib#1171
Signed-off-by: Andreas K. Hüttel <dilfridge@gentoo.org>
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.

3 participants