Skip to content

TensorRT Cuda Context Sharing#106

Open
ashishk98 wants to merge 17 commits intotriton-inference-server:mainfrom
ashishk98:main
Open

TensorRT Cuda Context Sharing#106
ashishk98 wants to merge 17 commits intotriton-inference-server:mainfrom
ashishk98:main

Conversation

@ashishk98
Copy link
Copy Markdown
Contributor

This PR enabled cuda context sharing with CiG streams on Windows Gaming applications

Comment thread CMakeLists.txt
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This change is Windows only as per my understanding, CMakeLists.txt doesn't reflect it and it cause a confusion.
Is it something that we consider to build by default then configuration change should reflect it accordingly.

Comment thread src/instance_state.cc
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Files in src folder may be related not only to Windows but other platforms as well. We need to make sure it doesn't impact the container build.

@GuanLuo are we OK with it?

@saturley-hall have we tested it well in CI for other stacks are we confident with result?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants