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

cubin files not cleaned up after linking #49680

Closed
ye-luo opened this issue May 14, 2021 · 6 comments
Closed

cubin files not cleaned up after linking #49680

ye-luo opened this issue May 14, 2021 · 6 comments
Labels
bugzilla Issues migrated from bugzilla openmp

Comments

@ye-luo
Copy link
Contributor

ye-luo commented May 14, 2021

Bugzilla Link 50336
Resolution FIXED
Resolved on Oct 11, 2021 20:29
Version unspecified
OS Linux
Blocks #51489
CC @jhuber6,@jdoerfert
Fixed by commit(s) 01d59c0 0dd4f00

Extended Description

tons of cubin files constantly fill /tmp

...
-rw-rw-r-- 1 yeluo yeluo 39056 May 13 15:22 RPAJastrow-6b9e81.cubin
-rw-rw-r-- 1 yeluo yeluo 39056 May 13 15:22 eeI_JastrowBuilder-f4a8b1.cubin
...

I think these files are created when object files are unbundled for linking but never got removed from disk up completion.

@jhuber6
Copy link
Contributor

jhuber6 commented Aug 6, 2021

I believe I fixed the problem in https://reviews.llvm.org/D107668, can you verify? Not sure how to make a test for this.

What was happening was two-phase builds were going through the offload-bundler tool to create a fat-binary. The NVPTX assembler creates a .cubin file that is used, but its name isn't present in the list of files seen by -ccc-print-phases so it was not being removed automatically.

@jdoerfert
Copy link
Member

Do we need to rebase this to 13?

@jhuber6
Copy link
Contributor

jhuber6 commented Aug 11, 2021

It wasn't originally listed as a release blocker. It's definitely a bug and should be probably be fixed in the 13.0 release. I'll put it as a blocker.

@ye-luo
Copy link
Contributor Author

ye-luo commented Aug 11, 2021

The commit 01d59c0
on main does resolve this issue. I'm happy that I no more needs to constantly cleaning the /tmp on my CI machine.

It is definitely super helpful to get release 13 fixed as well.
Reopen until it gets applied on 13 release branch.

@tstellar
Copy link
Collaborator

Merged: 0dd4f00

@tstellar
Copy link
Collaborator

mentioned in issue #51489

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla openmp
Projects
None yet
Development

No branches or pull requests

4 participants