Hi Rahul,

On 15/02/2021 09:12, Rahul Singh wrote:
Thanks for the testing. I noticed that my patch doesn't build because 
arm_iommu_unmap_page() doesn't have a parameter mfn. Can you confirm whether 
you had to replace mfn with _mfn(dfn_x(dfn))?

Yes I have to replace the mfn with _mfn(dfn_x(dfn)) to test the patch.

Thanks.

In the future, I would suggest to mention such things in your testing.

This will be easier to figure out how one managed to test it and whether the fix was the same.

Cheers,

--
Julien Grall

Reply via email to