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 <[email protected]>
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 <[email protected]>
