Claude review: accel/habanalabs: fix refcount leak in hl_direct_io()

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: accel/habanalabs: fix refcount leak in hl_direct_io()
Author: Wentao Liang <vulab@iscas.ac.cn>
Patches: 1
Reviewed: 2026-06-04T11:49:49.715054

---

This is a single-patch series that fixes a refcount leak in the `hl_direct_io()` function in the habanalabs NVMe Direct I/O path. The fix is **correct and appropriate**.

When `hl_dio_get_iopath()` succeeds (line 235), it acquires two resources: it increments a per-CPU inflight I/O counter and takes a reference on the context via `hl_ctx_get()` (see `hl_dio_get_iopath()` at line 162). The corresponding cleanup function `hl_dio_put_iopath()` (line 170-176) releases both: it calls `hl_ctx_put()` and decrements the per-CPU counter.

Before this patch, if the `vzalloc()` at line 250 failed, the function returned `-ENOMEM` directly without calling `hl_dio_put_iopath()`, leaking both the context reference and the inflight I/O count. The existing `cleanup` label at line 274 already calls both `vfree(io->bv)` and `hl_dio_put_iopath(io->f.ctx)`, and `vfree(NULL)` is safe, so jumping there would have been an alternative — but the inline fix is equally correct and arguably clearer since it matches the early-return style already in use at this point in the function.

The series is minimal, well-targeted, and appropriate for stable backport.

---
Generated by Claude Code Patch Reviewer