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.

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

mpir build failures with some Haswell CPUs #25858

Closed
embray opened this issue Jul 13, 2018 · 12 comments
Closed

mpir build failures with some Haswell CPUs #25858

embray opened this issue Jul 13, 2018 · 12 comments

Comments

@embray
Copy link
Contributor

embray commented Jul 13, 2018

See wbhart/mpir#252

Upstream: Fixed upstream, but not in a stable release.

Component: packages: standard

Author: Erik Bray

Branch/Commit: 50c1e0a

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/25858

@embray embray added this to the sage-8.3 milestone Jul 13, 2018
@embray
Copy link
Contributor Author

embray commented Jul 13, 2018

Commit: 50c1e0a

@embray
Copy link
Contributor Author

embray commented Jul 13, 2018

New commits:

50c1e0aadd a hand-customized version of the patch from the upstream PR which only patches the configure script and the relevant .c sources

@embray
Copy link
Contributor Author

embray commented Jul 13, 2018

Author: Erik Bray

@embray
Copy link
Contributor Author

embray commented Jul 13, 2018

Branch: u/embray/ticket-25858

@tscrim
Copy link
Collaborator

tscrim commented Jul 13, 2018

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jul 13, 2018

comment:2

You can set this to a positive review if upstream accepts the PR.

@embray
Copy link
Contributor Author

embray commented Jul 18, 2018

comment:3

I believe this issue can reasonably be addressed for Sage 8.4.

@embray embray modified the milestones: sage-8.3, sage-8.4 Jul 18, 2018
@embray
Copy link
Contributor Author

embray commented Jul 23, 2018

comment:4

Got confirmation from Odile that this patch works.

If possible I'd like to get this in 8.3 actually, but admittedly adding a patch to MPIR is a big rebuild, and it also affects only a few rare machines and has an easy workaround e.g. building MPIR with --build=skylake or some other earlier architecture.

@embray embray modified the milestones: sage-8.4, sage-8.3 Jul 23, 2018
@tscrim
Copy link
Collaborator

tscrim commented Jul 23, 2018

comment:5

I am much more split on this ticket (than the one that only affects Cygwin) as it could have more unintended side-effects. I am slightly on the side of waiting until 8.4 and having this go through beta cycles. If we get posts on sage-devel, you have at least seen the error and know the workaround. Plus this will only affect people building it from source on these rare machines, so the chances of someone being afflicted are marginal.

@embray
Copy link
Contributor Author

embray commented Jul 23, 2018

Changed upstream from Reported upstream. Developers acknowledge bug. to Fixed upstream, but not in a stable release.

@embray
Copy link
Contributor Author

embray commented Jul 23, 2018

comment:6

I agree, it's pretty marginal.

@vbraun
Copy link
Member

vbraun commented Aug 5, 2018

Changed branch from u/embray/ticket-25858 to 50c1e0a

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

No branches or pull requests

3 participants