Skip to content

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

}

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
}
}

#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 ?

ankitm3k pushed a commit that referenced this pull request Jul 2, 2025
* Catch exception with TDR

* Handle exceptions during parallel execution with OVEP

* Remove IO Buffer Implementation

---------

Co-authored-by: TejalKhade28 <tejal.khade@intel.com>
Co-authored-by: Preetha Veeramalai <preetha.veeramalai@intel.com>
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.

5 participants