Re: [PATCH 5/7] accel/ivpu: Abort all jobs after command queue unregister

[Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>]

Hi,

On 5/5/2025 8:03 AM, Greg KH wrote:
> On Wed, Apr 30, 2025 at 02:48:17PM +0200, Jacek Lawrynowicz wrote:
>> From: Karol Wachowski <karol.wachowski@intel.com>
>>
>> commit 5bbccadaf33eea2b879d8326ad59ae0663be47d1 upstream.
>>
>> With hardware scheduler it is not expected to receive JOB_DONE
>> notifications from NPU FW for the jobs aborted due to command queue destroy
>> JSM command.
>>
>> Remove jobs submitted to unregistered command queue from submitted_jobs_xa
>> to avoid triggering a TDR in such case.
>>
>> Add explicit submitted_jobs_lock that protects access to list of submitted
>> jobs which is now used to find jobs to abort.
>>
>> Move context abort procedure to separate work queue not to slow down
>> handling of IPCs or DCT requests in case where job abort takes longer,
>> especially when destruction of the last job of a specific context results
>> in context release.
>>
>> Cc: <stable@vger.kernel.org> # v6.12
>> Signed-off-by: Karol Wachowski <karol.wachowski@intel.com>
>> Signed-off-by: Maciej Falkowski <maciej.falkowski@linux.intel.com>
>> Reviewed-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
>> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
>> Link: https://patchwork.freedesktop.org/patch/msgid/20250107173238.381120-4-maciej.falkowski@linux.intel.com
>> ---
>>  drivers/accel/ivpu/ivpu_drv.c   | 32 +++----------
>>  drivers/accel/ivpu/ivpu_drv.h   |  2 +
>>  drivers/accel/ivpu/ivpu_job.c   | 82 +++++++++++++++++++++++++--------
>>  drivers/accel/ivpu/ivpu_job.h   |  1 +
>>  drivers/accel/ivpu/ivpu_mmu.c   |  3 +-
>>  drivers/accel/ivpu/ivpu_sysfs.c |  5 +-
>>  6 files changed, 77 insertions(+), 48 deletions(-)
> 
> Again, this is different from the original, so please document it as
> such.
> 
> Please fix up both backported series of patches and resubmit a v2 of
> them.

Sure, I've added descriptions of changes to commit messages and resubmitted the patchsets as v2.

Regards