Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 852236

Summary: sci-libs/rocBLAS-5.0.2 doesn't respect MAKEOPTS
Product: Gentoo Linux Reporter: Alessandro Barbieri <lssndrbarbieri>
Component: Current packagesAssignee: Gentoo Science Related Packages <sci>
Status: RESOLVED FIXED    
Severity: normal CC: gentoo, xgreenlandforwyy
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=822828
https://github.com/gentoo/gentoo/pull/26285
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 654178    

Description Alessandro Barbieri 2022-06-15 23:01:14 UTC
if I set MAKEOPTS="-j1" it will launch 12 jobs and it will later crash my system because all RAM and swap are full
I want to set less jobs to consume less memory
Comment 1 Alessandro Barbieri 2022-06-21 01:10:15 UTC
They added Tensile_CPU_THREADS in version 5.1.0
https://github.com/ROCmSoftwarePlatform/rocBLAS/commit/0ffee0626bf987bae6b81183a5c61569a489537a

also here https://rocblas.readthedocs.io/en/rocm-5.0.2/install.html#prerequisites
they say:

"As a general rule, 64GB of system memory is required for a full rocBLAS build. This value can be lower if rocBLAS is built with a different Tensile logic target (see the –logic command for ./install.sh). This value may also increase in the future as more functions are added to rocBLAS and dependencies such as Tensile grow."
Comment 2 Yiyang Wu 2022-06-21 03:55:06 UTC
(In reply to Alessandro Barbieri from comment #1)
> They added Tensile_CPU_THREADS in version 5.1.0
> https://github.com/ROCmSoftwarePlatform/rocBLAS/commit/
> 0ffee0626bf987bae6b81183a5c61569a489537a
> 
> also here
> https://rocblas.readthedocs.io/en/rocm-5.0.2/install.html#prerequisites
> they say:
> 
> "As a general rule, 64GB of system memory is required for a full rocBLAS
> build. This value can be lower if rocBLAS is built with a different Tensile
> logic target (see the –logic command for ./install.sh). This value may also
> increase in the future as more functions are added to rocBLAS and
> dependencies such as Tensile grow."

Thanks for the information. I'll try backporting this commit to dev-util/Tensile and sci-libs/rocBLAS version 5.0.2.
Comment 3 Larry the Git Cow gentoo-dev 2022-07-08 17:29:30 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=75f2ea4157fc3a52fa139b90db1890371dc3f6f7

commit 75f2ea4157fc3a52fa139b90db1890371dc3f6f7
Author:     Yiyang Wu <xgreenlandforwyy@gmail.com>
AuthorDate: 2022-07-08 13:01:54 +0000
Commit:     Benda Xu <heroxbd@gentoo.org>
CommitDate: 2022-07-08 17:28:31 +0000

    sci-libs/rocBLAS: respect MAKEOPTS when build Tensile libs
    
    Closes: https://github.com/gentoo/gentoo/pull/26285
    Closes: https://bugs.gentoo.org/852236
    Suggested-by: Alessandro Barbieri <lssndrbarbieri@gmail.com>
    Signed-off-by: Yiyang Wu <xgreenlandforwyy@gmail.com>
    Signed-off-by: Benda Xu <heroxbd@gentoo.org>

 .../files/rocBLAS-5.0.2-respect-makeopts.patch     | 73 ++++++++++++++++++++++
 ...ocBLAS-5.0.2.ebuild => rocBLAS-5.0.2-r1.ebuild} |  6 +-
 2 files changed, 77 insertions(+), 2 deletions(-)