Re: FAILED: patch "[PATCH] iommufd: Fail replace if device has not been attached" failed to apply to 6.14-stable tree

[Yi Liu <yi.l.liu@intel.com>]

On 2025/4/17 22:12, Greg KH wrote:
> On Thu, Apr 17, 2025 at 08:52:16PM +0800, Yi Liu wrote:
>> On 2025/4/17 19:42, gregkh@linuxfoundation.org wrote:
>>>
>>> The patch below does not apply to the 6.14-stable tree.
>>> If someone wants it applied there, or to any other stable or longterm
>>> tree, then please email the backport, including the original git commit
>>> id to <stable@vger.kernel.org>.
>>>
>>> To reproduce the conflict and resubmit, you may use the following commands:
>>>
>>> git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-6.14.y
>>> git checkout FETCH_HEAD
>>> git cherry-pick -x 55c85fa7579dc2e3f5399ef5bad67a44257c1a48
>>> # <resolve conflicts, build, test, etc.>
>>> git commit -s
>>> git send-email --to '<stable@vger.kernel.org>' --in-reply-to '2025041759-knee-unearned-530e@gregkh' --subject-prefix 'PATCH 6.14.y' HEAD^..
>>>
>>> Possible dependencies:
>>
>> I think the possible dependency is the below commit. This patch adds a
>> helper before iommufd_hwpt_attach_device() which is added by below commit.
>>
>> commit fb21b1568adaa76af7a8c853f37c60fba8b28661
>> Author: Nicolin Chen <nicolinc@nvidia.com>
>> Date:   Mon Feb 3 21:00:54 2025 -0800
>>
>>      iommufd: Make attach_handle generic than fault specific
>>
>>      "attach_handle" was added exclusively for the iommufd_fault_iopf_handler()
>>      used by IOPF/PRI use cases. Now, both the MSI and PASID series require to
>>      reuse the attach_handle for non-fault cases.
>>
>>      Add a set of new attach/detach/replace helpers that does the attach_handle
>>      allocation/releasing/replacement in the common path and also handles those
>>      fault specific routines such as iopf enabling/disabling and auto response.
>>
>>      This covers both non-fault and fault cases in a clean way, replacing those
>>      inline helpers in the header. The following patch will clean up those old
>>      helpers in the fault.c file.
>>
>>      Link: https://patch.msgid.link/r/32687df01c02291d89986a9fca897bbbe2b10987.1738645017.git.nicolinc@nvidia.com
>>      Signed-off-by: Nicolin Chen <nicolinc@nvidia.com>
>>      Reviewed-by: Yi Liu <yi.l.liu@intel.com>
>>      Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
>>
>> diff --git a/drivers/iommu/iommufd/device.c b/drivers/iommu/iommufd/device.c
>> index dfd0898fb6c1..0786290b4056 100644
>> --- a/drivers/iommu/iommufd/device.c
>> +++ b/drivers/iommu/iommufd/device.c
>> @@ -352,6 +352,111 @@ iommufd_device_attach_reserved_iova(struct
>> iommufd_device *idev,
>>          return 0;
>>   }
>>
>> +/* The device attach/detach/replace helpers for attach_handle */
>> +
>> +static int iommufd_hwpt_attach_device(struct iommufd_hw_pagetable *hwpt,
>> +                                     struct iommufd_device *idev)
>> +{
>> +       struct iommufd_attach_handle *handle;
>> +       int rc;
>> +
>> +       lockdep_assert_held(&idev->igroup->lock);
>>
>>
>> @Greg, anything I need to do here?
> 
> That should be it, thanks!
> 


you are welcome. For 6.6, it might be difficult to apply all dependencies.
I've posted a patch based on 6.6. Please let me know if it is not preferred.

-- 
Regards,
Yi Liu