Hi Joerg, 在 2021/5/18 17:08, Joerg Roedel 写道:
On Mon, Apr 19, 2021 at 03:13:35PM +0800, chenxiang wrote:
From: Xiang Chen <chenxiang66@hisilicon.com>
It is not necessary to put free_iova_mem() inside of spinlock/unlock iova_rbtree_lock which only leads to more completion for the spinlock. It has a small promote on the performance after the change. And also rename private_free_iova() as remove_iova() because the function will not free iova after that change.
Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com> --- drivers/iommu/iova.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) Looks good to me, but I'll wait for Robins opinion before applying.
I have re-sent v3, and Robin has given his acked-by: https://www.mail-archive.com/iommu@lists.linux-foundation.org/msg50950.html
.