Claude review: drm/amdkfd: convert CRIU event-private allocation to kvcalloc()

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

Patch Review

**Status: Clean / No issues**

The change at `drivers/gpu/drm/amd/amdkfd/kfd_events.c:551`:

```c
- ev_privs = kvzalloc(num_events * sizeof(*ev_privs), GFP_KERNEL);
+ ev_privs = kvcalloc(num_events, sizeof(*ev_privs), GFP_KERNEL);
```

**Correctness:**
- `kvcalloc(n, size, flags)` is functionally equivalent to `kvzalloc(n * size, flags)` but with overflow protection. Both zero the allocated memory. The semantics are preserved.
- `num_events` is a `uint32_t` and `sizeof(*ev_privs)` is a `size_t`, so the multiplication *could* overflow on 32-bit platforms with a sufficiently large `num_events` value. In practice `num_events` comes from `kfd_get_num_events()` which counts IDR entries, so it's bounded by resource limits, but the overflow-safe API is still the right thing to use.
- The `kvfree()` at line 603 remains correct for memory allocated by either `kvzalloc` or `kvcalloc`.

**Observation:** Lines 595 and 601 still use `num_events * sizeof(*ev_privs)` in the open-coded form (for `copy_to_user` and the offset bump). These are not allocation calls so `kvcalloc` doesn't apply, but if the author wanted to be thorough about overflow protection, those multiplications could also use `size_mul()`. This is not a blocker — it's a separate concern and arguably overkill for a size that was already validated at allocation time.

**Commit message:** Accurate and well-written. Correctly notes "no functional change."

Reviewed-by: would be appropriate here.

---
Generated by Claude Code Patch Reviewer