Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drop support for GCC < 6.3 in Sage 9.7 #33316

Closed
mkoeppe opened this issue Feb 8, 2022 · 44 comments
Closed

Drop support for GCC < 6.3 in Sage 9.7 #33316

mkoeppe opened this issue Feb 8, 2022 · 44 comments
Labels
c: build: configure p: major / 3 t: enhancement
Milestone
sage-9.7

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Feb 8, 2022

(see Meta-ticket #32074: Remove support for outdated distros)

This ticket can be used as a dependency for upgrade tickets that impose stricter compiler requirements.

CC: @dimpase @kliem

Component: build: configure

Author: Matthias Koeppe

Branch/Commit: 7c0072e

Reviewer: Jonathan Kliem

Issue created by migration from https://trac.sagemath.org/ticket/33316
@mkoeppe mkoeppe added this to the sage-9.7 milestone Feb 8, 2022
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 8, 2022

Dependencies: #33187

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 8, 2022

Branch: u/mkoeppe/drop_support_for_gcc___6_3_in_sage_9_7

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 8, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

bf96a8aREADME.md, src/doc/en/installation/source.rst: Update info on GCC versions

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 8, 2022

Commit: bf96a8a

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 8, 2022

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 9, 2022

Work Issues: Update platforms tested on GH Actions, best on top of #33062

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 20, 2022

Changed commit from bf96a8a to 4e8b2c2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 20, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

1124dedbuild/pkgs/gcc/spkg-configure.m4: Bump minimum version to 6.3
4e8b2c2README.md, src/doc/en/installation/source.rst: Update info on GCC versions

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 20, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

9b388ea.github/workflows/tox*.yml: Update platforms for drop of GCC < 6.3

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 20, 2022

Changed commit from 4e8b2c2 to 9b388ea

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 20, 2022

Changed work issues from Update platforms tested on GH Actions, best on top of #33062 to none

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 20, 2022

Changed dependencies from #33187 to none

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 21, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

e7ff2a8build/pkgs/gcc/spkg-configure.m4: Fixup

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 21, 2022

Changed commit from 9b388ea to e7ff2a8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 21, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

0713f53.github/workflows/tox*.yml: Replace ubuntu-xenial by ubuntu-xenial-toolchain-gcc_9

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 21, 2022

Changed commit from e7ff2a8 to 0713f53

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 29, 2022

Reviewer: https://github.com/mkoeppe/sage/actions/runs/2242737566, https://github.com/mkoeppe/sage/actions/runs/2242737575

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 29, 2022

comment:13

On ubuntu-trusty-toolchain-gcc_9-minimal (https://github.com/mkoeppe/sage/runs/6221011844) and -standard, nauty, ecm, ntl fail with
Error: operand size mismatch for vmovdqa64'orError: operand size mismatch for vmovdqu64'; and fflas_ffpack fails with many different operand size mismatch errors.

@kliem
Copy link
Contributor

kliem commented Apr 30, 2022

comment:14

I'm guessing the problem is in https://github.com/linbox-team/fflas-ffpack/blob/b51626baf2715c8b8bcb0c334856b204f9a4a901/fflas-ffpack/fflas/fflas_simd/simd512_double.inl

They are calling _mm512_and_pd with const __mm512d instead of just __m512d. The problem should mostly go away, because stuff is inlined, but it might fail with g++ 9.

It seems to work with g++ 11.3.0 for me. At least I cannot reproduce the bug, even though I have available the avx512... instructions. But then again, it works for the other workflows, so this doesn't seem to be a problem in other cases. Just in the toolchain-gcc_9-minimal workflow.

Can we configure fflas_ffpack with --disable-avx512dq in this case?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 1, 2022

comment:15

I'll try first if installing binutils-2.26 (and setting PATH="/usr/lib/binutils-2.26/bin:$PATH") helps.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 2, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

d650177Merge tag '9.6.rc2' into t/33316/drop_support_for_gcc___6_3_in_sage_9_7
a0e037fbuild/bin/write-dockerfile.sh [debian]: Handle EXTRA_PATH, EXTRA_SYSTEM_PACKAGES
dae2d08tox.ini (ubuntu-trusty-toolchain): Use binutils-2.26

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 2, 2022

Changed commit from 0713f53 to dae2d08

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 2, 2022

Changed reviewer from https://github.com/mkoeppe/sage/actions/runs/2242737566, https://github.com/mkoeppe/sage/actions/runs/2242737575 to https://github.com/mkoeppe/sage/actions/runs/2260846479

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 3, 2022

comment:18

Replying to @mkoeppe:

I'll try first if installing binutils-2.26 (and setting PATH="/usr/lib/binutils-2.26/bin:$PATH") helps.

It seems to work. Ready for review

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 3, 2022

Changed commit from dae2d08 to 33ca131

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 22, 2022

comment:26

See #32074 - we're dropping support for Debian 9.

@vbraun
Copy link
Member

vbraun commented May 22, 2022

comment:27

Debian 9 is still LTS

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 22, 2022

comment:28

Replying to @mkoeppe:

See #32074 - we're dropping support for Debian 9.

Sorry, I was off by one. Debian 9 is fine. What's the problem on your buildbot?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 22, 2022

comment:29

so needs_work but not on the ticket

@dimpase
Copy link
Member

dimpase commented May 22, 2022

comment:30

Replying to @vbraun:

Debian 9 is still LTS

but only till 01.07.2022, so we should be dropping it in 9.7, I guess.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 24, 2022

Dependencies: #33888

@vbraun
Copy link
Member

vbraun commented Jun 13, 2022

Changed dependencies from #33888 to none

@vbraun
Copy link
Member

vbraun commented Jun 13, 2022

comment:33

Merge failure on top of:

08609a40ad Trac #33829: sage.graphs: Do not use SAGE_TMP in doctests

eca52b2 Trac #33922: Add missing title to conf.py

e9bfcd2 Trac #33980: codespell in graphs: fix a few typos

28f70e0 Trac #31153: Fix latex of elements of multivariate polynomial quotient ring

b36c11c Trac #33983: sage_setup: Add missing dependency

6edac51 Trac #33970: Remove a period sneaked in the documentation website

6d60bf3 Trac #33977: fix pycodestyle E306 in geometry and schemes

edd7a74 Trac #33976: some pycodestyle fixes in pxd and pxi files

b0973af Trac #33975: get rid of commented old-style prints

b3d6a0b Trac #33956: some better .join with iterator inside

a33f2df Trac #33949: get rid of have_ring option in singular interface

ad1c3ac Trac #33946: fix pycodestyle E306 in rings and algebras

4942ca9 Trac #33945: fix pycodestyle E306 outside of algebras,rings,graphs,schemes,categories

8dad9ce Trac #30683: Nu Tamari

346b3b0 Trac #33968: Replace $...$ in docstrings by ...

f793558 Trac #33967: sage -t: Print git commit hash and SAGE_LOCAL, SAGE_VENV

0e48fec Trac #33963: wrong sign for symbolic Legendre polynomial at 0

34d562e Trac #33962: wrong sign for value of Legendre polynomial at 0

bd33f87 Trac #33961: compute square roots modulo powers of two in polynomial time

16af958 Trac #33940: enhance maple interface

8d921a4 Trac #33872: Upgrade memory allocator to 0.1.3

29499f8 Trac #33871: Don't run singular in spkg-configure.m4 when cross compiling

4cc978b Trac #33823: sage -t --optional='sage,!FEATURE'

6b6f0cf Trac #33776: move method distance_graph to graph.py

65e866a Trac #23443: More Schubert polynomial shenanigans

a6e696e Updated SageMath version to 9.7.beta2

merge was not clean: conflicts in .github/workflows/tox.yml

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 13, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

7c0072eMerge tag '9.7.beta2' into t/33316/drop_support_for_gcc___6_3_in_sage_9_7

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 13, 2022

Changed commit from 33ca131 to 7c0072e

@vbraun
Copy link
Member

vbraun commented Jun 19, 2022

Changed branch from u/mkoeppe/drop_support_for_gcc___6_3_in_sage_9_7 to 7c0072e

@vbraun vbraun closed this as completed in 8907ed5 Jun 19, 2022
@dimpase dimpase mentioned this issue Jan 6, 2023
Open
@ClementPernet ClementPernet mentioned this issue Dec 31, 2022
Closed
This was referenced May 22, 2022
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: build: configure p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-9.7
Development

No branches or pull requests
4 participants
@vbraun @dimpase @mkoeppe @kliem