Opened 4 years ago

Closed 18 months ago

#23933 closed defect (invalid)

OpenBLAS randomly crashes / deadlocks 2

Reported by: mderickx Owned by:
Priority: critical Milestone: sage-duplicate/invalid/wontfix
Component: linear algebra Keywords: patchbot
Cc: fbissey, dimpase Merged in:
Authors: Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

The first of the two bugs at #22021 popped up again in sage 8.1.beta6 on arando

sage -t --long src/sage/matrix/matrix_integer_dense.pyx
    Killed due to segmentation fault
**********************************************************************
Tests run before process (pid=3193) failed:
sage: a = matrix(ZZ, 3,3, range(9)); a ## line 23 ##
[0 1 2]
[3 4 5]
[6 7 8]
sage: a.det() ## line 27 ##
0
...
12
sage: A*C == d*B ## line 4265 ##
True
sage: A = random_matrix(ZZ, 4, 4) ## line 4270 ##
sage: B = random_matrix(ZZ, 2, 3) ## line 4271 ##
sage: B._solve_iml(A) ## line 4272 ##
sage: A._solve_iml(B, right=False) ## line 4276 ##
sage: A._solve_iml(B, right=True) ## line 4280 ##
sage: A = random_matrix(ZZ, 2000, 2000) ## line 4287 ##
sage: B = random_matrix(ZZ, 2000, 2000) ## line 4288 ##
sage: t0 = walltime() ## line 4289 ##
sage: alarm(2); A._solve_iml(B)  # long time ## line 4290 ##
------------------------------------------------------------------------
sage: t = walltime(t0) ## line 4294 ##

Change History (11)

comment:1 Changed 4 years ago by fbissey

  • Cc fbissey added

comment:2 Changed 4 years ago by mderickx

  • Keywords patchbot added

comment:3 Changed 4 years ago by embray

I think maybe this has been fixed upstream by https://github.com/xianyi/OpenBLAS/pull/1234 . I'm going to test it.

comment:4 Changed 4 years ago by jdemeyer

  • Dependencies set to #24638

Beware that #24638 also patches openblas.

comment:5 Changed 4 years ago by embray

Apparently we've already upgraded to OpenBLAS 0.2.20 which appears to include the fix. Then the only thing to do would be to drop the USE_THREAD=0 from the OpenBLAS build.

comment:6 Changed 4 years ago by embray

Seems to work on my Ubuntu machine. Indeed, this ticket was reported against 8.1.beta6 which was still on OpenBLAS 0.2.19, but this issue was only fixed in OpenBLAS 0.2.20

comment:7 Changed 4 years ago by embray

  • Dependencies #24638 deleted

comment:8 Changed 4 years ago by embray

If #24669 gets merged and works for people then this can just be closed as worksforme.

comment:9 Changed 18 months ago by mkoeppe

  • Cc dimpase added
  • Milestone changed from sage-8.1 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

Proposing to close this ticket as outdated -- we have updated OpenBLAS several times since this report, and also openblas/spkg-configure.m4 requires version >= 0.2.20 which fixes it as per the above.

comment:10 Changed 18 months ago by dimpase

  • Reviewers set to Dima Pasechnik
  • Status changed from needs_review to positive_review

indeed

comment:11 Changed 18 months ago by chapoton

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.