Skip to content

Sahar/tdr failure #698

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

Merged
merged 4 commits into from
Jun 5, 2025
Merged

Sahar/tdr failure #698

merged 4 commits into from
Jun 5, 2025

Conversation

sfatimar
Copy link

Description

Handle Exceptions in Parallel Execution with OpenVINO Execution Provider

Motivation and Context

Once a TDR Occurs a Inference request must be cancelled and removed from queue
Perf test was experiencing a race condition as lock mutex was not out of same thread pool

  • If it fixes an open issue, please link to the issue here. -->

@sfatimar sfatimar requested review from MayureshV1 and vthaniel June 2, 2025 06:33
@sfatimar
Copy link
Author

sfatimar commented Jun 2, 2025

@MayureshV1 please review

@@ -294,12 +262,16 @@ void OVInferRequest::Infer() {
}

void OVInferRequest::WaitRequest() {
ovInfReq.wait();

Choose a reason for hiding this comment

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

We should have an exception handling for waitRequest as well

Copy link

@preetha-intel preetha-intel left a comment

Choose a reason for hiding this comment

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

LGTM

@preetha-intel preetha-intel merged commit be8f8be into ovep-develop Jun 5, 2025
6 of 8 checks passed
@@ -459,150 +439,46 @@ void BasicBackend::StartAsyncInference(Ort::KernelContext& context, OVInferReque
}
}

#ifdef IO_BUFFER_ENABLED

Choose a reason for hiding this comment

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

@sfatimar . Why is IO Buffer Optimization removed ?

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