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

Enable CUDA kernels compression in the final binary #2833

Merged
merged 1 commit into from
Apr 1, 2021

Conversation

JanuszL
Copy link
Contributor

@JanuszL JanuszL commented Apr 1, 2021

Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com

Why we need this PR?

Pick one, remove the rest

  • It adds CUDA kernels compression in the final binary

What happened in this PR?

Fill relevant points, put NA otherwise. Replace anything inside []

  • What solution was applied:
    enables CUDA kernels compression in the final binary
  • Affected modules and functionalities:
    main CMakeLists.txt
  • Key points relevant for the review:
    NA
  • Validation and testing:
    current test applies
  • Documentation (including examples):
    NA

JIRA TASK: [NA]

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL
Copy link
Contributor Author

JanuszL commented Apr 1, 2021

!build

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [2225520]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [2225520]: BUILD FAILED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [2225520]: BUILD PASSED

@JanuszL
Copy link
Contributor Author

JanuszL commented Apr 1, 2021

It shrinks libdali_operators.so from 600MB to 200MB.

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [2226178]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [2226178]: BUILD PASSED

@JanuszL
Copy link
Contributor Author

JanuszL commented Apr 1, 2021

For cuda 11 it is 900MB to 300MB.

@JanuszL JanuszL merged commit 308fddc into NVIDIA:master Apr 1, 2021
@JanuszL JanuszL deleted the kernels_compression branch April 1, 2021 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants