[maleadt]

The buffer that resize! allocates was freed by a finalizer calling free(buf) instead of release(buf), as the array constructor does. That bypassed the allocation accounting used by the proactive-GC heuristics, the queue synchronization on the LTS stack that keeps NEO from unmapping memory with work in flight, and the blocking free policy. It also broke arrays resized to zero length: release skips empty buffers, while free passed the null pointer to zeMemFree and threw ZE_RESULT_ERROR_INVALID_ARGUMENT.

Fixes #667

[github-actions]

Your PR requires formatting changes to meet the project's style guidelines.
Please consider running Runic (git runic main) to apply these changes.

Click here to view the suggested changes.

diff --git a/test/array.jl b/test/array.jl
index c09cfbd..e2b714c 100644
--- a/test/array.jl
+++ b/test/array.jl
@@ -210,21 +210,21 @@ end
   resize!(b, 1)
   @test length(b) == 1
 
-  # resized buffers are released through the pool, like freshly allocated ones
-  c = oneArray{Float32}(undef, 4)
-  resize!(c, 0)
-  oneAPI.unsafe_free!(c)
-
-  GC.enable(false)
-  try
-    d = oneArray{Float32}(undef, 1024)
-    before = oneAPI._allocated_bytes[]
-    resize!(d, 1 << 20)
-    oneAPI.unsafe_free!(d)
-    @test oneAPI._allocated_bytes[] == before - sizeof(Float32) * 1024
-  finally
-    GC.enable(true)
-  end
+    # resized buffers are released through the pool, like freshly allocated ones
+    c = oneArray{Float32}(undef, 4)
+    resize!(c, 0)
+    oneAPI.unsafe_free!(c)
+
+    GC.enable(false)
+    try
+        d = oneArray{Float32}(undef, 1024)
+        before = oneAPI._allocated_bytes[]
+        resize!(d, 1 << 20)
+        oneAPI.unsafe_free!(d)
+        @test oneAPI._allocated_bytes[] == before - sizeof(Float32) * 1024
+    finally
+        GC.enable(true)
+    end
 end
 
 # Reductions and scans come from GPUArrays (AcceleratedKernels). oneAPI used to work around

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.58%. Comparing base (fa0f23a) to head (15bd554).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #684      +/-   ##
==========================================
- Coverage   81.17%   79.58%   -1.59%     
==========================================
  Files          52       51       -1     
  Lines        4085     3953     -132     
==========================================
- Hits         3316     3146     -170     
- Misses        769      807      +38     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.