Claude review: drm/syncobj: Fix xa_alloc allocation flags

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

Patch Review

**Correctness: Good.** Both call sites (`drm_syncobj_get_handle` at line 604 and `drm_syncobj_fd_to_handle` at line 718) are in process/ioctl context with no spinlocks held, so `GFP_KERNEL` is safe to use.

The commit message clearly explains the rationale: `idr_preload(GFP_KERNEL)` + `idr_alloc(..., GFP_NOWAIT)` was a standard IDR pattern where preload pre-allocated memory allowing the subsequent alloc to use `GFP_NOWAIT`. The xarray conversion removed `idr_preload` but kept `GFP_NOWAIT`, meaning `xa_alloc` could fail under memory pressure where the old code would have succeeded.

**Tags:** The `Fixes:` tag correctly references the commit that introduced the regression. `Reported-by` and `Cc` tags are present.

**Minor nit:** The `Fixes:` tag is placed after `Signed-off-by:`, which is unconventional — it's more common to place `Fixes:` before `Signed-off-by:`. This is a stylistic preference and doesn't affect anything functionally.

**No issues found.** The patch is clean, minimal, and correct.

**Reviewed-by worthy:** Yes.

---
Generated by Claude Code Patch Reviewer