[PATCH] accel/ivpu: Flush pending jobs of device's workqueues

[Jeff Hugo]

On 4/1/2025 9:57 AM, Maciej Falkowski wrote:
> Use flush_work() instead of cancel_work_sync() for driver
> workqueues to guarantee that remaining pending work
> will be handled.
> 
> Fixes: bc3e5f48b7ee ("accel/ivpu: Use workqueue for IRQ handling")
> Signed-off-by: Maciej Falkowski <maciej.falkowski at linux.intel.com>

The fixes tag suggests that this is a bug fix, but the commit text 
doesn't really tell me why cancel_work_sync() is bad.  Does it leave the 
driver in some incomplete state that prevents later recovery?

Put another way, "why is it necessary for the pending work to be 
handled?" appears unanswered.

With that
Reviewed-by: Jeff Hugo <jeff.hugo at oss.qualcomm.com>