Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 852236 - sci-libs/rocBLAS-5.0.2 doesn't respect MAKEOPTS
Summary: sci-libs/rocBLAS-5.0.2 doesn't respect MAKEOPTS
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Related Packages
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks: makeopts-ignored
  Show dependency tree
 
Reported: 2022-06-15 23:01 UTC by Alessandro Barbieri
Modified: 2022-07-08 17:29 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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(-)