Related-To: NEO-13225
Replace blocking obtainUniqueOwnership with non-blocking
tryObtainUniqueOwnership in isDirectSubmissionIdle.
Treat compute engine as not idle when lock is contended
(conservative; prevents false stops).
Avoid deadlock scenarios caused by holding
directSubmissionsMutex while waiting on a CSR lock.
Signed-off-by: Slawomir Milczarek <slawomir.milczarek@intel.com>
Noticed an issue on machine with following `sycl-ls`:
When running `sycl-ls` with a recent debug build of NEO, it exposed a
heap alignment problem:
```shell
Assert was called at 324 line in file:
../../src/neo/shared/source/memory_manager/gfx_partition.cpp
```
Related-To: NEO-16287
Signed-off-by: Jack Myers <jack.myers@intel.com>
Related-To: NEO-13325
Replace blocking obtainUniqueOwnership with non-blocking
tryObtainUniqueOwnership in isCopyEngineOnDeviceIdle.
Treat copy engine as not idle when lock is contended
(conservative; prevents false stops).
Avoid deadlock scenarios caused by holding
directSubmissionsMutex while waiting on a CSR lock.
Signed-off-by: Slawomir Milczarek <slawomir.milczarek@intel.com>
When relocation points to symbol that is not defined within module mark
it as optional. When symbol is available at dynamic linking time then
info from the function is retrieved but when the symbol is not available
then ignore the dependency.
Any unresolved symbol needed for module linking is already handled
in a separate place.
Related-To: NEO-16243, NEO-16263, NEO-16262, NEO-16268
Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>
Related-To: NEO-13325
Replace blocking obtainUniqueOwnership() with tryObtainUniqueOwnership()
in critical paths to prevent deadlock between controller thread holding
directSubmissionsMutex and submission threads holding CSR locks.
Changes:
- Add CommandStreamReceiver::tryObtainUniqueOwnership() method
- Use try_lock in checkNewSubmissions() and context group idle detection
- Skip contended CSRs instead of blocking (conservative approach)
- Add comprehensive unit tests for try_lock functionality
Fixes deadlock scenario where registerDirectSubmission() waits for
directSubmissionsMutex while checkNewSubmissions() holds it and waits
for CSR ownership locks.
Signed-off-by: Slawomir Milczarek <slawomir.milczarek@intel.com>
This test naively compares estimated blit command stream sizes instead
of actually checking if a command stream is correctly programmed.
The tested functionality is already covered by
givenMultiRootDeviceSyncNodeWhenFlushBcsTaskThenMiFlushAdded.
Signed-off-by: Radoslaw Jablonski <radoslaw.jablonski@intel.com>
On new append calls, reset flag if previous submissions
are completed.
Related-To: NEO-16184
Signed-off-by: Bellekallu Rajkiran <bellekallu.rajkiran@intel.com>
The usage of host function allocation is similar to tagBuffer.
Ensure it's uncached and it follows tag buffer's cache policy.
Host function allocation will be readable and writable
by both: cpu and gpu.
Related-To: NEO-14577
Signed-off-by: Kamil Kopryk <kamil.kopryk@intel.com>
Related-To: NEO-13715
Provides implmentation of getAtomicAccessAttribute using system
allocator.
Completes the implementation of the madvise feature using the
new IOCTLs in xe_drm header.
Signed-off-by: Chandio, Bibrak Qamar <bibrak.qamar.chandio@intel.com>
in case of AOT kernels compiled with older encoder version
printf calls are present when printf buffer arg is present
Related-To: NEO-15211
Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>
