Claude review: drm/imagination: Don't timeout job if its fence has been signaled

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

Patch Review

**Verdict: Bug — NULL pointer dereference**

The added check:

```c
if (dma_fence_is_signaled(s_job->s_fence->parent))
    return DRM_GPU_SCHED_STAT_NO_HANG;
```

`s_job->s_fence->parent` can be NULL. The parent fence is assigned by `drm_sched_fence_set_parent()` during `run_job`. If the timeout fires before the job has been run (e.g., the job was stuck in the pending queue), `parent` is NULL. This will cause a NULL dereference inside `dma_fence_is_signaled()`, which does not check for NULL.

The code later in the same function even has:

```c
WARN_ON(job->base.s_fence->parent);
job->base.s_fence->parent = dma_fence_get(job->done_fence);
```

...which explicitly expects `parent` to be NULL in some cases.

**Fix:** Add a NULL check:

```c
if (s_job->s_fence->parent &&
    dma_fence_is_signaled(s_job->s_fence->parent))
    return DRM_GPU_SCHED_STAT_NO_HANG;
```

The overall idea (avoid false timeouts when the fence has already signaled) is reasonable and used in other DRM scheduler drivers.

---
Generated by Claude Code Patch Reviewer