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.

Sign up for GitHub

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

Jump to bottom

[GPU] Release ov::Model from memory once it's no longer needed #29786

Open
wants to merge 1 commit into
base: releases/2025/1
Choose a base branch
from
Open

[GPU] Release ov::Model from memory once it's no longer needed #29786

wants to merge 1 commit into from
+1 −0

Conversation

tkrupa-intel
Copy link
Contributor

@tkrupa-intel tkrupa-intel commented Mar 27, 2025 •
edited
Loading

Details:

Clean build:

  1st run [MB] 2nd run [MB]
WSS after pipeline creation 2034 2729
PWSS after pipeline creation 1904 1719
WSS after pipe.generate() 2078 2818
PWSS after pipe.generate() 1934 1784

After change:

  1st run [MB] 2nd run [MB]
WSS after pipeline creation 2033 2013
PWSS after pipeline creation 1904 1875
WSS after pipe.generate() 2078 2056
PWSS after pipe.generate() 1948 1918

Tickets:

@tkrupa-intel tkrupa-intel requested review from a team as code owners March 27, 2025 14:28
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Mar 27, 2025
@tkrupa-intel tkrupa-intel force-pushed the gpu_cache_with_ov_model_fix branch from f6d4f75 to ab4077c Compare March 27, 2025 14:33
@tkrupa-intel tkrupa-intel mentioned this pull request Mar 27, 2025
Open
@yeonbok
Copy link
Contributor

yeonbok commented Mar 27, 2025

build_jenkins

@Dmi3shin Dmi3shin requested review from geunhwan and Dmi3shin March 28, 2025 13:05
@sammysun0711
Copy link
Collaborator

sammysun0711 commented Mar 28, 2025 •
edited
Loading

This PR is not ready and not fix original issue, we should merge #29805 in releases/2025/1 branch only.

Dmi3shin
Dmi3shin requested changes Mar 28, 2025
View reviewed changes
Copy link

@Dmi3shin Dmi3shin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Subjected PR won't fix the issue, we need this PR instead:
#29805

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dmi3shin Dmi3shin Dmi3shin requested changes

@geunhwan geunhwan Awaiting requested review from geunhwan

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
category: GPU OpenVINO GPU plugin do not merge do_not_merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tkrupa-intel @yeonbok @sammysun0711 @Dmi3shin